We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
There was an error while loading. Please reload this page.
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
I was trying to add tests to #247 by modifying tests/helpers.CursorBase().
tests/helpers.CursorBase()
But then, when running py.test tests/test_pymssql.py they weren't discovered at all.
py.test tests/test_pymssql.py
It was then that I realized py.test is skipping them:
SKIP [1] /path/to/lib/python2.7/site-packages/_pytest/python.py:503: class tests.test_pymssql.TestCursor with __init__ won't get collected
(I saw that SKIP message before every time I ran the full suite but ignored it until now)
Reason is that py.test skips classes that have an __init__() method. This is an intended and documented behavior.
__init__()
This means there are 14 basic DB-API cursor test cases we aren't running.
I'm not sure how we could fix this while at the same time maintaining nose compatibility. Perhaps it's time we switch completely to py.test ?
The text was updated successfully, but these errors were encountered:
b818434
No branches or pull requests
I was trying to add tests to #247 by modifying
tests/helpers.CursorBase()
.But then, when running
py.test tests/test_pymssql.py
they weren't discovered at all.It was then that I realized py.test is skipping them:
(I saw that SKIP message before every time I ran the full suite but ignored it until now)
Reason is that py.test skips classes that have an
__init__()
method. This is an intended and documented behavior.This means there are 14 basic DB-API cursor test cases we aren't running.
I'm not sure how we could fix this while at the same time maintaining nose compatibility. Perhaps it's time we switch completely to py.test ?
The text was updated successfully, but these errors were encountered: