Modify telemetry to contain trigger time as property #1850

Workflow file for this run

.github/workflows/pr-check.yml at d38561e

	name: PR/CI Check

	on:
	pull_request:
	push:
	branches-ignore:
	- main
	- release*

	env:
	NODE_VERSION: 18.17.1
	PYTHON_VERSION: '3.10' # YML treats 3.10 the number as 3.1, so quotes around 3.10
	MOCHA_REPORTER_JUNIT: true # Use the mocha-multi-reporters and send output to both console (spec) and JUnit (mocha-junit-reporter). Also enables a reporter which exits the process running the tests if it haven't already.
	ARTIFACT_NAME_VSIX: ms-python-insiders-vsix
	VSIX_NAME: ms-python-insiders.vsix
	TEST_RESULTS_DIRECTORY: .
	# Force a path with spaces and to test extension works in these scenarios
	# Unicode characters are causing 2.7 failures so skip that for now.
	special-working-directory: './path with spaces'
	special-working-directory-relative: 'path with spaces'

	jobs:
	build-vsix:
	name: Create VSIX
	runs-on: ubuntu-latest
	steps:
	- name: Checkout
	uses: actions/checkout@v4

	- name: Build VSIX
	uses: ./.github/actions/build-vsix
	with:
	node_version: ${{ env.NODE_VERSION}}
	vsix_name: ${{ env.VSIX_NAME }}
	artifact_name: ${{ env.ARTIFACT_NAME_VSIX }}

	lint:
	name: Lint
	runs-on: ubuntu-latest
	steps:
	- name: Checkout
	uses: actions/checkout@v4

	- name: Lint
	uses: ./.github/actions/lint
	with:
	node_version: ${{ env.NODE_VERSION }}

	check-types:
	name: Check Python types
	runs-on: ubuntu-latest
	steps:
	- name: Use Python ${{ env.PYTHON_VERSION }}
	uses: actions/setup-python@v5
	with:
	python-version: ${{ env.PYTHON_VERSION }}

	- name: Checkout
	uses: actions/checkout@v4

	- name: Install base Python requirements
	uses: brettcannon/pip-secure-install@v1
	with:
	options: '-t ./pythonFiles/lib/python --no-cache-dir --implementation py'

	- name: Install Jedi requirements
	uses: brettcannon/pip-secure-install@v1
	with:
	requirements-file: './pythonFiles/jedilsp_requirements/requirements.txt'
	options: '-t ./pythonFiles/lib/jedilsp --no-cache-dir --implementation py'

	- name: Install other Python requirements
	run: \|
	python -m pip --disable-pip-version-check install -t ./pythonFiles/lib/python --no-cache-dir --implementation py --no-deps --upgrade --pre debugpy
	python -m pip install --upgrade -r build/test-requirements.txt

	- name: Run Pyright
	uses: jakebailey/pyright-action@v2
	with:
	version: 1.1.308
	working-directory: 'pythonFiles'

	python-tests:
	name: Python Tests
	# The value of runs-on is the OS of the current job (specified in the strategy matrix below) instead of being hardcoded.
	runs-on: ${{ matrix.os }}
	defaults:
	run:
	working-directory: ${{ env.special-working-directory }}
	strategy:
	fail-fast: false
	matrix:
	# We're not running CI on macOS for now because it's one less matrix entry to lower the number of runners used,
	# macOS runners are expensive, and we assume that Ubuntu is enough to cover the Unix case.
	os: [ubuntu-latest, windows-latest]
	# Run the tests on the oldest and most recent versions of Python.
	python: ['3.8', '3.x'] # run for 3 pytest versions, most recent stable, oldest version supported and pre-release
	pytest-version: ['pytest', 'pytest@pre-release', 'pytest==6.2.0']

	steps:
	- name: Checkout
	uses: actions/checkout@v4
	with:
	path: ${{ env.special-working-directory-relative }}

	- name: Use Python ${{ matrix.python }}
	uses: actions/setup-python@v5
	with:
	python-version: ${{ matrix.python }}

	- name: Install specific pytest version
	if: matrix.pytest-version == 'pytest@pre-release'
	run: \|
	python -m pip install --pre pytest

	- name: Install specific pytest version
	if: matrix.pytest-version != 'pytest@pre-release'
	run: \|
	python -m pip install "${{ matrix.pytest-version }}"

	- name: Install specific pytest version
	run: python -m pytest --version
	- name: Install base Python requirements
	uses: brettcannon/pip-secure-install@v1
	with:
	requirements-file: '"${{ env.special-working-directory-relative }}/requirements.txt"'
	options: '-t "${{ env.special-working-directory-relative }}/pythonFiles/lib/python" --no-cache-dir --implementation py'

	- name: Install test requirements
	run: python -m pip install --upgrade -r build/test-requirements.txt

	- name: Run Python unit tests
	run: python pythonFiles/tests/run_all.py

	tests:
	name: Tests
	# The value of runs-on is the OS of the current job (specified in the strategy matrix below) instead of being hardcoded.
	runs-on: ${{ matrix.os }}
	defaults:
	run:
	working-directory: ${{ env.special-working-directory }}
	strategy:
	fail-fast: false
	matrix:
	# We're not running CI on macOS for now because it's one less matrix entry to lower the number of runners used,
	# macOS runners are expensive, and we assume that Ubuntu is enough to cover the Unix case.
	os: [ubuntu-latest, windows-latest]
	# Run the tests on the oldest and most recent versions of Python.
	python: ['3.x']
	test-suite: [ts-unit, venv, single-workspace, debugger, functional]

	steps:
	- name: Checkout
	uses: actions/checkout@v4
	with:
	path: ${{ env.special-working-directory-relative }}

	- name: Install Node
	uses: actions/setup-node@v4
	with:
	node-version: ${{ env.NODE_VERSION }}
	cache: 'npm'
	cache-dependency-path: ${{ env.special-working-directory-relative }}/package-lock.json

	- name: Install dependencies (npm ci)
	run: npm ci

	- name: Compile
	run: npx gulp prePublishNonBundle

	- name: Localization
	run: npx @vscode/l10n-dev@latest export ./src

	- name: Use Python ${{ matrix.python }}
	uses: actions/setup-python@v5
	with:
	python-version: ${{ matrix.python }}

	- name: Install debugpy
	run: \|
	# We need to have debugpy so that tests relying on it keep passing, but we don't need install_debugpy's logic in the test phase.
	python -m pip --disable-pip-version-check install -t ./pythonFiles/lib/python --no-cache-dir --implementation py --no-deps --upgrade --pre debugpy

	- name: Download get-pip.py
	run: \|
	python -m pip install wheel
	python -m pip install -r build/build-install-requirements.txt
	python ./pythonFiles/download_get_pip.py
	shell: bash

	- name: Install base Python requirements
	uses: brettcannon/pip-secure-install@v1
	with:
	requirements-file: '"${{ env.special-working-directory-relative }}/requirements.txt"'
	options: '-t "${{ env.special-working-directory-relative }}/pythonFiles/lib/python" --no-cache-dir --implementation py'

	- name: Install Jedi requirements
	uses: brettcannon/pip-secure-install@v1
	with:
	requirements-file: '"${{ env.special-working-directory-relative }}/pythonFiles/jedilsp_requirements/requirements.txt"'
	options: '-t "${{ env.special-working-directory-relative }}/pythonFiles/lib/jedilsp" --no-cache-dir --implementation py'

	- name: Install test requirements
	run: python -m pip install --upgrade -r build/test-requirements.txt

	- name: Install debugpy wheels (Python ${{ matrix.python }})
	run: \|
	python -m pip install wheel
	python -m pip --disable-pip-version-check install -r build/build-install-requirements.txt
	python ./pythonFiles/install_debugpy.py
	shell: bash
	if: matrix.test-suite == 'debugger'

	- name: Install functional test requirements
	run: python -m pip install --upgrade -r ./build/functional-test-requirements.txt
	if: matrix.test-suite == 'functional'

	- name: Prepare pipenv for venv tests
	env:
	TEST_FILES_SUFFIX: testvirtualenvs
	PYTHON_VIRTUAL_ENVS_LOCATION: './src/tmp/envPaths.json'
	shell: pwsh
	if: matrix.test-suite == 'venv'
	run: \|
	python -m pip install pipenv
	python -m pipenv run python ./build/ci/addEnvPath.py ${{ env.PYTHON_VIRTUAL_ENVS_LOCATION }} pipenvPath

	- name: Prepare poetry for venv tests
	env:
	TEST_FILES_SUFFIX: testvirtualenvs
	shell: pwsh
	if: matrix.test-suite == 'venv'
	run: \|
	python -m pip install poetry
	Move-Item -Path ".\build\ci\pyproject.toml" -Destination .
	poetry env use python

	- name: Prepare virtualenv for venv tests
	env:
	TEST_FILES_SUFFIX: testvirtualenvs
	PYTHON_VIRTUAL_ENVS_LOCATION: './src/tmp/envPaths.json'
	shell: pwsh
	if: matrix.test-suite == 'venv'
	run: \|
	python -m pip install virtualenv
	python -m virtualenv .virtualenv/
	if ('${{ matrix.os }}' -match 'windows-latest') {
	& ".virtualenv/Scripts/python.exe" ./build/ci/addEnvPath.py ${{ env.PYTHON_VIRTUAL_ENVS_LOCATION }} virtualEnvPath
	} else {
	& ".virtualenv/bin/python" ./build/ci/addEnvPath.py ${{ env.PYTHON_VIRTUAL_ENVS_LOCATION }} virtualEnvPath
	}

	- name: Prepare venv for venv tests
	env:
	TEST_FILES_SUFFIX: testvirtualenvs
	PYTHON_VIRTUAL_ENVS_LOCATION: './src/tmp/envPaths.json'
	shell: pwsh
	if: matrix.test-suite == 'venv' && startsWith(matrix.python, 3.)
	run: \|
	python -m venv .venv
	if ('${{ matrix.os }}' -match 'windows-latest') {
	& ".venv/Scripts/python.exe" ./build/ci/addEnvPath.py ${{ env.PYTHON_VIRTUAL_ENVS_LOCATION }} venvPath
	} else {
	& ".venv/bin/python" ./build/ci/addEnvPath.py ${{ env.PYTHON_VIRTUAL_ENVS_LOCATION }} venvPath
	}

	- name: Prepare conda for venv tests
	env:
	TEST_FILES_SUFFIX: testvirtualenvs
	PYTHON_VIRTUAL_ENVS_LOCATION: './src/tmp/envPaths.json'
	shell: pwsh
	if: matrix.test-suite == 'venv'
	run: \|
	# 1. For `*.testvirtualenvs.test.ts`
	if ('${{ matrix.os }}' -match 'windows-latest') {
	$condaPythonPath = Join-Path -Path $Env:CONDA -ChildPath python.exe
	$condaExecPath = Join-Path -Path $Env:CONDA -ChildPath Scripts \| Join-Path -ChildPath conda
	} else{
	$condaPythonPath = Join-Path -Path $Env:CONDA -ChildPath bin \| Join-Path -ChildPath python
	$condaExecPath = Join-Path -Path $Env:CONDA -ChildPath bin \| Join-Path -ChildPath conda
	}
	& $condaPythonPath ./build/ci/addEnvPath.py ${{ env.PYTHON_VIRTUAL_ENVS_LOCATION }} condaExecPath $condaExecPath
	& $condaPythonPath ./build/ci/addEnvPath.py ${{ env.PYTHON_VIRTUAL_ENVS_LOCATION }} condaPath
	& $condaExecPath init --all

	- name: Set CI_PYTHON_PATH and CI_DISABLE_AUTO_SELECTION
	run: \|
	echo "CI_PYTHON_PATH=python" >> $GITHUB_ENV
	echo "CI_DISABLE_AUTO_SELECTION=1" >> $GITHUB_ENV
	shell: bash
	if: matrix.test-suite != 'ts-unit'

	# Run TypeScript unit tests only for Python 3.X.
	- name: Run TypeScript unit tests
	run: npm run test:unittests
	if: matrix.test-suite == 'ts-unit' && startsWith(matrix.python, 3.)

	# The virtual environment based tests use the `testSingleWorkspace` set of tests
	# with the environment variable `TEST_FILES_SUFFIX` set to `testvirtualenvs`,
	# which is set in the "Prepare environment for venv tests" step.
	# We also use a third-party GitHub Action to install xvfb on Linux,
	# run tests and then clean up the process once the tests ran.
	# See https://github.com/GabrielBB/xvfb-action
	- name: Run venv tests
	env:
	TEST_FILES_SUFFIX: testvirtualenvs
	CI_PYTHON_VERSION: ${{ matrix.python }}
	uses: GabrielBB/xvfb-action@v1.6
	with:
	run: npm run testSingleWorkspace
	working-directory: ${{ env.special-working-directory }}
	if: matrix.test-suite == 'venv'

	- name: Run single-workspace tests
	env:
	CI_PYTHON_VERSION: ${{ matrix.python }}
	uses: GabrielBB/xvfb-action@v1.6
	with:
	run: npm run testSingleWorkspace
	working-directory: ${{ env.special-working-directory }}
	if: matrix.test-suite == 'single-workspace'

	- name: Run debugger tests
	env:
	CI_PYTHON_VERSION: ${{ matrix.python }}
	uses: GabrielBB/xvfb-action@v1.6
	with:
	run: npm run testDebugger
	working-directory: ${{ env.special-working-directory }}
	if: matrix.test-suite == 'debugger'

	# Run TypeScript functional tests
	- name: Run TypeScript functional tests
	run: npm run test:functional
	if: matrix.test-suite == 'functional'

	smoke-tests:
	name: Smoke tests
	# The value of runs-on is the OS of the current job (specified in the strategy matrix below) instead of being hardcoded.
	runs-on: ${{ matrix.os }}
	needs: [build-vsix]
	strategy:
	fail-fast: false
	matrix:
	# We're not running CI on macOS for now because it's one less matrix entry to lower the number of runners used,
	# macOS runners are expensive, and we assume that Ubuntu is enough to cover the UNIX case.
	os: [ubuntu-latest, windows-latest]
	steps:
	# Need the source to have the tests available.
	- name: Checkout
	uses: actions/checkout@v4

	- name: Smoke tests
	uses: ./.github/actions/smoke-tests
	with:
	node_version: ${{ env.NODE_VERSION }}
	artifact_name: ${{ env.ARTIFACT_NAME_VSIX }}

	### Coverage run
	coverage:
	name: Coverage
	# The value of runs-on is the OS of the current job (specified in the strategy matrix below) instead of being hardcoded.
	runs-on: ${{ matrix.os }}
	strategy:
	fail-fast: false
	matrix:
	# Only run coverage on linux for PRs
	os: [ubuntu-latest]

	steps:
	- name: Checkout
	uses: actions/checkout@v4

	- name: Install Node
	uses: actions/setup-node@v4
	with:
	node-version: ${{ env.NODE_VERSION }}
	cache: 'npm'

	- name: Install dependencies (npm ci)
	run: npm ci

	- name: Compile
	run: npx gulp prePublishNonBundle

	- name: Localization
	run: npx @vscode/l10n-dev@latest export ./src

	- name: Use Python ${{ env.PYTHON_VERSION }}
	uses: actions/setup-python@v5
	with:
	python-version: ${{ env.PYTHON_VERSION }}
	cache: 'pip'
	cache-dependency-path: \|
	requirements.txt
	pythonFiles/jedilsp_requirements/requirements.txt
	build/test-requirements.txt
	build/functional-test-requirements.txt

	- name: Install base Python requirements
	uses: brettcannon/pip-secure-install@v1
	with:
	options: '-t ./pythonFiles/lib/python --implementation py'

	- name: Install Jedi requirements
	uses: brettcannon/pip-secure-install@v1
	with:
	requirements-file: './pythonFiles/jedilsp_requirements/requirements.txt'
	options: '-t ./pythonFiles/lib/jedilsp --implementation py'

	- name: Install debugpy
	run: \|
	# We need to have debugpy so that tests relying on it keep passing, but we don't need install_debugpy's logic in the test phase.
	python -m pip --disable-pip-version-check install -t ./pythonFiles/lib/python --implementation py --no-deps --upgrade --pre debugpy

	- name: Install test requirements
	run: python -m pip install --upgrade -r build/test-requirements.txt

	- name: Install functional test requirements
	run: python -m pip install --upgrade -r ./build/functional-test-requirements.txt

	- name: Prepare pipenv for venv tests
	env:
	TEST_FILES_SUFFIX: testvirtualenvs
	PYTHON_VIRTUAL_ENVS_LOCATION: './src/tmp/envPaths.json'
	shell: pwsh
	run: \|
	python -m pip install pipenv
	python -m pipenv run python ./build/ci/addEnvPath.py ${{ env.PYTHON_VIRTUAL_ENVS_LOCATION }} pipenvPath

	- name: Prepare poetry for venv tests
	env:
	TEST_FILES_SUFFIX: testvirtualenvs
	shell: pwsh
	run: \|
	python -m pip install poetry
	Move-Item -Path ".\build\ci\pyproject.toml" -Destination .
	poetry env use python

	- name: Prepare virtualenv for venv tests
	env:
	TEST_FILES_SUFFIX: testvirtualenvs
	PYTHON_VIRTUAL_ENVS_LOCATION: './src/tmp/envPaths.json'
	shell: pwsh
	run: \|
	python -m pip install virtualenv
	python -m virtualenv .virtualenv/
	if ('${{ matrix.os }}' -match 'windows-latest') {
	& ".virtualenv/Scripts/python.exe" ./build/ci/addEnvPath.py ${{ env.PYTHON_VIRTUAL_ENVS_LOCATION }} virtualEnvPath
	} else {
	& ".virtualenv/bin/python" ./build/ci/addEnvPath.py ${{ env.PYTHON_VIRTUAL_ENVS_LOCATION }} virtualEnvPath
	}

	- name: Prepare venv for venv tests
	env:
	TEST_FILES_SUFFIX: testvirtualenvs
	PYTHON_VIRTUAL_ENVS_LOCATION: './src/tmp/envPaths.json'
	shell: pwsh
	run: \|
	python -m venv .venv
	if ('${{ matrix.os }}' -match 'windows-latest') {
	& ".venv/Scripts/python.exe" ./build/ci/addEnvPath.py ${{ env.PYTHON_VIRTUAL_ENVS_LOCATION }} venvPath
	} else {
	& ".venv/bin/python" ./build/ci/addEnvPath.py ${{ env.PYTHON_VIRTUAL_ENVS_LOCATION }} venvPath
	}

	- name: Prepare conda for venv tests
	env:
	TEST_FILES_SUFFIX: testvirtualenvs
	PYTHON_VIRTUAL_ENVS_LOCATION: './src/tmp/envPaths.json'
	shell: pwsh
	run: \|
	# 1. For `*.testvirtualenvs.test.ts`
	if ('${{ matrix.os }}' -match 'windows-latest') {
	$condaPythonPath = Join-Path -Path $Env:CONDA -ChildPath python.exe
	$condaExecPath = Join-Path -Path $Env:CONDA -ChildPath Scripts \| Join-Path -ChildPath conda
	} else{
	$condaPythonPath = Join-Path -Path $Env:CONDA -ChildPath bin \| Join-Path -ChildPath python
	$condaExecPath = Join-Path -Path $Env:CONDA -ChildPath bin \| Join-Path -ChildPath conda
	}
	& $condaPythonPath ./build/ci/addEnvPath.py ${{ env.PYTHON_VIRTUAL_ENVS_LOCATION }} condaExecPath $condaExecPath
	& $condaPythonPath ./build/ci/addEnvPath.py ${{ env.PYTHON_VIRTUAL_ENVS_LOCATION }} condaPath
	& $condaExecPath init --all

	- name: Run TypeScript unit tests
	run: npm run test:unittests:cover

	- name: Run Python unit tests
	run: \|
	python pythonFiles/tests/run_all.py

	# The virtual environment based tests use the `testSingleWorkspace` set of tests
	# with the environment variable `TEST_FILES_SUFFIX` set to `testvirtualenvs`,
	# which is set in the "Prepare environment for venv tests" step.
	# We also use a third-party GitHub Action to install xvfb on Linux,
	# run tests and then clean up the process once the tests ran.
	# See https://github.com/GabrielBB/xvfb-action
	- name: Run venv tests
	env:
	TEST_FILES_SUFFIX: testvirtualenvs
	CI_PYTHON_VERSION: ${{ env.PYTHON_VERSION }}
	CI_DISABLE_AUTO_SELECTION: 1
	uses: GabrielBB/xvfb-action@v1.6
	with:
	run: npm run testSingleWorkspace:cover

	- name: Run single-workspace tests
	env:
	CI_PYTHON_VERSION: ${{ env.PYTHON_VERSION }}
	CI_DISABLE_AUTO_SELECTION: 1
	uses: GabrielBB/xvfb-action@v1.6
	with:
	run: npm run testSingleWorkspace:cover

	# Enable these tests when coverage is setup for multiroot workspace tests
	# - name: Run multi-workspace tests
	# env:
	# CI_PYTHON_VERSION: ${{ env.PYTHON_VERSION }}
	# CI_DISABLE_AUTO_SELECTION: 1
	# uses: GabrielBB/xvfb-action@v1.6
	# with:
	# run: npm run testMultiWorkspace:cover

	# Enable these tests when coverage is setup for debugger tests
	# - name: Run debugger tests
	# env:
	# CI_PYTHON_VERSION: ${{ env.PYTHON_VERSION }}
	# CI_DISABLE_AUTO_SELECTION: 1
	# uses: GabrielBB/xvfb-action@v1.6
	# with:
	# run: npm run testDebugger:cover

	# Run TypeScript functional tests
	- name: Run TypeScript functional tests
	env:
	CI_PYTHON_VERSION: ${{ env.PYTHON_VERSION }}
	CI_DISABLE_AUTO_SELECTION: 1
	run: npm run test:functional:cover

	- name: Generate coverage reports
	run: npm run test:cover:report

	- name: Upload HTML report
	uses: actions/upload-artifact@v4
	with:
	name: ${{ runner.os }}-coverage-report-html
	path: ./coverage
	retention-days: 1

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Modify telemetry to contain trigger time as property #1850

Workflow file

Modify telemetry to contain trigger time as property #1850

Jobs

Run details

Workflow file for this run