diff options
| author | 2022-11-28 09:34:50 +0800 | |
|---|---|---|
| committer | 2022-11-28 09:34:50 +0800 | |
| commit | 2da0ddf14625cfd8037c198881ab789827746c40 (patch) | |
| tree | 4b4f791617e1fc002d130b9fe1bdf8a8cc723b90 | |
| parent | Docstring phrasing update (diff) | |
Add additional tests for py_args
| -rw-r--r-- | tests/test_nsjail.py | 23 | 
1 files changed, 14 insertions, 9 deletions
| diff --git a/tests/test_nsjail.py b/tests/test_nsjail.py index 127753e..9f99465 100644 --- a/tests/test_nsjail.py +++ b/tests/test_nsjail.py @@ -369,18 +369,23 @@ class NsJailTests(unittest.TestCase):          self.assertEqual(result.args[end - len(args) : end], args)      def test_py_args(self): -        expected = ["-m", "timeit"] -        args = [ -            ["", "-m", "timeit"], -            ["", "", "-m", "timeit"], -            ["", "", "", "-m", "timeit"], +        cases = [ +            # Normal args +            (["-c", "print('hello')"], ["-c", "print('hello')"]), +            # Leading empty strings should be removed +            (["", "-m", "timeit"], ["-m", "timeit"]), +            (["", "", "-m", "timeit"], ["-m", "timeit"]), +            (["", "", "", "-m", "timeit"], ["-m", "timeit"]), +            # Non-leading empty strings should be preserved +            (["-m", "timeit", ""], ["-m", "timeit", ""]),          ] -        # Leading empty strings should be removed -        for case in args: + +        for args, expected in cases:              with self.subTest(args=args): -                result = self.nsjail.python3(case) +                result = self.nsjail.python3(py_args=args) +                idx = result.args.index("-Squ") +                self.assertEqual(result.args[idx + 1 :], expected)                  self.assertEqual(result.returncode, 0) -                self.assertEqual(result.args[-2:], expected)  class NsJailArgsTests(unittest.TestCase): | 
