Re: [PATCH] random_normal function

Hello old thread.

On Jan 19, 2023, at 01:01, Tom Lane <tgl@sss.pgh.pa.us> wrote:

> I think "ignore:" was a kluge we put in twenty-plus years ago when our
> testing standards were a lot lower, and it's way past time we got
> rid of it.

Today I discovered a bad regex in the pgTAP Makefile that matched 18.0 as an 8.0 release and used it `ignore:` a test
requiringRLS. I found it after I saw this test failure on Postgres 18: 

# syntax error in schedule file "test/build/run.sch" line 3: ignore: policy

I fixed the regex (removed any detection of pre-9.0 matching, in fact), so all good there. But I wanted to point out
that`ignore:` is documented in the wiki[1], so other extensions may use it. If no one else notices perhaps it’s not a
bigdeal, but I want to call out that pg_regress features, “documented” or not, might be used outside the core. 

Maybe the proper way to address this issue is to add formal docs for pg_regress. I always have the damnedest time
findingits docs and usually end up on the Wiki. If there was official documentation for it like there is for `psql`,
etc.,it might help to prevent such issues in the future. Minor as they may be, admittedly. 

Thoughts?

Best,

David

[1]: https://wiki.postgresql.org/wiki/Regression_test_authoring