[Asterisk-code-review] Testsuite: Refactored Batched RLS Test Driver for Better Log... (testsuite[master])

Ashley Sanders asteriskteam at digium.com
Tue Nov 10 09:53:00 CST 2015


Hello Mark Michelson, Anonymous Coward #1000019,

I'd like you to reexamine a change.  Please visit

    https://gerrit.asterisk.org/1437

to look at the new patch set (#3).

Change subject: Testsuite: Refactored Batched RLS Test Driver for Better Logging
......................................................................

Testsuite: Refactored Batched RLS Test Driver for Better Logging

While this does not actually correct the issue with the test failure (which
incidentally turns out is not localized to this test), this patch modifies the
the test driver such that the test output produces log messages that help to
elimanate it as the suspect in the test failure. Also, this patch includes
fixes for pylint errors (e.g. missing docstrings).

This is part one (1) of an n-patch series of refactorings to help determine
the root cause of the test failure and correct pylint issues.

ASTERISK-25430
Reported By: Ashley Sanders

Change-Id: I15c005ec6a43919e5fe533c8f2dc8fb3d5c41e64
---
M tests/channels/pjsip/subscriptions/rls/lists/nominal/presence/batched/basic/driver.py
M tests/channels/pjsip/subscriptions/rls/lists/nominal/presence/batched/basic/test-config.yaml
2 files changed, 266 insertions(+), 64 deletions(-)


  git pull ssh://gerrit.asterisk.org:29418/testsuite refs/changes/37/1437/3
-- 
To view, visit https://gerrit.asterisk.org/1437
To unsubscribe, visit https://gerrit.asterisk.org/settings

Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I15c005ec6a43919e5fe533c8f2dc8fb3d5c41e64
Gerrit-PatchSet: 3
Gerrit-Project: testsuite
Gerrit-Branch: master
Gerrit-Owner: Ashley Sanders <asanders at digium.com>
Gerrit-Reviewer: Anonymous Coward #1000019
Gerrit-Reviewer: Mark Michelson <mmichelson at digium.com>



More information about the asterisk-code-review mailing list