[Asterisk-code-review] confbridge waitmarked kick: reactor times out too soon (testsuite[master])

[Mark Michelson]

Mark Michelson has submitted this change and it was merged.

Change subject: confbridge_waitmarked_kick: reactor times out too soon
......................................................................


confbridge_waitmarked_kick: reactor times out too soon

This test failed once and it appears simply to be due to the reactor timing
out. At various points during the AppTest it resets/extends the timeout value.
It is possible on slower systems that the timeout is reached before it can
be extended. This patch increases the test's reactor timeout.

Change-Id: Iab55d34645d3e0039b33945ebe5029111075af74
---
M tests/apps/confbridge/confbridge_waitmarked_kick/test-config.yaml
1 file changed, 1 insertion(+), 0 deletions(-)

Approvals:
  Mark Michelson: Looks good to me, approved
  Anonymous Coward #1000019: Verified
  Joshua Colp: Looks good to me, but someone else must approve



diff --git a/tests/apps/confbridge/confbridge_waitmarked_kick/test-config.yaml b/tests/apps/confbridge/confbridge_waitmarked_kick/test-config.yaml
index 0e59d41..27ede46 100644
--- a/tests/apps/confbridge/confbridge_waitmarked_kick/test-config.yaml
+++ b/tests/apps/confbridge/confbridge_waitmarked_kick/test-config.yaml
@@ -16,6 +16,7 @@
         typename: 'apptest.AppTest'
 
 test-object-config:
+  reactor-timeout: '50'
   app: 'ConfBridge'
   scenarios:
     -

-- 
To view, visit https://gerrit.asterisk.org/1830
To unsubscribe, visit https://gerrit.asterisk.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: Iab55d34645d3e0039b33945ebe5029111075af74
Gerrit-PatchSet: 1
Gerrit-Project: testsuite
Gerrit-Branch: master
Gerrit-Owner: Kevin Harwell <kharwell at digium.com>
Gerrit-Reviewer: Anonymous Coward #1000019
Gerrit-Reviewer: Joshua Colp <jcolp at digium.com>
Gerrit-Reviewer: Mark Michelson <mmichelson at digium.com>