Issue #420 - Decrease per-test pytest timeout to 100

[jiridanek]

Failing tests need to be reviewed.

• RouterTestSslInterRouter::test_connected_tls_sasl_routers, part of system_tests_ssl, takes a long time (>300s) #420

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 76.70%. Comparing base (acd49e4) to head (7750dbb).
⚠️ Report is 241 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #456      +/-   ##
==========================================
- Coverage   76.71%   76.70%   -0.01%     
==========================================
  Files         247      247              
  Lines       63983    63983              
  Branches     5902     6024     +122     
==========================================
- Hits        49083    49077       -6     
+ Misses      12332    12329       -3     
- Partials     2568     2577       +9     

Flag	Coverage Δ
pysystemtests	87.73% <ø> (ø)
pyunittests	54.60% <ø> (ø)
systemtests	69.48% <ø> (-0.03%)	⬇️
unittests	21.90% <ø> (+<0.01%)	⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Components	Coverage Δ
calculator	25.99% <ø> (+<0.01%)	⬆️
systemtests	77.37% <ø> (-0.02%)	⬇️

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[ganeshmurthy]

@jiridanek do you still want to do this ? Going from 400 to 100 ? I would rather not do this and find out that we need to move it back up again ?

[jiridanek]

@jiridanek do you still want to do this ? Going from 400 to 100 ? I would rather not do this and find out that we need to move it back up again ?

@ganeshmurthy This timeout is the default per-test timeout in pytest, so if you write a single python test method that takes more than 100 seconds, you can

1. split it into multiple test methods
2. add some pytest annotation to extend the timeout for this specific test method

IIRC, when this PR was newly created, there was one or two tests exceeding the 100 seconds, and I suspected in both cases it is because of some test bug, I don't exactly recall what it was. Looks like there is a few tests going over the 100 seconds.

I'll try to look why they take that much time and if they truly need it, or if there is something wrong going on.