diff --git a/.github/actions/install-python/action.yml b/.github/actions/install-python/action.yml index 11cdc188f8c..e5678eb3cd9 100644 --- a/.github/actions/install-python/action.yml +++ b/.github/actions/install-python/action.yml @@ -1,32 +1,8 @@ name: "Setup Python" -description: "Install specified Python version." -inputs: - version: - description: "The version of Python to install (e.g., 3.10)" - required: true - default: "3.10" +description: "Install Python dependencies." runs: using: "composite" steps: - - name: Install Specified Python Version - shell: bash - run: | - # Extract Python version from input - PYTHON_VERSION="${{ inputs.version }}" - - # Add the deadsnakes PPA repository and update apt - sudo add-apt-repository -y ppa:deadsnakes/ppa - sudo apt-get update - - # Install the specific Python version and necessary packages - sudo apt-get install -y \ - python${PYTHON_VERSION} \ - python${PYTHON_VERSION}-dev \ - python${PYTHON_VERSION}-venv - - # Set up update-alternatives for managing multiple Python versions - sudo update-alternatives --install /usr/bin/python3 python3 /usr/bin/python${PYTHON_VERSION} 1 - - name: Install Python dependencies shell: bash run: | diff --git a/.github/actions/setup-test-env/action.yml b/.github/actions/setup-test-env/action.yml index 6e0609b66d5..8e337d53350 100644 --- a/.github/actions/setup-test-env/action.yml +++ b/.github/actions/setup-test-env/action.yml @@ -44,8 +44,6 @@ runs: - name: Setup Python uses: ./.github/actions/install-python - with: - version: "3.10" - name: Setup R uses: ./.github/actions/install-r diff --git a/.github/workflows/test-e2e-linux.yml b/.github/workflows/test-e2e-linux.yml index 95d5c437e7a..5ba3a65bf1b 100644 --- a/.github/workflows/test-e2e-linux.yml +++ b/.github/workflows/test-e2e-linux.yml @@ -127,9 +127,12 @@ jobs: uses: ./.github/actions/gen-report-dir if: ${{ !cancelled() }} + - name: Alter AppArmor Restrictions for Playwright (Electron) + run: sudo sysctl -w kernel.apparmor_restrict_unprivileged_userns=0 + - name: Run Playwright Tests (Electron) env: - POSITRON_PY_VER_SEL: 3.10.12 + POSITRON_PY_VER_SEL: 3.12.3 POSITRON_R_VER_SEL: 4.4.0 CURRENTS_RECORD_KEY: ${{ secrets.CURRENTS_RECORD_KEY }} CURRENTS_CI_BUILD_ID: ${{ github.run_id }}-${{ github.run_attempt }} diff --git a/.github/workflows/test-e2e-release.yml b/.github/workflows/test-e2e-release.yml index a483a4de9bc..d5b8239915d 100644 --- a/.github/workflows/test-e2e-release.yml +++ b/.github/workflows/test-e2e-release.yml @@ -102,7 +102,7 @@ jobs: - name: Run Tests (Electron) if: ${{ !cancelled() }} env: - POSITRON_PY_VER_SEL: 3.10.12 + POSITRON_PY_VER_SEL: 3.12.3 POSITRON_R_VER_SEL: 4.4.0 CURRENTS_RECORD_KEY: ${{ secrets.CURRENTS_RECORD_KEY }} CURRENTS_CI_BUILD_ID: ${{ github.run_id }}-${{ github.run_attempt }} diff --git a/scripts/test-integration.sh b/scripts/test-integration.sh index 5d690998686..5e5632f720c 100755 --- a/scripts/test-integration.sh +++ b/scripts/test-integration.sh @@ -8,7 +8,7 @@ else ROOT=$(dirname $(dirname $(readlink -f $0))) # --disable-dev-shm-usage: when run on docker containers where size of /dev/shm # partition < 64MB which causes OOM failure for chromium compositor that uses the partition for shared memory - LINUX_EXTRA_ARGS="--disable-dev-shm-usage" + LINUX_EXTRA_ARGS="--disable-dev-shm-usage --no-sandbox" fi VSCODEUSERDATADIR=`mktemp -d 2>/dev/null` @@ -71,7 +71,7 @@ npm run test-extension -- -l vscode-colorize-tests kill_app echo -echo "### Terminal Suggest tests" +echo "### Terminal Suggest tests" echo npm run test-extension -- -l terminal-suggest --enable-proposed-api=vscode.vscode-api-tests kill_app diff --git a/scripts/test-remote-integration.sh b/scripts/test-remote-integration.sh index 4695dfa12e3..8e06b404884 100755 --- a/scripts/test-remote-integration.sh +++ b/scripts/test-remote-integration.sh @@ -8,7 +8,7 @@ else ROOT=$(dirname $(dirname $(readlink -f $0))) # --disable-dev-shm-usage: when run on docker containers where size of /dev/shm # partition < 64MB which causes OOM failure for chromium compositor that uses the partition for shared memory - LINUX_EXTRA_ARGS="--disable-dev-shm-usage" + LINUX_EXTRA_ARGS="--disable-dev-shm-usage --no-sandbox" fi VSCODEUSERDATADIR=`mktemp -d 2>/dev/null` diff --git a/test/e2e/tests/data-explorer/data-explorer-python-pandas.test.ts b/test/e2e/tests/data-explorer/data-explorer-python-pandas.test.ts index ae3180275f1..0f40cc78a20 100644 --- a/test/e2e/tests/data-explorer/data-explorer-python-pandas.test.ts +++ b/test/e2e/tests/data-explorer/data-explorer-python-pandas.test.ts @@ -179,8 +179,19 @@ df2 = pd.DataFrame(data)`; test('Python - Open Data Explorer for the second time brings focus back [C1078833]', async function ({ app, python }) { const script = `import pandas as pd -from pydataset import data -Data_Frame = data('mtcars')`; +Data_Frame = pd.DataFrame({ + "mpg": [21.0, 21.0, 22.8, 21.4, 18.7], + "cyl": [6, 6, 4, 6, 8], + "disp": [160.0, 160.0, 108.0, 258.0, 360.0], + "hp": [110, 110, 93, 110, 175], + "drat": [3.90, 3.90, 3.85, 3.08, 3.15], + "wt": [2.62, 2.875, 2.32, 3.215, 3.44], + "qsec": [16.46, 17.02, 18.61, 19.44, 17.02], + "vs": [0, 0, 1, 1, 0], + "am": [1, 1, 1, 0, 0], + "gear": [4, 4, 4, 3, 3], + "carb": [4, 4, 1, 1, 2] +})`; await app.workbench.console.executeCode('Python', script, '>>>'); await app.workbench.quickaccess.runCommand('workbench.panel.positronVariables.focus'); diff --git a/test/e2e/tests/plots/autos-linux.png b/test/e2e/tests/plots/autos-linux.png index e324426e6ce..25942b74712 100644 Binary files a/test/e2e/tests/plots/autos-linux.png and b/test/e2e/tests/plots/autos-linux.png differ diff --git a/test/e2e/tests/plots/graphviz-linux.png b/test/e2e/tests/plots/graphviz-linux.png index fb875079dd0..9601462bdc8 100644 Binary files a/test/e2e/tests/plots/graphviz-linux.png and b/test/e2e/tests/plots/graphviz-linux.png differ diff --git a/test/e2e/tests/plots/pythonScatterplot-linux.png b/test/e2e/tests/plots/pythonScatterplot-linux.png index 97388812dd0..96c98028975 100644 Binary files a/test/e2e/tests/plots/pythonScatterplot-linux.png and b/test/e2e/tests/plots/pythonScatterplot-linux.png differ