Repository navigation
Test Unittest Rewrite #21092
Description
Refs: (overarching issue) #19098
- macOS Don Jayamanne (@DonJayamanne)
- linux Karthik Nadig (@karthiknadig)
- windows Michael Lively (@Yoyokrazy)
- anyOS Ladislau Szomoru (@lszomoru)
Complexity: 5
Author: Eleanor Boyd (@eleanorjboyd)
Thank you for doing this testing! I apologize for how long it is and please let me know if you have any questions!
Prerequisites:
- Use the following repo of tests or use your own repo of pytests https://github.com/eleanorjboyd/inc_dec_example_repo
Background Information:
This rewrite is for the underlying infrastructure of how python handles testing therefore all behavior visible to the user should be pretty much the same (minus maybe some error logs going in a better place or test names shortened). Therefore throughout this testing, you can reference the current testing in Python to compare if you think something doesn't look right (or you can always post here too!). Secondly, debugging is not supported yet so there is no need to test debugging on the rewrite which will also be reflected in the TPI steps.
Steps:
Part 0: Setup
- launch vscode with the environment variable
ENABLE_PYTHON_TESTING_REWRITE = '1' - Now open the
inc_dec_example_repo/inc_dec_example_test_suitetesting repo folder in vscode - use the
create environmentcommand from the command palette, select venv and whatever python version you want >=3.7 - in the terminal run
python -m pip install pytest - from the command palette run
python: configure testsselectunittestand"." (root directory)andtest_*.py
Part 1: General
- Go to the testing explorer, and confirm all the tests are discovered and nested correctly (please see attached image for reference on how it should look or compare it to the stable version of the extension since it will look the same)
- There are two types of test cases in unittest, regular test cases and subtests test cases (examples of both are in the test repo). Please try the following for both
- Add a test case, reload
- Remove a test case, reload
- Run via the gutter
- Run via the test explorer (individually as as part of a folder)
- Run select, non-adjacent tests (cmd+click to select individual tests)
- Now go to the output tab and select
python: output logs, run tests and confirm that the output appears with all the necessary information (how many tests were run, which succeeded, explanation of failure if that occurred)
Part 2: Error Handling
- Make one of your test files have a syntax error then run test discovery
- Discover tests, then make a syntax error, then try and run the test (check error message appears)
Part 3: Adding Unittest Args
- Add
-vto your pytest args in your settings.json and run the test folder, check that this arg worked (it works if it stops after a single test failure)
Part 4: Additional Commands
- Try out different commands from the command palette related to testing to see if they work
- Try out the different items on the top righthand side of the testing panel such as
collapse test casesandclear all results
Part 5: Different Testing Pattern
- Rerun the command
python: configure testsselectunittestand"." (root directory)now instead select*test.py - reload the window and go to test discovery
- confirm that the only test files that are found are the ones with the matching pattern, here is a screenshot of how it should look
test explorer reference image (from Part 1)


Activity
lszomoru commentedon Apr 25, 2023
Eleanor Boyd (@eleanorjboyd), I was wondering if you could provide some assistance as I have the feeling that I am missing something. The GitHub repository linked in the TPI, contains only one folder - "inc_dec_example_test_suite". The TPI states that one should open the repository folder in VS Code, but if I do that, even if I go through the setup steps I get the following in the Test Explorer:
eleanorjboyd commentedon Apr 25, 2023
hi Ladislau Szomoru (@lszomoru), sorry for the confusion! If you could instead open the folder "inc_dec_example_test_suite" as the workspace that should make the rest of the steps work as desired- sorry!
DonJayamanne commentedon Apr 25, 2023
Ok, thats the same issue I'm running into, been scratching my head for a while now
DonJayamanne commentedon Apr 25, 2023
I have updated the TPI accordingly, could help others.