Skip to content
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

Rewrite testing adapter #17242

Open
20 of 22 tasks
kimadeline opened this issue Sep 1, 2021 · 22 comments
Open
20 of 22 tasks

Rewrite testing adapter #17242

kimadeline opened this issue Sep 1, 2021 · 22 comments
Assignees
Labels
area-testing feature-request Request for new features or functionality meta Issue that is tracking an overall project needs PR Ready to be worked on

Comments

@kimadeline
Copy link

kimadeline commented Sep 1, 2021

Meta-issue: #16936

Unittest

Pytest

Cleanup

  • Clean up unittest-related issues
  • Clean up pytest-related issues
  • Make sure there are tests for the test controller after the rewrite

Notes

Public issues

  1. python.testing.cwd ignored when discovering or running tests #8678
  2. Tests not discovered, but running tests anyway works fine. #16565
  3. Handle exclude lists in a test run request #16929
  4. Go to Test doesn't work for test class items in Test Explorer #17107
  5. Tests are discovered correctly but then an incorrect argument is supplied to PyTest, leading test-result discovery to fail (Git Submodules?) #17903
  6. Unittest discovery not working when sys.stdout is set in global scope #17957

Issues we expect this rewrite will fix

pytest feedback on how to things the "right way"

unittest missing features

@kimadeline kimadeline added feature-request Request for new features or functionality needs PR area-testing labels Sep 1, 2021
@kimadeline kimadeline added this to the September 2021 milestone Sep 1, 2021
@kimadeline kimadeline self-assigned this Sep 1, 2021
@github-actions github-actions bot added the triage-needed Needs assignment to the proper sub-team label Sep 1, 2021
@karthiknadig karthiknadig removed the triage-needed Needs assignment to the proper sub-team label Sep 2, 2021
@Bilalh
Copy link

Bilalh commented Sep 25, 2021

It would be nice if other python testing frameworks such as behave (a BDD framework) https://behave.readthedocs.io/en/stable/index.html were supported by the testing adapter.

@kimadeline
Copy link
Author

Hi @Bilalh, support for behave is not within scope of this issue. Feel free to open a feature request on the repo and we'll take a look at it. Thanks!

@karthiknadig karthiknadig removed this from the November 2021 milestone Nov 29, 2021
@eleanorjboyd
Copy link
Member

Hello everyone! We are wrapping up the rewrite of the testing infrastructure and heading into the stage of testing issues and seeing which ones remain. Currently, we are rolling out this new infrastructure in parallel and users can try out the new changes by setting "python.experiments.optInto": ["pythonTestAdapter"] in their user settings.json. Right now you must be on vscode insiders to test but we are in the process of switching all users to the rewrite incrementally so if you do not have insiders, watch our release notes to get updated on when it will begin to hit stable.

I will be triaging our backlog of testing issues and determining which issues still exist. In the meantime, if you have an issue that was waiting on the rewrite to potentially be fixed now is a great time to test it! If you do give it a try, please update the specific issue with the outcome (success or how it is still failing) so I can more quickly determine the ongoing issues following the rewrite. Thanks everyone for your interest and patience as we get this rolled out!

@JensMadsen
Copy link

JensMadsen commented Aug 21, 2024

"python.experiments.optInto": ["pythonTestAdapter"]

@eleanorjboyd I have enabled the pythonTestAdapter. I still see that the startup time is very long. I use unittest. When I run a single unit test stuff related to tests elsewhere (e.g. other folders) is initialized. Example: I run a straightforward test that tests some simple string gymnastics. Before the test starts I see in the output terminal that e.g. Pytorch is imported which takes a long time.

Please let me know if I can help in any way debugging this :-)

@eleanorjboyd
Copy link
Member

Hi @JensMadsen! Could you open a new issue and include some of these logs? From there I can help you out a bit more- thanks!

@JensMadsen
Copy link

Hi @JensMadsen! Could you open a new issue and include some of these logs? From there I can help you out a bit more- thanks!

@eleanorjboyd I will do that but I am unsure what logs you refer to.

@eleanorjboyd
Copy link
Member

If you go to the command palette and look for the command- python: report issue... that is the best way submit all the logs and info to us as a new issue so I can take a look. Hopefully that would work- thanks!

@JensMadsen
Copy link

@eleanorjboyd done #24022

@ssbarnea
Copy link

ssbarnea commented Dec 3, 2024

Apparently vscode complains about not being able to activate the experimental feature when configuring at folder level.... and guess what, most people use multi-root workspaces.

  "python.experiments.optInto": [
        "pythonTestAdapter"
    ],

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area-testing feature-request Request for new features or functionality meta Issue that is tracking an overall project needs PR Ready to be worked on
Projects
None yet
Development

No branches or pull requests