Skip to content

Conversation

kondratyev-nv
Copy link

Broken with #16696

Test discovery is not working when using default unittest arguments.

Traceback (most recent call last):
  File "<string>", line 4, in <module>
  File "C:\Program Files\WindowsApps\PythonSoftwareFoundation.Python.3.9_3.9.1776.0_x64__qbz5n2kfra8p0\lib\unittest\loader.py", line 346, in discover
    raise ImportError('Start directory is not importable: %r' % start_dir)
ImportError: Start directory is not importable: '-v'

Related to kondratyev-nv/vscode-python-test-adapter#262 and kondratyev-nv/vscode-python-test-adapter#266 (comment).

@kondratyev-nv
Copy link
Author

btw I don't think it's worth a news entry. Let me know if this is still required.

@karthiknadig
Copy link
Member

@kondratyev-nv I created #16882. Please do and a news item. Thank you for this fix.

Copy link

@karrtikr karrtikr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please add a news entry. It's useful to track using it if something breaks.

@karthiknadig karthiknadig linked an issue Aug 6, 2021 that may be closed by this pull request
@karthiknadig karthiknadig added the skip package*.json package.json and package-lock.json don't both need updating label Aug 6, 2021
@karthiknadig karthiknadig merged commit b2f2091 into microsoft:main Aug 6, 2021
@kondratyev-nv kondratyev-nv deleted the fix_default_args branch August 6, 2021 16:39
karthiknadig pushed a commit to karthiknadig/vscode-python that referenced this pull request Aug 6, 2021
* Fix order of default unittest arguments

* Add news entry
karthiknadig pushed a commit that referenced this pull request Aug 6, 2021
* Fix order of default unittest arguments

* Add news entry
karthiknadig added a commit that referenced this pull request Aug 9, 2021
* Update version and change log. (#16868)

* Deprecate inExperiment API in favor of inExperimentSync (#16848)

* Deprecate inExperiment API in favour of inExperimentSync

* Ensure experiments are activated before they are used

* News entry

* Fix order of default unittest arguments (#16880)

* Fix order of default unittest arguments

* Add news entry

* Update version and change log

Co-authored-by: Kartik Raj <karraj@microsoft.com>
Co-authored-by: Nikolay Kondratyev <4085884+kondratyev-nv@users.noreply.github.com>
karthiknadig added a commit that referenced this pull request Aug 23, 2021
* Update version and change log. (#16868)

* Deprecate inExperiment API in favor of inExperimentSync (#16848)

* Deprecate inExperiment API in favour of inExperimentSync

* Ensure experiments are activated before they are used

* News entry

* Fix order of default unittest arguments (#16880)

* Fix order of default unittest arguments

* Add news entry

* Update version and change log

* Add webpack build for browser entrypoint (#16912)

* Get telemetry client and LS middleware working in the browser (#17010)

* Add typings/types to browser tsconfig

* Wait for pylance instead of failing in the browser (#17033) (#17036)

* Update version number and change log for point release (#17023)

* Update versions

* Update change log.

* Update CHANGELOG date

* Update version and change log for point release. (#17074)

* Update `vsce` to latest. (#17051)

* Update vscode-extension-telemetry to 0.2.9 (#17071)

* Update vscode-extension-telemetry to 0.2.9

* Remove todo

* Remove other todo

* Exact

* Update version and change log for point release.

Co-authored-by: Jake Bailey <5341706+jakebailey@users.noreply.github.com>

Co-authored-by: Kartik Raj <karraj@microsoft.com>
Co-authored-by: Nikolay Kondratyev <4085884+kondratyev-nv@users.noreply.github.com>
Co-authored-by: Jake Bailey <5341706+jakebailey@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

skip package*.json package.json and package-lock.json don't both need updating

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Default unittest arguments are in wrong order

3 participants