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

add tests #1

Closed hayd closed 10 years ago

hayd commented 10 years ago

don't need to check all features of autopep8, just that it is only working on part of the file...

I was thinking:

Will need to add script (for travis) to create git (and hg) repos in test dir and commit the original_ files, to use pep8radius against. Then ensure that travis is "in" the correct directory, I'm hoping this will "just work". Should test against autopep8 master as well as latest release.

hayd commented 10 years ago

Tests should "just work" with autopep8 master... though need more of them (esp some unit tests)!

I think I'll wait til they make a release a new version to update tests here. (I need a script to create git/hg repos and commit, for travis.)

hayd commented 10 years ago

For some reason adding this is not sufficient for travis to accept that hg has a username:

echo -e "[ui]\nusername = Your Name <your@email.com>" ~/.hgrc

Update: duh, forgot the >.

It's working great with git. TBH I think I should rework it to, as that'll be far less noise:

hayd commented 10 years ago

refactored that, much much better. I think there's a bug in hg. Will try and fix sometime.

Now passing travis (with autopep8 master) https://travis-ci.org/hayd/btyfi/builds/22314827 whoop