Fix bug in test driver
authorJacob Bachmeyer <jcb@gnu.org>
Sat, 25 Mar 2023 01:02:00 +0000 (20:02 -0500)
committerJacob Bachmeyer <jcb@gnu.org>
Sat, 25 Mar 2023 01:02:00 +0000 (20:02 -0500)
commit14d9bba1fce6b032119bba38bf0b1adf7d7f38e6
tree4f610e6ce5b188cf4c5538e7dd0f227d7d1a5182
parent20cd125946146141fcc2af4042588b0831440645
Fix bug in test driver

This bug was exposed in commit fc58d8e82444d9f949f2ac5cbbed928ad6b12c30,
but only affected testsuite runs when coverage data is collected.

The exact cause is unknown, but that commit changed the processing used
for verifying signatures and the bug caused the testsuite to prematurely
declare the gatekeeper run complete and move on to the next test when a
signature was verified.  Presumably, the improved signature verification
somehow interacted poorly with Expect.

Bizarrely, the issue only occurred when Devel::Cover was used to check
the coverage of the testsuite and did not occur under regular testing.
The root cause is surely timing related, since perl is much slower when
collecting code coverage data.  This is also the reason the bug went
unnoticed for so long:  coverage runs are infrequent except when making
improvements to the testsuite.
testsuite/lib/gatekeeper.exp