Skip to content

Large subtest qty freezes some UI elements #21339

Closed
#21351
Closed
Large subtest qty freezes some UI elements#21339
#21351
Assignees
eleanorjboyd
Labels
area-testingbugIssue identified by VS Code Team member as probable bug

Description

@Yoyokrazy
Michael Lively (Yoyokrazy)
opened on May 30, 2023

Testing #21306

subtest with range (0, 10k) caused a number of UI elements to freeze while running, most apparent when resizing the window.

*note: all actions resolve and render properly immediately upon test completion

image image

While the 10k subtests are running, the window cannot be closed nor reloaded. UI elements can be interacted with, but nothing happens other than visually showing that they were clicked upon.

ex: folders showing as expanded, but not showing contents
image

other graphical issues
image

Activity

  1. Michael Lively (Yoyokrazy)
    assigned
    Eleanor Boyd (eleanorjboyd)
    on May 30, 2023
  2. github-actions
    added
    triage-neededNeeds assignment to the proper sub-team
    on May 30, 2023

  3. Yoyokrazy commented on May 30, 2023

    @Yoyokrazy
    Michael Lively (Yoyokrazy)
    on May 30, 2023
    Author

    1st run = success
    2nd run = failure + no running
    run with debug after first =
    image

  4. eleanorjboyd commented on May 30, 2023

    @eleanorjboyd
    Eleanor Boyd (eleanorjboyd)
    on May 30, 2023
    Member

    What I think is happening is it is sending a test_id per subtest but all the subtests have the same test_id. Therefore if there are 100 subtests, it is running 100 subtests, 100 times since it is running the subtest parent 100 times. This will be fixed if only unique test_ids are passed into the run command so then it will run the list of subtests only a single time. Here is a display of the logs, you can see it runs a single test first then attempts to run 100.

    python /Users/eleanorboyd/.vscode-insiders/extensions/ms-python.python-2023.9.11491011/pythonFiles/unittestadapter/discovery.py --port 64192 --uuid cf44fb3a-baf9-49a2-aa74-201e08ad72e2 --udiscovery -v -s . -p test_*.py
python /Users/eleanorboyd/.vscode-insiders/extensions/ms-python.python-2023.9.11491011/pythonFiles/unittestadapter/execution.py --port 64192 --uuid e4ad7d0b-e7f8-4ede-8fbb-870a67e4df11 --testids test_subtests.NumbersTest.test_even --udiscovery -v -s . -p test_*.py
.
----------------------------------------------------------------------
Ran 1 test in 0.091s

OK
python /Users/eleanorboyd/.vscode-insiders/extensions/ms-python.python-2023.9.11491011/pythonFiles/unittestadapter/discovery.py --port 64192 --uuid 22a6d774-a419-4e10-bc21-c8e095a52298 --udiscovery -v -s . -p test_*.py
python /Users/eleanorboyd/.vscode-insiders/extensions/ms-python.python-2023.9.11491011/pythonFiles/unittestadapter/execution.py --port 64192 --uuid 39a5d687-e14a-4bdf-9e18-fcf2b9ffd45a --testids test_subtests.NumbersTest.test_even --udiscovery -v -s . -p test_*.py
.
----------------------------------------------------------------------
Ran 1 test in 0.001s

