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

Create (Unit) Tests #42

Open
daknob opened this issue Oct 4, 2016 · 4 comments
Open

Create (Unit) Tests #42

daknob opened this issue Oct 4, 2016 · 4 comments

Comments

@daknob
Copy link
Owner

daknob commented Oct 4, 2016

It is very important to have tests written for TorPaste. Currently this has been ignored and manual testing was used instead, but the process has already caused some issues.

Which testing frameworks do you use for Python? Have you used Flask's testing framework?

@j11e
Copy link
Contributor

j11e commented Oct 4, 2016

http://flask.pocoo.org/docs/0.11/testing/ is probably a good starting point for research.

@abit2
Copy link

abit2 commented Oct 4, 2016

hey there i would to work on this

@j11e
Copy link
Contributor

j11e commented Oct 4, 2016

I don't see a problem with that! Just please keep us updated regarding your choices (testing framework, what you test, etc.), so that we can make sure we don't forget anything. Discussing before coding is better than asking you to change things when you send your pull request ;)

@daknob
Copy link
Owner Author

daknob commented Oct 5, 2016

Exactly, this is always a good thing: to discuss how we're going to solve an issue before we write the code. Unfortunately I think GitHub can only add repository owners as assignees, so I'm not quite sure if I can add you as one. In any case, keep us posted on your decisions, and we can discuss them together.

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

No branches or pull requests

3 participants