oauth: Correct test dependency on oauth_hook_client master github/master
authorJacob Champion <jchampion@postgresql.org>
Tue, 27 Jan 2026 19:56:44 +0000 (11:56 -0800)
committerJacob Champion <jchampion@postgresql.org>
Tue, 27 Jan 2026 19:56:44 +0000 (11:56 -0800)
The oauth_validator tests missed the lessons of c89525d57 et al, so
certain combinations of command-line build order and `meson test`
options can result in

    Command 'oauth_hook_client' not found in [...] at src/test/perl/PostgreSQL/Test/Utils.pm line 427.

Add the missing dependency on the test executable. This fixes, for
example,

    $ ninja clean && ninja meson-test-prereq && PG_TEST_EXTRA=oauth meson test --no-rebuild

Reported-by: Jonathan Gonzalez V. <jonathan.abdiel@gmail.com>
Author: Jonathan Gonzalez V. <jonathan.abdiel@gmail.com>
Discussion: https://postgr.es/m/6e8f4f7c23faf77c4b6564c4b7dc5d3de64aa491.camel@gmail.com
Discussion: https://postgr.es/m/qh4c5tvkgjef7jikjig56rclbcdrrotngnwpycukd2n3k25zi2%4044hxxvtwmgum
Backpatch-through: 18

src/test/modules/oauth_validator/meson.build

index 3e8854739a28892ed094f318a67169f241e83ab7..c4b73e05297ed6837a2ddc846a50ab09d8b0b550 100644 (file)
@@ -81,5 +81,6 @@ tests += {
       'with_libcurl': oauth_flow_supported ? 'yes' : 'no',
       'with_python': 'yes',
     },
       'with_libcurl': oauth_flow_supported ? 'yes' : 'no',
       'with_python': 'yes',
     },
+    'deps': [oauth_hook_client],
   },
 }
   },
 }