[Asterisk-code-review] app_if: Add tests for conditional branch apps (testsuite[master])

Joshua Colp asteriskteam at digium.com
Wed Feb 2 11:43:42 CST 2022


Attention is currently required from: N A.
Joshua Colp has posted comments on this change. ( https://gerrit.asterisk.org/c/testsuite/+/17794 )

Change subject: app_if: Add tests for conditional branch apps
......................................................................


Patch Set 2: Code-Review-1

(2 comments)

Patchset:

PS2: 
Marking -1 until


File tests/apps/if/test-config.yaml:

https://gerrit.asterisk.org/c/testsuite/+/17794/comment/ec08d08a_d9a456a1 
PS2, Line 4:         'This tests the If, ElseIf, Else, EndIf, and ExitIf applications to make sure
Going forward tests should be more descriptive about what they are doing, as it can help immensely when another individual goes to look at the test so they can understand precisely what was being achieved instead of deciphering dialplan logic or other logic.



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

Gerrit-Project: testsuite
Gerrit-Branch: master
Gerrit-Change-Id: Iff228c42c9338e1461950ab9fb85f3edf404daae
Gerrit-Change-Number: 17794
Gerrit-PatchSet: 2
Gerrit-Owner: N A <mail at interlinked.x10host.com>
Gerrit-Reviewer: Friendly Automation
Gerrit-Reviewer: Joshua Colp <jcolp at sangoma.com>
Gerrit-Attention: N A <mail at interlinked.x10host.com>
Gerrit-Comment-Date: Wed, 02 Feb 2022 17:43:42 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-code-review/attachments/20220202/00d238a2/attachment.html>


More information about the asterisk-code-review mailing list