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

Tests for local.py #77

Closed
wants to merge 1 commit into from

Conversation

tusharmakkar08
Copy link
Contributor

@tusharmakkar08 tusharmakkar08 commented Oct 4, 2016

This includes tests for local.py. It is a prerequisite for TODO mentioned in 9e03533 . This will also increase code coverage.

@coveralls
Copy link

coveralls commented Oct 4, 2016

Coverage Status

Coverage increased (+3.3%) to 63.869% when pulling a4d5907 on tusharmakkar08:local_tests into 64b6b41 on dsoprea:master.

@dsoprea
Copy link
Owner

dsoprea commented Oct 4, 2016

You don't have to create a repository, here. Just check one in under tests/resources.

@tusharmakkar08
Copy link
Contributor Author

tusharmakkar08 commented Oct 5, 2016

@dsoprea I feel it's better to create a local repo only otherwise checking out a repo is just unnecessary baggage in Pysvn code repository.

@tusharmakkar08
Copy link
Contributor Author

Hey @dsoprea ,

Any updates on this?

@dsoprea
Copy link
Owner

dsoprea commented Oct 19, 2016

It's already covered by the new "Admin" class I created and the test that I created for it. Sorry.. I didn't see this PR in order to respond to it.

@dsoprea dsoprea closed this Oct 19, 2016
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

Successfully merging this pull request may close these issues.

None yet

3 participants