Generate scenarios inside subunit.tests.test_suite, not by subclassing WithScenarios.
authorThomi Richards <thomi.richards@canonical.com>
Mon, 9 Dec 2013 04:58:22 +0000 (17:58 +1300)
committerThomi Richards <thomi.richards@canonical.com>
Mon, 9 Dec 2013 04:58:22 +0000 (17:58 +1300)
python/subunit/tests/__init__.py
python/subunit/tests/test_output_filter.py

index b5a7fdc..c1c2c64 100644 (file)
@@ -17,6 +17,8 @@
 import sys
 from unittest import TestLoader
 
+from testscenarios import generate_scenarios
+
 
 # Before the test module imports to avoid circularity.
 # For testing: different pythons have different str() implementations.
@@ -61,5 +63,7 @@ def test_suite():
     result.addTest(loader.loadTestsFromModule(test_subunit_tags))
     result.addTest(loader.loadTestsFromModule(test_subunit_stats))
     result.addTest(loader.loadTestsFromModule(test_run))
-    result.addTest(loader.loadTestsFromModule(test_output_filter))
+    result.addTests(
+        generate_scenarios(loader.loadTestsFromModule(test_output_filter))
+    )
     return result
index f03a7b7..3373d48 100644 (file)
@@ -22,7 +22,6 @@ import sys
 from tempfile import NamedTemporaryFile
 
 from contextlib import contextmanager
-from testscenarios import WithScenarios
 from testtools import TestCase
 from testtools.compat import _u
 from testtools.matchers import (
@@ -58,7 +57,7 @@ class SafeOptionParser(optparse.OptionParser):
 safe_parse_arguments = partial(parse_arguments, ParserClass=SafeOptionParser)
 
 
-class TestStatusArgParserTests(WithScenarios, TestCase):
+class TestStatusArgParserTests(TestCase):
 
     scenarios = [
         (cmd, dict(command=cmd, option='--' + cmd)) for cmd in _ALL_ACTIONS
@@ -204,7 +203,7 @@ def temp_file_contents(data):
         yield f
 
 
-class StatusStreamResultTests(WithScenarios, TestCase):
+class StatusStreamResultTests(TestCase):
 
     scenarios = [
         (s, dict(status=s, option='--' + s)) for s in _ALL_ACTIONS