-
Notifications
You must be signed in to change notification settings - Fork 37
Add a sixth tutorial: The Testing Game #45
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
Conversation
|
Added some more details to this one. |
…his report, wrote on html coverage report
tutorials/06-the-testing-game.rst
Outdated
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
See http://nedbatchelder.com/code/coverage/branch.html
for a good example:
def my_partial_fn(x): # line 1
if x: # 2
y = 10 # 3
return y # 4
my_partial_fn(1)
This directory should not be stored under version control. It is a directory of meta content generated based on our code so it would be listed in a .gitignore in a real project.
|
I have made a lot of progress the first part of this extra tutorial. Anyone want to give it a look? I'll be working on the second part of the tutorial tomorrow. |
Also: - use "detail" consistently for DetailView (not details) - rename home page tests
- myblog.sqlite3 is now db.sqlite3
|
Hoping that my brain is more engaged this morning and that I pushed to the right place vs. the crazy pull of last night. If I messed up again, I'm missing a workflow nuance :) |
|
@willingc ah I didn't realize your pull request included this one. The changes look great. I like the shortened explanations. I made a couple more changes on top of yours. Now we just need to decide on how/whether the Gravatar tutorial will work. I can pull that out and make that part of a separate pull request if that makes more sense. |
|
@treyhunner Thanks for cleaning up the formatting. It might be best to create a new branch for the gravatar tutorial. The gravatar tutorial could add the gravatar code (which you have mostly done); have the user run coverage to see that the coverage decreases; then add tests; run coverage again to see that coverage increases. That would cover the open tasks for this issue within the gravatar tutorial :) |
|
@willingc I removed the gravatar tutorial. I think this may be ready for merging now. |
|
Merged the testing game tutorial after wrestling with some very odd merge conflicts. |
.coveragercRelated to issue #36.