search

scionproto / scion

SCION Internet Architecture
https://scion.org
Apache License 2.0
379 stars 158 forks source link

acceptance: sig_short_exp_time is broken #3935

Open matzf opened 3 years ago

matzf commented 3 years ago

sig_short_exp_time is failing (at least) since the new SIG was introduced in #3932.

For some reason the acceptance test framework still sees a successful test.; this part has been fixed, it's no longer false positive, but it's still broken.

Note: The setup for this test is using a fake sciond connection, configured by fake_sciond.json data files. Such a fake sciond connection was supported explicitly by the old SIG implementation, specifically for this test. The new SIG does not (yet) have support for this.

matzf commented 2 years ago

In #4182

Fix the false-positive result of sig_short_exp_time, and explicitly skip the now failing test from running.

The issue was that ping failed (as it should) but the exit code was ignored.