OK
python /Users/eleanorboyd/.vscode-insiders/extensions/ms-python.python-2023.9.11491011/pythonFiles/unittestadapter/execution.py --port 64192 --uuid a4e94e4a-b5a5-48b9-ba5f-257efee1d613 --testids test_subtests.NumbersTest.test_even test_subtests.NumbersTest.test_even test_subtests.NumbersTest.test_even test_subtests.NumbersTest.test_even test_subtests.NumbersTest.test_even test_subtests.NumbersTest.test_even test_subtests.NumbersTest.test_even test_subtests.NumbersTest.test_even test_subtests.NumbersTest.test_even test_subtests.NumbersTest.test_even test_subtests.NumbersTest.test_even test_subtests.NumbersTest.test_even test_subtests.NumbersTest.test_even test_subtests.NumbersTest.test_even test_subtests.NumbersTest.test_even test_subtests.NumbersTest.test_even test_subtests.NumbersTest.test_even test_subtests.NumbersTest.test_even test_subtests.NumbersTest.test_even test_subtests.NumbersTest.test_even test_subtests.NumbersTest.test_even test_subtests.NumbersTest.test_even test_subtests.NumbersTest.test_even test_subtests.NumbersTest.test_even test_subtests.NumbersTest.test_even test_subtests.NumbersTest.test_even test_subtests.NumbersTest.test_even test_subtests.NumbersTest.test_even test_subtests.NumbersTest.test_even test_subtests.NumbersTest.test_even test_subtests.NumbersTest.test_even test_subtests.NumbersTest.test_even test_subtests.NumbersTest.test_even test_subtests.NumbersTest.test_even test_subtests.NumbersTest.test_even test_subtests.NumbersTest.test_even test_subtests.NumbersTest.test_even test_subtests.NumbersTest.test_even test_subtests.NumbersTest.test_even test_subtests.NumbersTest.test_even test_subtests.NumbersTest.test_even test_subtests.NumbersTest.test_even test_subtests.NumbersTest.test_even test_subtests.NumbersTest.test_even test_subtests.NumbersTest.test_even test_subtests.NumbersTest.test_even test_subtests.NumbersTest.test_even test_subtests.NumbersTest.test_even test_subtests.NumbersTest.test_even test_subtests.NumbersTest.test_even test_subtests.NumbersTest.test_even test_subtests.NumbersTest.test_even test_subtests.NumbersTest.test_even test_subtests.NumbersTest.test_even test_subtests.NumbersTest.test_even test_subtests.NumbersTest.test_even test_subtests.NumbersTest.test_even test_subtests.NumbersTest.test_even test_subtests.NumbersTest.test_even test_subtests.NumbersTest.test_even test_subtests.NumbersTest.test_even test_subtests.NumbersTest.test_even test_subtests.NumbersTest.test_even test_subtests.NumbersTest.test_even test_subtests.NumbersTest.test_even test_subtests.NumbersTest.test_even test_subtests.NumbersTest.test_even test_subtests.NumbersTest.test_even test_subtests.NumbersTest.test_even test_subtests.NumbersTest.test_even test_subtests.NumbersTest.test_even test_subtests.NumbersTest.test_even test_subtests.NumbersTest.test_even test_subtests.NumbersTest.test_even test_subtests.NumbersTest.test_even test_subtests.NumbersTest.test_even test_subtests.NumbersTest.test_even test_subtests.NumbersTest.test_even test_subtests.NumbersTest.test_even test_subtests.NumbersTest.test_even test_subtests.NumbersTest.test_even test_subtests.NumbersTest.test_even test_subtests.NumbersTest.test_even test_subtests.NumbersTest.test_even test_subtests.NumbersTest.test_even test_subtests.NumbersTest.test_even test_subtests.NumbersTest.test_even test_subtests.NumbersTest.test_even test_subtests.NumbersTest.test_even test_subtests.NumbersTest.test_even test_subtests.NumbersTest.test_even test_subtests.NumbersTest.test_even test_subtests.NumbersTest.test_even test_subtests.NumbersTest.test_even test_subtests.NumbersTest.test_even test_subtests.NumbersTest.test_even test_subtests.NumbersTest.test_even test_subtests.NumbersTest.test_even test_subtests.NumbersTest.test_even test_subtests.NumbersTest.test_even test_subtests.NumbersTest.test_even --udiscovery -v -s . -p test_*.py
.....................................................................................................
----------------------------------------------------------------------
Ran 101 tests in 0.082s

OK
  5. github-actions
    added
    info-neededIssue requires more information from poster
    on May 30, 2023
  6. Eleanor Boyd (eleanorjboyd)
    added a commit that references this issue on May 31, 2023

    remove duplicates from test_ids array (#21347)

    e9a8dd5
  7. Eleanor Boyd (eleanorjboyd)
    added
    bugIssue identified by VS Code Team member as probable bug
    area-testing
    and removed
    info-neededIssue requires more information from poster
    triage-neededNeeds assignment to the proper sub-team
    on Jun 1, 2023
  8. Eleanor Boyd (eleanorjboyd)
    closed this as completedin #21351on Jun 5, 2023
  9. Eleanor Boyd (eleanorjboyd)
    added 2 commits that reference this issue on Jun 5, 2023

    Unittest for large workspaces (#21351)

    be829b3

    Unittest for large workspaces (microsoft#21351)

    29fa3fc
  10. github-actions
    locked as resolved and limited conversation to collaborators on Jul 6, 2023
  11. Wes McKinney (wesm)
    added 2 commits that reference this issue on Mar 28, 2024

    Merged PR posit-dev/positron-python#122: Merging upstream vscode-pyth…

    bf63313

    Merged PR posit-dev/positron-python#122: Merging upstream vscode-pyth…

    c908a36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

Labels

area-testingbugIssue identified by VS Code Team member as probable bug

Type

No type

Projects

No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions