]> granicus.if.org Git - postgresql/commit
Clean up/tighten up coercibility checks in opr_sanity regression test.
authorTom Lane <tgl@sss.pgh.pa.us>
Sun, 14 Oct 2018 16:11:16 +0000 (12:11 -0400)
committerTom Lane <tgl@sss.pgh.pa.us>
Sun, 14 Oct 2018 16:11:16 +0000 (12:11 -0400)
commite9f42d529f990f94e1b7bdcec4a1111465c85326
tree22e6cb444f581f133c121ea5dc9ad2e8da9d48f8
parent1df21ddb19c6e764fc9378c900515a5d642ad820
Clean up/tighten up coercibility checks in opr_sanity regression test.

With the removal of the old abstime type, there are no longer any cases
in this test where we need to use the weaker castcontext-ignoring form
of binary coercibility check.  (The other major source of such headaches,
apparently-incompatible hash functions, is now hashvalidate()'s problem
not this test script's problem.)  Hence, just use binary_coercible()
everywhere, and remove the comments explaining why we don't do so ---
which were broken anyway by cda6a8d01.

I left physically_coercible() in place but renamed it to better
match what it's actually testing, and added some comments.

Also, in test queries that have an assumption about the maximum number
of function arguments they need to handle, add a clause to make them fail
if someday there's a relevant function with more arguments.  Otherwise
we're likely not to notice that we need to extend the queries.

Discussion: https://postgr.es/m/27637.1539388060@sss.pgh.pa.us
src/test/regress/expected/opr_sanity.out
src/test/regress/sql/opr_sanity.sql