This is fantastic,@vitcpp!

I'm not sure I understand why there are twonewexpected/*.out files, but no corresponding new*.sql files to generate them. Is that intentional?

Hi@esabol, once I've changed the default behaviour of the output by improving the precision, some tests show differences in numbers. It is why I've added two new test variant comparison files. But, it seems it may be a mistake. It would be better to just replace the old files. I will check. Thank you. BTW, I think to add some a new test for testing output precision with different settings.

P.S. Test variant comparison files are used if the output may vary on different platforms:https://www.postgresql.org/docs/current/regress-variant.html

P.P.S. I'm going to check it on 32 bit platform as well.

The changes should definitely have tests. I was just confused as to how the newexpected/*.out files did that. Variant comparison files are not something I was familiar with. Thank you for the link explaining the concept. If you think this is the best way to implement these tests, I am fine with it.

expected/epochprop.out differs for 10, 11 and for 12+ versions, because the default precision was 15 significant digits prior to PG12. It is why I've added one more variant comparison file. At the left - output for PG 10-11, at the right - output for PG 12+.

Why not putset extra_float_digits = 2; at the top of each "general" test file, and only check for the varying precision in a specific test for that purpose, if at all? (Ok, I see it's only the gist bounding box test that's affected, but that seems unrelated functionality.)

Why not putset extra_float_digits = 2; at the top of each "general" test file, and only check for the varying precision in a specific test for that purpose, if at all? (Ok, I see it's only the gist bounding box test that's affected, but that seems unrelated functionality.)

It makes sense, thank you! It seems, set_sphere_output_precision(8) is set at the top of almost all tests.

I've added one more commit to set extra_float_digits = 2 in epochprop and bounding_box_gist tests as proposed by@df7cb . It allows to remove expexted/bounding_box_gist_2.out but expected/epochprop_1.out is still required. The reason to create expected/epochprop_1.out is that the earlier versions output exactly 17 digits if extra_float_digits is 2, some of these digits may be meaningless. PG versions 12+ use a special function that outputs only meaningful digits even if extra_float-digits = 2, it is why in PG 12+ some numbers may contain less than 17 digits if the text representation if enough to define the double number.