Pylons
Quick Tutorial lacks full test coverage and has incorrect tests
Due to a lot of copy-pasta in the Quick Tutorial, many of its tests are incorrect and have less than full coverage.
The tutorial documentation should also be updated to reflect the coverage command below. In the introduction, we promise to cover coverage, but it never happens.
$VENV/bin/py.test tutorial/tests.py --cov-report term-missing --cov=tutorial -q
I plan to work on this issue during the PyCon sprints. Posting this here to notify any other sprinting members to consider this taken.
The GET request should also not have the password sent in the clear across the wire.
@cyphus: any progress on this? If someone were to pick this up, you already have some work done somewhere?
@zupo I'd say this issue is up for grabs. I've also added a label for DragonSprint.
Hey there. I have some partial work done on this issue but I'm glad to hand it over in case anyone wants to work on it. I'll push it up to a fork tonight and link to it here.
I cleaned up what I've done so far and pushed to a branch here: https://github.com/cyphus/pyramid/tree/2504-quick-tutorial-test-coverage
I'll be happy if we get this in, but I'm dropping it from the 1.8 milestone so as not to hold up the release.
Is anyone planning to work on this for 1.9 or should I push it off again?
I will be mostly offline during the summer so cannot focus on this before end of Sept.
Unless it gets picked up at PyCon sprints, then let's push it off again.
I'm probably releasing 1.9 before the sprints. At least that's my current plan.
