Skip to content

Fixes issue 1001. #1186

Closed
wants to merge 1 commit into from

3 participants

@mgrbyte
mgrbyte commented Nov 13, 2013

Provides sample source code for integration and functional tests.
Moved credit to Ian Bicking for WebTest to glossary.

@mgrbyte mgrbyte Fixes issues 1001.
Provides sample source code for integration and functional tests.
Moved credit to Ian Bicking for WebTest to glossary.
dbe6bd3
@mgrbyte
mgrbyte commented Nov 21, 2013

Wondering if someone could provide some feedback on this one?
If there's anything more I can do here, please let me know.

@mcdonc
Pylons Project member
mcdonc commented Nov 22, 2013

Sorry Matthew for the radio silence on this. I'll review soon.

@mgrbyte
mgrbyte commented Nov 22, 2013

Thanks Chris!

I think the actually doc changes themselves OK, not so sure about the inclusion of all the files the scaffold generates in necessary. Possibly better to just include the python files that define the tests and reduce bloat.

@tseaver
Pylons Project member
tseaver commented Nov 19, 2014

@mgrbyte +1 for excluding the scaffold files which are not pulled into Sphinx via .. literalinclude:: (i.e., setup.py, myapp/views.py, and myapp/tests.py).

@mgrbyte
mgrbyte commented Nov 19, 2014

@tseaver I'll see if I can reboot this one later on, thanks

@mgrbyte
mgrbyte commented Nov 19, 2014

@tseaver I created #1467 to replace this PR, closing.

@mgrbyte mgrbyte closed this Nov 19, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Something went wrong with that request. Please try again.