Skip to content

Commit

Permalink
Appveyor fixes (#402)
Browse files Browse the repository at this point in the history
* Appveyor fixes

* Fixing run_tee test for Windows

* Dropping py34 test on Appveyor, pypiwin32 does not support it
  • Loading branch information
henryiii authored Aug 10, 2018
1 parent bf1eac7 commit 56912be
Show file tree
Hide file tree
Showing 2 changed files with 11 additions and 10 deletions.
8 changes: 3 additions & 5 deletions .appveyor.yml
Original file line number Diff line number Diff line change
@@ -1,9 +1,8 @@
environment:
matrix:
- PYTHON: "C:\\Python27"
- PYTHON: "C:\\Python34"
- PYTHON: "C:\\Python35"
- PYTHON: "C:\\Python36"
- PYTHON: "C:\\Python37"

install:
- set PATH=%PYTHON%;%PYTHON%\Scripts;%PATH%
Expand All @@ -12,9 +11,8 @@ install:
- python --version
- python -c "import struct; print(struct.calcsize('P') * 8)"

# Upgrade to the latest version of pip to avoid it displaying warnings
# about it being out of date.
- pip install --disable-pip-version-check --user --upgrade pip
- python -m pip --version
# - python -m pip install --disable-pip-version-check --user --upgrade pip

- pip install -r dev-requirements.txt
- pip install wheel
Expand Down
13 changes: 8 additions & 5 deletions tests/test_local.py
Original file line number Diff line number Diff line change
Expand Up @@ -1141,12 +1141,15 @@ def test_run_fg(self, capfd):
stdout = capfd.readouterr()[0]
assert 'test_local.py' in stdout

@skip_on_windows
@pytest.mark.xfail(reason=
'This test randomly fails on Mac and PyPy on Travis, not sure why')
def test_run_tee(self, capfd):
from plumbum.cmd import ls
f = ls["-l"].run_tee()
stdout = capfd.readouterr()[0]
assert 'test_local.py' in stdout
assert 'test_local.py' in f[1]
from plumbum.cmd import echo

result = echo['This is fun'].run_tee()
assert result[1] == 'This is fun\n'
assert 'This is fun\n' == capfd.readouterr()[0]

def test_run_tf(self):
from plumbum.cmd import ls
Expand Down

0 comments on commit 56912be

Please sign in to comment.