[Asterisk-code-review] Inbound registration path: Early invite causes test timeout (testsuite[13])

[Jenkins2]

Jenkins2 has submitted this change and it was merged. ( https://gerrit.asterisk.org/10123 )

Change subject: Inbound registration path: Early invite causes test timeout
......................................................................

Inbound registration path: Early invite causes test timeout

The pjsip/registration/inbound/nominal/path/ test was set to originate a call
at 'ami_connect'. However, the SIPp scenario meant to receive the invite was
set to start later (possibly much later on slower systems). If the scenario
started after the invite timed out then the test would end up failing due to
the scenario timing out (never gets the invite).

This patch makes it so the call is not originated until after the receiving
invite scenario has been started.

Change-Id: I29d6f05492941f0d87bd4a8d6f3f20ff6a242ab4
---
M tests/channels/pjsip/registration/inbound/nominal/path/test-config.yaml
1 file changed, 1 insertion(+), 1 deletion(-)

Approvals:
  Joshua Colp: Looks good to me, but someone else must approve
  George Joseph: Looks good to me, approved
  Jenkins2: Approved for Submit



diff --git a/tests/channels/pjsip/registration/inbound/nominal/path/test-config.yaml b/tests/channels/pjsip/registration/inbound/nominal/path/test-config.yaml
index e2d7d9d..2283be9 100644
--- a/tests/channels/pjsip/registration/inbound/nominal/path/test-config.yaml
+++ b/tests/channels/pjsip/registration/inbound/nominal/path/test-config.yaml
@@ -30,7 +30,7 @@
             typename: 'pluggable_modules.Originator'
 
 originator-ua1_invite_recv:
-    trigger: 'ami_connect'
+    scenario-trigger-after: '3'
 
 sipp-config:
     memcheck-delay-stop: 7

-- 
To view, visit https://gerrit.asterisk.org/10123
To unsubscribe, or for help writing mail filters, visit https://gerrit.asterisk.org/settings

Gerrit-Project: testsuite
Gerrit-Branch: 13
Gerrit-MessageType: merged
Gerrit-Change-Id: I29d6f05492941f0d87bd4a8d6f3f20ff6a242ab4
Gerrit-Change-Number: 10123
Gerrit-PatchSet: 1
Gerrit-Owner: Kevin Harwell <kharwell at digium.com>
Gerrit-Reviewer: George Joseph <gjoseph at digium.com>
Gerrit-Reviewer: Jenkins2
Gerrit-Reviewer: Joshua Colp <jcolp at digium.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-code-review/attachments/20180913/fadee80c/attachment-0001.html>