-
-
Notifications
You must be signed in to change notification settings - Fork 100
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
Convert the tests to use pytest #123
Comments
I would definitely like to use a more conventional test runner for What do others think? |
One of the best features of pytest is the composability that using fixtures bring as well as nice simple I wouldn't be scared of adding test dependencies though, for example the find_memleak.py could be considerably shorter and more readable if One last thing, I think it would be good to separate the tests out into two types: automated and manual. For example there should definitely be a test for automating Microsoft Word/Excel as this is one of the main reasons people use |
+1 for of breaking up the tests cleanly into automated and manual tests. Also for using Ultimately, anything that cleans up the tests and makes them easier to write and maintain is a good thing. |
This issue has not been in motion for years. Time has passed since this issue was submitted, and with contributions to improve the test, including #267, #271 and #298, the situation is much different now than it was then. There is also a more comprehensive discussion in #480 on whether we should introduce modern tools such as So, I will close this issue, and redirect the discussion to #480. Thank you. |
From the discussion in #121.
The text was updated successfully, but these errors were encountered: