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

Added traceInfo for server port #1823

Open
wants to merge 290 commits into
base: main
Choose a base branch
from

Conversation

lifelearner2
Copy link

Added trace info for server port so others can know which port is used.

Kartik Raj and others added 30 commits April 11, 2023 10:24
no change logs because this feature is not released yet.

Fixes microsoft#21039
Closes microsoft#20906

---------

Co-authored-by: Courtney Webster <60238438+cwebster-99@users.noreply.github.com>
closes microsoft#20078 and
closes microsoft#20085 (which is
about the testing work to support this code)

This logic now successfully works to discover the pytest repo tests.
This branch takes into account all the previous comments made to the
python discovery logic in the previous PR (located on my personal fork).
Therefore this is a second round of edits on this code. It now works for
the pytest library (discovers all the tests in the pytest library).

---------

Co-authored-by: Brett Cannon <brcan@microsoft.com>
Co-authored-by: Karthik Nadig <kanadig@microsoft.com>
…oft#21062)

In this PR:
1. Changes the python extension logging to use LogOutputChannel
2. Changes the language server logger with LogOutputChannel
3. Test output channel uses OutputChannel as it needs to show test
output and not really logging. Also, using logging test output makes it
pretty much unreadable.
4. Simplifies logging channel and output channel registration.

We need to do this now to make it easier for new test work to integrate
with output logging.

For microsoft#20844

This still doesn't get rid of the log level setting.
closes microsoft#20897
closes microsoft#20084
closes microsoft#20081

---------
Co-authored-by: Karthik Nadig <kanadig@microsoft.com>
closes microsoft#21038
also updates the subtests names

---------

Co-authored-by: Kartik Raj <karraj@microsoft.com>
Deprecates the logging level setting. The default for logging level from
VS Code is Info, so in this PR moved a few items to verbose so that we
get the same logging experience.

Closes microsoft#20844
I now overload the method signature for discovery and execution so
instead the only place where the new rewrite code needs to be enabled is
in the controller where it calls either the old method signature without
the`pythonExecFactory` or provides it to use the new code.
The UI string is currently using the internal variable name
(workspaceFolderToken) instead of the one actually used in the file
(workspaceFolder). This PR fixes that.
)

Co-authored-by: paulacamargo25 <paulitacv25@gmail.com>
eleanorjboyd and others added 30 commits September 11, 2023 09:37
)

an additional print statement was left in the pytest plugin which
unnecessarily printed all absolute paths calculated.
Drop Python 3.7 support, and replace with Python 3.8
Resolves: microsoft#21532

/vscode-python/requirements.txt generated same hash even when running:
```pip-compile --generate-hashes requirements.in```
from the Python3.8 virtual environment.

- Same result with pythonFiles/jedilsp_requirements/requirements.txt when running:
```pip-compile --generate-hashes
pythonFiles/jedilsp_requirements/requirements.in```
closes: microsoft#21662

Not only does this make sure segfaults are correct for unittest but also for pytest.
Adds support for the end of transmission (EOT) operator to all pytest
and unittest responses.

this PR includes:
- addition of an EOT that is added to run and discovery returns and
processed by the extension to initiate cleanup after run/discovery
finishes
- updates to all tests to support the use of EOT
- redesign of how cleanup works following run/discover to make it more
streamlined
- full functional tests that check multiple different types of payload
splitting from the buffer
- tests for the cancellation token during run and debug modes
…sions" (microsoft#22019)

There are other formatter options besides Black and Autopep8, but the
language of this notice suggests otherwise.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet