search

hayd / pep8radius

PEP8 clean only the parts of the files touched since the last commit, a previous commit or (the merge-base of) a branch.
MIT License
183 stars 9 forks source link

Windows fix 2 #49

Closed schodge closed 10 years ago

schodge commented 10 years ago

Need to change two paths to unix-style to make the tests pass. Also change an assertEqual to assert_equal for easier debugging.

schodge commented 10 years ago

These are the only errors left, and this is my work machine, which is the worst case scenario. I don't see another way to do this unless we find upstream what's generating the unix-style paths.

c:\github\pep8radius (windows_fix_2)
λ nosetests tests\test_pep8radius.py
...SSS......F.
======================================================================
FAIL: test_using_vc (test_pep8radius.TestRadiusNoVCS)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "c:\github\pep8radius\tests\test_pep8radius.py", line 97, in test_using_vc
    self.assertTrue(using_hg())
AssertionError: False is not true

----------------------------------------------------------------------
Ran 14 tests in 4.101s

FAILED (SKIP=3, failures=1)
hayd commented 10 years ago

ooh, good spot on that last one, will fix that in the color branch I'm working on (hiding usingHg behind a if branch of the repo creation being sucessful!) we're getting there!

hayd commented 10 years ago

fixed via https://github.com/hayd/pep8radius/commit/a40e253e2166b10c69888551ec79bcccfcedec6a I think.