Skip to content

Commit

Permalink
Get rid of sys.executable usage
Browse files Browse the repository at this point in the history
That makes things worse not better. It causes "python setup.py test" to
fail if pyflakes isn't installed already.
  • Loading branch information
myint committed Jul 17, 2013
1 parent 6d7ed18 commit fd14dda
Showing 1 changed file with 4 additions and 6 deletions.
10 changes: 4 additions & 6 deletions test_autoflake.py
Original file line number Diff line number Diff line change
Expand Up @@ -4,6 +4,7 @@
"""Test suite for autoflake."""

from __future__ import unicode_literals
from __future__ import print_function

import contextlib
import io
Expand Down Expand Up @@ -778,8 +779,7 @@ def test_end_to_end(self):
x = os.sep
print(x)
""") as filename:
process = subprocess.Popen([sys.executable,
'./autoflake',
process = subprocess.Popen(['./autoflake',
'--imports=fake_foo,fake_bar',
filename],
stdout=subprocess.PIPE)
Expand All @@ -800,8 +800,7 @@ def test_end_to_end_with_remove_all_unused_imports(self):
x = os.sep
print(x)
""") as filename:
process = subprocess.Popen([sys.executable,
'./autoflake',
process = subprocess.Popen(['./autoflake',
'--remove-all',
filename],
stdout=subprocess.PIPE)
Expand All @@ -820,8 +819,7 @@ def test_end_to_end_with_error(self):
x = os.sep
print(x)
""") as filename:
process = subprocess.Popen([sys.executable,
'./autoflake',
process = subprocess.Popen(['./autoflake',
'--imports=fake_foo,fake_bar',
'--remove-all',
filename],
Expand Down

0 comments on commit fd14dda

Please sign in to comment.