Re: psql - improve test coverage from 41% to 88% - Mailing list pgsql-hackers

From vignesh C
Subject Re: psql - improve test coverage from 41% to 88%
Date
Msg-id CALDaNm0Z_ntaxCc-prkdYFzgDPYjmxcTxxX2pDeOUZX_hBFD2Q@mail.gmail.com
Whole thread Raw
In response to Re: psql - improve test coverage from 41% to 88%  (Fabien COELHO <coelho@cri.ensmp.fr>)
Responses Re: psql - improve test coverage from 41% to 88%
List pgsql-hackers
On Thu, Sep 12, 2019 at 2:15 PM Fabien COELHO <coelho@cri.ensmp.fr> wrote:
>
>
> >> Ok. Rebased version added, with some minor changes to improve readability
> >> (comments, variables).
> >
> > Few comments: [...]
> >
> > Commented line can be removed
> > Commented lines can be removed
> > ??? can be changed to some suitable heading
> > tab-complation to be changed to tab-completion
> > Commented lines can be removed, some more are present below these lines also.
>
> Thanks for this review.
>
> The lines were really tests I did that had some issues because of the way
> the Expect module works, and are not useful for inclusion in the code
> base.
>
> Here is a v5.
Few more in icommand_checks subroutine:

+
+ #$ps->slave->stty(qw(raw -echo));
+ $ps->slave->stty(qw(raw));
+ my $n = 0;
+ for my $test (@$inout)
+ {
+ #warn "test: @$test";
+ my ($in, @out) = @$test;
+ $n++;
+ #warn "in: $in";
+ #warn "out: @out";
+ $ps->send($in);

Few unwanted code can be removed.

Regards,
Vignesh
EnterpriseDB: http://www.enterprisedb.com



pgsql-hackers by date:

Previous
From: Kyotaro Horiguchi
Date:
Subject: Re: let's kill AtSubStart_Notify
Next
From: Michael Paquier
Date:
Subject: Re: [HACKERS] [PATCH] pageinspect function to decode infomasks