> On March 24, 2015, 11:23 a.m., Matt Jordan wrote: > > ./asterisk/trunk/tests/rest_api/applications/stasis_status/test_case.py, > > lines 20-26 > > <https://reviewboard.asterisk.org/r/4520/diff/1/?file=72750#file72750line20> > > > > You may want to consider moving this class back into runtest. > > > > While pulling other classes out of the runtest 'module' may be good, > > particularly since some of those could act as infrastructure for other > > tests, the typical approach in the testsuite is to put the class that > > orchestrates a test into that module.
For this, I would argue that the test_case introduced by this test is actually a new, generalized variant of the base type of TestCase (the name needs to be modified a bit, though). I think that it would make more sense to move the test_scenario_factory back into run-test, considering that is where all the specialized logic lives. Thoughts? - Ashley ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviewboard.asterisk.org/r/4520/#review14783 ----------------------------------------------------------- On March 22, 2015, 11:34 p.m., Ashley Sanders wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviewboard.asterisk.org/r/4520/ > ----------------------------------------------------------- > > (Updated March 22, 2015, 11:34 p.m.) > > > Review request for Asterisk Developers. > > > Bugs: ASTERISK-24802 > https://issues.asterisk.org/jira/browse/ASTERISK-24802 > > > Repository: testsuite > > > Description > ------- > > When an error occurs while writing to a web socket, the web socket is > disconnected and the event is logged. A side-effect of this, however, is that > any application on the other side waiting for a response from Stasis is left > hanging indefinitely (as there is no mechanism presently available for > notifying interested parties about web socket error states in Stasis). > > This patch introduces a new channel variable: STASIS_STATUS to give outside > applications context when errors occur in Stasis that interrupt normal > processing. > > This test exercises three scenarios to elicit updates to the STASIS_STATUS > channel variable: > 1) The 'Babs' scenario: tests a nominal path of Stasis to verify the 'ACTIVE' > state is correctly applied. For this test, a call is originated under normal > conditions and then the system is polled for the value of STASIS_STATUS > before the channel is hung up. > 2) The 'Bugs' scenario: tests the situation where a call is originated > requesting an app that was never registered in Stasis to verify the 'FAILED' > state is correctly applied. > 3) The 'Buster' scenario: tests the situation where an app that was > registered in Stasis when call A was originated (and while call A is still > active) but is no longer registered when call B is originated. Determines if > the 'FAILED' state is correctly applied. > > ***Note*** This is a test. It is only a test. The review for the Asterisk > source can be found at: https://reviewboard.asterisk.org/r/4519/ > > > Diffs > ----- > > ./asterisk/trunk/tests/rest_api/applications/tests.yaml 6547 > > ./asterisk/trunk/tests/rest_api/applications/stasis_status/test_scenario_factory.py > PRE-CREATION > ./asterisk/trunk/tests/rest_api/applications/stasis_status/test_scenario.py > PRE-CREATION > ./asterisk/trunk/tests/rest_api/applications/stasis_status/test_case.py > PRE-CREATION > ./asterisk/trunk/tests/rest_api/applications/stasis_status/test-config.yaml > PRE-CREATION > ./asterisk/trunk/tests/rest_api/applications/stasis_status/run-test > PRE-CREATION > > ./asterisk/trunk/tests/rest_api/applications/stasis_status/observable_object.py > PRE-CREATION > > ./asterisk/trunk/tests/rest_api/applications/stasis_status/configs/ast1/sip.conf > PRE-CREATION > > ./asterisk/trunk/tests/rest_api/applications/stasis_status/configs/ast1/http.conf > PRE-CREATION > > ./asterisk/trunk/tests/rest_api/applications/stasis_status/configs/ast1/extensions.conf > PRE-CREATION > > ./asterisk/trunk/tests/rest_api/applications/stasis_status/configs/ast1/ari.conf > PRE-CREATION > ./asterisk/trunk/tests/rest_api/applications/stasis_status/ari_client.py > PRE-CREATION > > Diff: https://reviewboard.asterisk.org/r/4520/diff/ > > > Testing > ------- > > > Thanks, > > Ashley Sanders > >
-- _____________________________________________________________________ -- Bandwidth and Colocation Provided by http://www.api-digital.com -- asterisk-dev mailing list To UNSUBSCRIBE or update options visit: http://lists.digium.com/mailman/listinfo/asterisk-dev
