Skip to content

Commit

Permalink
TESTS: Increase test_resp_idle_timeout* timeout
Browse files Browse the repository at this point in the history
As suggested by Sumit, let's increase the timeout in the
test_resp_idle_timeout* as those are a little bit on the edge.

Related:
https://pagure.io/SSSD/sssd/issue/3633

Resolves:
https://pagure.io/SSSD/sssd/issue/3730

Signed-off-by: Fabiano Fidêncio <fidencio@redhat.com>
Reviewed-by: Jakub Hrozek <jhrozek@redhat.com>
  • Loading branch information
fidencio committed May 16, 2018
1 parent cf4f5e0 commit 10a0bda
Showing 1 changed file with 8 additions and 4 deletions.
12 changes: 8 additions & 4 deletions src/tests/intg/test_secrets.py
Expand Up @@ -643,7 +643,9 @@ def test_resp_idle_timeout_shutdown_slow(setup_for_resp_timeout_test):
# so it would tick at 30, 60 and 90 seconds and the responder_idle_timeout
# uses a greater-than comparison, so the 60-seconds tick wouldn't yet
# trigger the process' shutdown.
p.wait(timeout=90)
# 100s has been chosen in order to take a safer path when running our CI
# tests.
p.wait(timeout=100)
assert p.is_running() is False


Expand Down Expand Up @@ -678,7 +680,9 @@ def test_resp_idle_timeout_postpone_shutdown_slow(setup_for_resp_timeout_test,
# updated).
assert p.is_running() is True

# Wait more 60s in order to be sure that the process actually is shutdown
# when it should be.
p.wait(timeout=60)
# Wait at least 60s in order to be sure that the process actually is
# shutdown when it should be.
# 70s has been chosen in order to take a safer path when running our CI
# tests.
p.wait(timeout=70)
assert p.is_running() is False

0 comments on commit 10a0bda

Please sign in to comment.