Re: old_snapshot: add test for coverage - Mailing list pgsql-hackers

From Dong Wook Lee
Subject Re: old_snapshot: add test for coverage
Date
Msg-id CAAcByaKHb3AwCvL=Nz2S6+fC2pnuwhq98-tO89mt6Bpvpzv34w@mail.gmail.com
Whole thread Raw
In response to Re: old_snapshot: add test for coverage  (Tom Lane <tgl@sss.pgh.pa.us>)
List pgsql-hackers
On Mon, Aug 8, 2022 at 2:37 PM Tom Lane <tgl@sss.pgh.pa.us> wrote:
>
> Dong Wook Lee <sh95119@gmail.com> writes:
> > I wrote a test of the old_snapshot extension for coverage.
>
> Hmm, does this really provide any meaningful coverage?  The test
> sure looks like it's not doing much.

Previously written tests were simply test codes to increase coverage.
Therefore, we can make a better test.
I'll think about it by this week. If this work exceeds my ability, I
will let you know by reply.
It's okay that the issue should be closed unless I write a meaningful test.

---
Regards,
DongWook Lee.



pgsql-hackers by date:

Previous
From: Tom Lane
Date:
Subject: Re: Missing MaterialPath support in reparameterize_path_by_child
Next
From: Alexander Korotkov
Date:
Subject: Re: Allow placeholders in ALTER ROLE w/o superuser