Remove the unimplemented but ignored without='-E' parameters being passed to
script_helper.assert_python_failure(). No such feature has ever existed, thus it doesn't do what the comment claims. (It does add a 'without' variable to the environment of the child process but that was not intended)
This commit is contained in:
parent
fe7c5d6e4e
commit
566646133e
|
@ -426,13 +426,11 @@ class CmdLineTest(unittest.TestCase):
|
|||
self.assertIn(b'Unknown option: -z', err)
|
||||
self.assertEqual(err.splitlines().count(b'Unknown option: -z'), 1)
|
||||
self.assertEqual(b'', out)
|
||||
# Add "without='-E'" to prevent _assert_python to append -E
|
||||
# to env_vars and change the output of stderr
|
||||
rc, out, err = assert_python_failure('-z', without='-E')
|
||||
rc, out, err = assert_python_failure('-z')
|
||||
self.assertIn(b'Unknown option: -z', err)
|
||||
self.assertEqual(err.splitlines().count(b'Unknown option: -z'), 1)
|
||||
self.assertEqual(b'', out)
|
||||
rc, out, err = assert_python_failure('-a', '-z', without='-E')
|
||||
rc, out, err = assert_python_failure('-a', '-z')
|
||||
self.assertIn(b'Unknown option: -a', err)
|
||||
# only the first unknown option is reported
|
||||
self.assertNotIn(b'Unknown option: -z', err)
|
||||
|
|
Loading…
Reference in New Issue