[asterisk-commits] mjordan: testsuite/asterisk/trunk r4458 - /asterisk/trunk/lib/python/asterisk/

SVN commits to the Asterisk project asterisk-commits at lists.digium.com
Sat Dec 14 11:14:40 CST 2013


Author: mjordan
Date: Sat Dec 14 11:14:31 2013
New Revision: 4458

URL: http://svnview.digium.com/svn/testsuite?view=rev&rev=4458
Log:
apptest: Fix shadowing of test_config in base class

AppTest made a poor choice in variable names by shadowing the TestCase class's
test_config attribute. This caused problems when that variable was used during
the constructor in TestCase. This patch fixes AppTest to use a different name
for the raw YAML test config it is passed.

Modified:
    asterisk/trunk/lib/python/asterisk/apptest.py

Modified: asterisk/trunk/lib/python/asterisk/apptest.py
URL: http://svnview.digium.com/svn/testsuite/asterisk/trunk/lib/python/asterisk/apptest.py?view=diff&rev=4458&r1=4457&r2=4458
==============================================================================
--- asterisk/trunk/lib/python/asterisk/apptest.py (original)
+++ asterisk/trunk/lib/python/asterisk/apptest.py Sat Dec 14 11:14:31 2013
@@ -56,9 +56,9 @@
         self._expected_results = {}     # Expected results for all scenarios
         self._event_instances = []      # The ApplicationEventInstance objects
 
-        self.test_config = test_config
-        self._application = self.test_config['app']
-        self._scenarios = self.test_config['scenarios']
+        self.raw_test_config = test_config
+        self._application = self.raw_test_config['app']
+        self._scenarios = self.raw_test_config['scenarios']
 
         self.register_ami_observer(self.__ami_connect_handler)
         self.register_stop_observer(self.end_scenario)




More information about the asterisk-commits mailing list