[Asterisk-code-review] app queue: Added reason pause of member (asterisk[13])

Matt Jordan asteriskteam at digium.com
Tue Oct 20 09:07:46 CDT 2015


Matt Jordan has posted comments on this change.

Change subject: app_queue: Added reason pause of member
......................................................................


Patch Set 1: Code-Review-1

As a new feature being added to Asterisk 13, this *must* have an accompanying test in either the Test Suite or unit test in Asterisk.

Tests are always a "good thing", but for modifications to an LTS, they are a requirement. See https://wiki.asterisk.org/wiki/display/AST/Software+Configuration+Management+Policies#SoftwareConfigurationManagementPolicies-FeaturePolicy for more information.

If you'd like some pointers on writing a test for this in the Asterisk Test Suite, please just comment on here and I'd be happy to provide some guidance.

Given the functionality being

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

Gerrit-MessageType: comment
Gerrit-Change-Id: Ia5db503482f50764c15e2020196c785f59d4a68e
Gerrit-PatchSet: 1
Gerrit-Project: asterisk
Gerrit-Branch: 13
Gerrit-Owner: Rodrigo Ramirez Norambuena <a at rodrigoramirez.com>
Gerrit-Reviewer: Anonymous Coward #1000019
Gerrit-Reviewer: Matt Jordan <mjordan at digium.com>
Gerrit-HasComments: No



More information about the asterisk-code-review mailing list