Chromium Code Reviews| Index: Tools/Scripts/webkitpy/common/system/executive_unittest.py |
| diff --git a/Tools/Scripts/webkitpy/common/system/executive_unittest.py b/Tools/Scripts/webkitpy/common/system/executive_unittest.py |
| index 0f3d9170443683013aee6ee608178144cec960c0..57f39b9320464a2336ab3861938d824c14e5850d 100644 |
| --- a/Tools/Scripts/webkitpy/common/system/executive_unittest.py |
| +++ b/Tools/Scripts/webkitpy/common/system/executive_unittest.py |
| @@ -88,11 +88,7 @@ class ExecutiveTest(unittest.TestCase): |
| def test_interpreter_for_script(self): |
| self.assert_interpreter_for_content(None, '') |
| self.assert_interpreter_for_content(None, 'abcd\nefgh\nijklm') |
| - self.assert_interpreter_for_content(None, '##/usr/bin/perl') |
| - self.assert_interpreter_for_content('perl', '#!/usr/bin/env perl') |
| - self.assert_interpreter_for_content('perl', '#!/usr/bin/env perl\nfirst\nsecond') |
| - self.assert_interpreter_for_content('perl', '#!/usr/bin/perl') |
| - self.assert_interpreter_for_content('perl', '#!/usr/bin/perl -w') |
| + self.assert_interpreter_for_content(None, '##/usr/bin/env python') |
|
Julien - ping for review
2015/07/31 21:51:25
Dirk, I think this unit test updates makes sense.
Dirk Pranke
2015/07/31 22:01:24
Yup, that's fine. Still lgtm.
|
| self.assert_interpreter_for_content(sys.executable, '#!/usr/bin/env python') |
| self.assert_interpreter_for_content(sys.executable, '#!/usr/bin/env python\nfirst\nsecond') |
| self.assert_interpreter_for_content(sys.executable, '#!/usr/bin/python') |