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

Create tests based on test files in scripts folder ($50) #75

Closed
iurisilvio opened this Issue Feb 10, 2013 · 4 comments

Comments

Projects
None yet
2 participants
@iurisilvio
Contributor

iurisilvio commented Feb 10, 2013

No description provided.

@campadrenalin

This comment has been minimized.

Show comment
Hide comment
@campadrenalin

campadrenalin Feb 10, 2013

Owner

Bounty is $50.

http://www.freedomsponsors.org/core/issue/170/create-tests-based-on-test-files-in-scripts-folder


(Copied from acceptance criteria)

Convert all test scripts into unittests. There may be some very old stuff in some scripts, so the scope of this issue is up for revision based on what you find - don't be afraid to ask questions!

Owner

campadrenalin commented Feb 10, 2013

Bounty is $50.

http://www.freedomsponsors.org/core/issue/170/create-tests-based-on-test-files-in-scripts-folder


(Copied from acceptance criteria)

Convert all test scripts into unittests. There may be some very old stuff in some scripts, so the scope of this issue is up for revision based on what you find - don't be afraid to ask questions!

@iurisilvio

This comment has been minimized.

Show comment
Hide comment
@iurisilvio

iurisilvio Feb 11, 2013

Contributor

Almost done in my branch.

https://github.com/iurisilvio/EJTP-lib-python/tree/test_scripts

Only Python 3.3 does not work because some issue with imp.load_module. Looks like a Python regression bug, but I will investigate.

https://travis-ci.org/iurisilvio/EJTP-lib-python/builds/4723643

Contributor

iurisilvio commented Feb 11, 2013

Almost done in my branch.

https://github.com/iurisilvio/EJTP-lib-python/tree/test_scripts

Only Python 3.3 does not work because some issue with imp.load_module. Looks like a Python regression bug, but I will investigate.

https://travis-ci.org/iurisilvio/EJTP-lib-python/builds/4723643

@iurisilvio

This comment has been minimized.

Show comment
Hide comment
@iurisilvio

iurisilvio Feb 12, 2013

Contributor

Solved the imp.load_module issue and will create a pull request now.

Contributor

iurisilvio commented Feb 12, 2013

Solved the imp.load_module issue and will create a pull request now.

@iurisilvio iurisilvio referenced this issue Feb 12, 2013

Merged

Test scripts #84

@campadrenalin

This comment has been minimized.

Show comment
Hide comment
@campadrenalin

campadrenalin Feb 12, 2013

Owner

Merged via #84 plus some of my own minor fixes.

Owner

campadrenalin commented Feb 12, 2013

Merged via #84 plus some of my own minor fixes.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment