From bd6e22bf6fdd6d77ff0a92cdbe7f50f22a5e509d Mon Sep 17 00:00:00 2001 From: David Miller Date: Sat, 14 Apr 2018 21:42:10 +0100 Subject: [PATCH] Add tests to ensure we can combine -t and -c in opal test refs #1502 --- opal/tests/test_core_test_runner.py | 13 +++++++++++++ 1 file changed, 13 insertions(+) diff --git a/opal/tests/test_core_test_runner.py b/opal/tests/test_core_test_runner.py index e5711d471..3e1e74b07 100644 --- a/opal/tests/test_core_test_runner.py +++ b/opal/tests/test_core_test_runner.py @@ -59,6 +59,19 @@ def test_run_tests_with_coverage(self, check_call): check_call.assert_has_calls(calls) + @patch('subprocess.check_call') + def test_run_tests_with_coverage_and_test_arg(self, check_call): + mock_args = MagicMock(name="args") + mock_args.userland_here = ffs.Path('.') + mock_args.coverage = True + mock_args.test = 'opal.tests.foo' + mock_args.failfast = False + test_runner._run_py_tests(mock_args) + calls = [ + call(['coverage', 'run', 'runtests.py', 'opal.tests.foo']), + call(['coverage', 'html']) + ] + @patch('subprocess.check_call') @patch.object(test_runner.sys, 'exit') def test_run_tests_with_coverage_errors(self, exiter, check_call):