Hi Victor<small>,</small><br /><br /><div class="moz-cite-prefix">On 2/1/16 5:05 PM, Alvaro Herrera wrote:<br
/></div><blockquotecite="mid:%3C20160201220540.GA99599@alvherre.pgsql%3E" type="cite"><pre wrap="">Victor Wagner
wrote:
</pre><blockquote type="cite"><pre wrap="">On Fri, 22 Jan 2016 16:36:15 -0300
Alvaro Herrera <a class="moz-txt-link-rfc2396E"
href="mailto:alvherre@2ndquadrant.com"><alvherre@2ndquadrant.com></a>wrote:
</pre><blockquote type="cite"><pre wrap="">You're editing the expected file for the libpq-regress thingy, but you
haven't added any new lines to test the new capability. I think it'd
be good to add some there. (I already said this earlier in the
thread; is there any reason you ignored it the first time?)
</pre></blockquote><pre wrap="">
I seriously doubt that this program can be used to test new
capabilities.
All it does, it calls PQconninfoParse and than examines some fields of
PGconn structure.
</pre></blockquote><pre wrap="">
Ah, you're right, I didn't remember that.
</pre><blockquote type="cite"><pre wrap="">If I add some new uris, than only thing I can test is that comma is
properly copied from the URI to this field. And may be that some syntax
errors are properly detected.
</pre></blockquote><pre wrap="">
Yeah, we should do that.
</pre><blockquote type="cite"><pre wrap="">So, I think that new functionality need other approach for testing.
There should be test of real connection to real temporary cluster.
Probably, based on Perl TAP framework which is actively used in the
Postgres recently.
</pre></blockquote><pre wrap="">
Yes, agreed. So please have a look at that one and share your opinion
about it. It'd be useful.
Meanwhile I'm moving the patch to the next commitfest.
</pre></blockquote><br /> There hasn't been any movement on this patch in a while. Will you have a new tests ready for
reviewsoon? <br /><br /> Thanks, <br /><pre class="moz-signature" cols="72">--
-David
<a class="moz-txt-link-abbreviated" href="mailto:david@pgmasters.net">david@pgmasters.net</a></pre>