[Asterisk-code-review] StatsD: Write Testsuite tests (testsuite[master])

Kevin Harwell asteriskteam at digium.com
Tue Oct 13 17:25:34 CDT 2015


Kevin Harwell has posted comments on this change.

Change subject: StatsD: Write Testsuite tests
......................................................................


Patch Set 2:

(1 comment)

https://gerrit.asterisk.org/#/c/1392/2/tests/apps/statsd/mockd.py
File tests/apps/statsd/mockd.py:

Line 95:         if not is_correct:
       :             LOGGER.debug('Packet not specified in configuration')
> If it reaches this debug message, then is_correct is False. This will retur
Yeah I didn't notice that before myself. You are right. It will return false and then stop the test.


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

Gerrit-MessageType: comment
Gerrit-Change-Id: Id93dbeea53cf978461151e9002af2afe1029ce8b
Gerrit-PatchSet: 2
Gerrit-Project: testsuite
Gerrit-Branch: master
Gerrit-Owner: Tyler Cambron <tcambron at digium.com>
Gerrit-Reviewer: Anonymous Coward #1000019
Gerrit-Reviewer: Kevin Harwell <kharwell at digium.com>
Gerrit-Reviewer: Tyler Cambron <tcambron at digium.com>
Gerrit-HasComments: Yes



More information about the asterisk-code-review mailing list