Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Windows test failures #76

Open
schodge opened this issue Oct 13, 2014 · 1 comment
Open

Windows test failures #76

schodge opened this issue Oct 13, 2014 · 1 comment

Comments

@schodge
Copy link
Contributor

schodge commented Oct 13, 2014

Sorry for being AWOL lately. Here's the current output of tests on one Windows box. Accidentally reverted to an older verison of git (1.8.x), not sure if that's the cause of anything:

λ nosetests .\tests
.F........FSSSSSSSS......F.SSSSSSSS.S.
======================================================================
FAIL: test_autopep8_args (tests.test_main.TestMain)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "c:\GitHub\pep8radius\tests\test_main.py", line 27, in test_autopep8_args
    self.assertEqual(us.ignore, them.ignore)
AssertionError: set(['E24']) != [u'E24']

======================================================================
FAIL: test_from_diff (tests.test_radius.TestRadiusFromDiff)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "c:\GitHub\pep8radius\tests\test_radius.py", line 181, in test_from_diff
    self.assertEqual(out.getvalue(), exp_diff)
AssertionError: '' != '--- original\\foo.py\n+++ fixed\\foo.py\n@@ -4,7 +4,9 @@\n   return a + b\n \n foo = 1; bar = 2; print(foo * bar)\n-a=1; b=42; c=3\n+a = 1\n+b = 42\n+c = 3\n d=7\n \n def f(x = 1, y = 2):\n'

======================================================================
FAIL: test_one_line_from_subdirectory (tests.test_vcs.TestRadiusGit)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "c:\GitHub\pep8radius\tests\test_radius.py", line 92, in test_one_line_from_subdirectory
    cwd=SUBTEMP_DIR)
  File "c:\GitHub\pep8radius\tests\test_radius.py", line 50, in check
    self.assert_equal(out.getvalue(), exp_diff, test_name)
  File "c:\GitHub\pep8radius\tests\test_radius.py", line 74, in assert_equal
    'expected', 'result'))
AssertionError: --- expected\test_one_line
+++ result\test_one_line
@@ -1,5 +1,5 @@
---- c:\GitHub\pep8radius\tests\temp\subtemp\temp.py
-+++ c:\GitHub\pep8radius\tests\temp\subtemp\temp.py
+--- c:\GitHub\pep8radius\tests\temp\subtemp/temp.py
++++ c:\GitHub\pep8radius\tests\temp\subtemp/temp.py
 @@ -4,7 +4,9 @@
    return a + b



----------------------------------------------------------------------
Ran 38 tests in 11.170s

FAILED (SKIP=17, failures=3)
@hayd
Copy link
Owner

hayd commented Oct 13, 2014

No worries, hope you've been keeping yourself busy!

  • The set != list thing is an update in autopep8, so run pip install autopep8 --update.
  • Hmm test_one_line_from_subdirectory looks like there should be an os.join somewhere or something.
  • No idea about test_from_diff! Maybe check out what the diff variable is in the test (is it empty for some reason).
  • version of git shouldn't matter!

p.s. if you are using conda, you might like to check out ctox for testing on different platforms. :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants