Repository navigation

Commit abbc6b2

Eleanor Boyd

authored and

committed

Feature branch testing overflow bug fix (microsoft#21812)

This merges in two PRs that were reverted because of a bug introduced that caused subprocess overflow. reverted PRs: microsoft#21667, microsoft#21682 This now implements these two PRs allowing for absolute testIds and an execObservable for the subprocess. This PR also adds a bug fix and functional tests to ensure this doesn't happen again. Since this PR is large, all items in it have already been reviewed as they were merged into the feature branch.

1 parent c35b6a3 commit abbc6b2Copy full SHA for abbc6b2

25 files changed

pythonFiles
- tests/pytestadapter
- vscode_pytest
  - __init__.py
src
- client/testing/testController
  - common
    - server.ts
    - types.ts
  - pytest
    - pytestDiscoveryAdapter.ts
    - pytestExecutionAdapter.ts
  - unittest
    - testDiscoveryAdapter.ts
    - testExecutionAdapter.ts
- testTestingRootWkspc
  - largeWorkspace
    - test_parameterized_subtest.py
  - smallWorkspace
    - test_simple.py
- test
  - linters
    - lint.functional.test.ts
  - mocks
    - helper.ts
    - mockChildProcess.ts
  - testing
    - common
      - testingAdapter.test.ts
    - testController

`‎pythonFiles/tests/pytestadapter/.data/root/tests/pytest.ini‎`

Whitespace-only changes.

`‎pythonFiles/tests/pytestadapter/.data/root/tests/test_a.py‎`

-Original file line number
+Diff line change
@@ @@ -0,0 +1,6 @@ @@
 +# Copyright (c) Microsoft Corporation. All rights reserved.
 +# Licensed under the MIT License.
++
++
 +def test_a_function():  # test_marker--test_a_function
 +    assert True

`‎pythonFiles/tests/pytestadapter/.data/root/tests/test_b.py‎`

-Original file line number
+Diff line change
@@ @@ -0,0 +1,6 @@ @@
 +# Copyright (c) Microsoft Corporation. All rights reserved.
 +# Licensed under the MIT License.
++
++
 +def test_b_function():  # test_marker--test_b_function
 +    assert True

`‎pythonFiles/tests/pytestadapter/expected_discovery_test_output.py‎`

Large diffs are not rendered by default.

Comments

(0)