From d10548b37cfcf919bebb44d0523a45f7069d8c83 Mon Sep 17 00:00:00 2001 From: Tyler Jang Date: Thu, 11 Jul 2024 12:24:36 -0700 Subject: [PATCH] remove ci debugger --- .github/actions/action_tests/action.yaml | 19 +------------------ .github/actions/linter_tests/action.yaml | 24 +----------------------- .github/actions/tool_tests/action.yaml | 20 +------------------- tests/driver/driver.ts | 4 ++-- 4 files changed, 5 insertions(+), 62 deletions(-) diff --git a/.github/actions/action_tests/action.yaml b/.github/actions/action_tests/action.yaml index 2da8737b8..5c7e7cd3c 100644 --- a/.github/actions/action_tests/action.yaml +++ b/.github/actions/action_tests/action.yaml @@ -17,7 +17,7 @@ inputs: required: false default: actions/ -- trunk-token: - description: CI debugger api token + description: Test analytics api token required: true runs: @@ -49,7 +49,6 @@ runs: working-directory: ${{ inputs.path }} - name: Run action tests - if: runner.os == 'Windows' run: npm test ${{ inputs.append-args }} ${{ env.PLATFORM_APPEND_ARGS }} --passWithNoTests --ci shell: bash working-directory: ${{ inputs.path }} @@ -58,22 +57,6 @@ runs: PLUGINS_TEST_CLI_VERSION: ${{ inputs.cli-version }} PLUGINS_TEST_CLI_PATH: ${{ env.CLI_PATH }} - - name: Run action tests - if: runner.os != 'Windows' - uses: trunk-io/breakpoint@v1.3.0 - with: - breakpoint-id: trunk-plugins-action-tests - shell: bash - working-directory: ${{ inputs.path }} - trunk-token: ${{ inputs.trunk-token }} - org: trunk-staging-org - run: - npm test ${{ inputs.append-args }} ${{ env.PLATFORM_APPEND_ARGS }} --passWithNoTests --ci - env: - JEST_SUITE_NAME: Action Tests - PLUGINS_TEST_CLI_VERSION: ${{ inputs.cli-version }} - PLUGINS_TEST_CLI_PATH: ${{ env.CLI_PATH }} - - name: Upload results # TODO(Tyler): Add upload on MacOS/Windows once the action supports it. if: "!cancelled() && runner.os == 'Linux'" diff --git a/.github/actions/linter_tests/action.yaml b/.github/actions/linter_tests/action.yaml index 16e09101f..0165007e6 100644 --- a/.github/actions/linter_tests/action.yaml +++ b/.github/actions/linter_tests/action.yaml @@ -24,7 +24,7 @@ inputs: description: Token to login for sourcery test required: true trunk-token: - description: CI debugger api token (org token) + description: Test analytics api token (org token) required: true ref-type: description: release or main @@ -92,7 +92,6 @@ runs: working-directory: ${{ inputs.path }} - name: Run plugin tests - if: runner.os == 'Windows' run: npm test ${{ inputs.append-args }} ${{ env.PLATFORM_APPEND_ARGS }} --ci shell: bash working-directory: ${{ inputs.path }} @@ -107,27 +106,6 @@ runs: JEST_JUNIT_SUITE_NAME: "{title} ${{ runner.os }} ${{ inputs.ref-type }} ${{ inputs.linter-version }}" - - name: Run plugin tests - if: runner.os != 'Windows' - # trunk-ignore(semgrep/yaml.github-actions.security.third-party-action-not-pinned-to-commit-sha.third-party-action-not-pinned-to-commit-sha) - uses: trunk-io/breakpoint@v1.3.0 - with: - breakpoint-id: trunk-plugins-linter-tests - shell: bash - working-directory: ${{ inputs.path }} - trunk-token: ${{ inputs.trunk-token }} - org: trunk-staging-org - run: npm test ${{ inputs.append-args }} ${{ env.PLATFORM_APPEND_ARGS }} --ci --runInBand - env: - PLUGINS_TEST_LINTER_VERSION: ${{ inputs.linter-version }} - PLUGINS_TEST_CLI_VERSION: ${{ inputs.cli-version }} - PLUGINS_TEST_CLI_PATH: ${{ env.CLI_PATH }} - SOURCERY_TOKEN: ${{ inputs.sourcery-token }} - DEBUG: Driver:nixpkgs-fmt:*, Driver:eslint:* - JEST_SUITE_NAME: Linter Tests - JEST_JUNIT_SUITE_NAME: - "{title} ${{ runner.os }} ${{ inputs.ref-type }} ${{ env.JEST_LINTER_VERSION }}" - - name: Upload results # TODO(Tyler): Add upload on MacOS/Windows once the action supports it. if: "!cancelled() && runner.os == 'Linux'" diff --git a/.github/actions/tool_tests/action.yaml b/.github/actions/tool_tests/action.yaml index 9d36c17e3..5366e2c28 100644 --- a/.github/actions/tool_tests/action.yaml +++ b/.github/actions/tool_tests/action.yaml @@ -17,7 +17,7 @@ inputs: required: false default: tools -- trunk-token: - description: CI debugger api token (org token) + description: Test analytics api token (org token) required: true ref-type: description: release or main @@ -65,7 +65,6 @@ runs: working-directory: ${{ inputs.path }} - name: Run plugin tests - if: runner.os == 'Windows' run: npm test ${{ inputs.append-args }} ${{ env.PLATFORM_APPEND_ARGS }} --ci shell: bash working-directory: ${{ inputs.path }} @@ -75,23 +74,6 @@ runs: JEST_SUITE_NAME: Tool Tests JEST_JUNIT_SUITE_NAME: "{title} ${{ runner.os }} ${{ inputs.ref-type }}" - - name: Run plugin tests - if: runner.os != 'Windows' - # trunk-ignore(semgrep/yaml.github-actions.security.third-party-action-not-pinned-to-commit-sha.third-party-action-not-pinned-to-commit-sha) - uses: trunk-io/breakpoint@v1.3.0 - with: - breakpoint-id: trunk-plugins-tool-tests - shell: bash - working-directory: ${{ inputs.path }} - trunk-token: ${{ inputs.trunk-token }} - org: trunk-staging-org - run: npm test ${{ inputs.append-args }} ${{ env.PLATFORM_APPEND_ARGS }} --ci - env: - PLUGINS_TEST_CLI_VERSION: ${{ inputs.cli-version }} - PLUGINS_TEST_CLI_PATH: ${{ env.CLI_PATH }} - JEST_SUITE_NAME: Tool Tests - JEST_JUNIT_SUITE_NAME: "{title} ${{ runner.os }} ${{ inputs.ref-type }}" - - name: Upload results # TODO(Tyler): Add upload on MacOS/Windows once the action supports it. if: "!cancelled() && runner.os == 'Linux'" diff --git a/tests/driver/driver.ts b/tests/driver/driver.ts index 478ca93b4..6716ecc9a 100644 --- a/tests/driver/driver.ts +++ b/tests/driver/driver.ts @@ -31,8 +31,8 @@ const UNINITIALIZED_ERROR = `You have attempted to modify the sandbox before it Please call this method after setup has been called.`; export const executionEnv = (sandbox: string) => { - // trunk-ignore(eslint/@typescript-eslint/no-unused-vars): Strip TRUNK_CLI_VERSION from CI-Debugger - const { PWD, INIT_CWD, TRUNK_CLI_VERSION, ...strippedEnv } = process.env; + // trunk-ignore(eslint/@typescript-eslint/no-unused-vars): Remove vars. + const { PWD, INIT_CWD, ...strippedEnv } = process.env; return { ...strippedEnv, // This keeps test downloads separate from manual trunk invocations