Skip to content

chore: Cleanup after integration test#2577

Merged
bitsandfoxes merged 90 commits intomainfrom
chore/cleanup-integration-test
Mar 17, 2026
Merged

chore: Cleanup after integration test#2577
bitsandfoxes merged 90 commits intomainfrom
chore/cleanup-integration-test

Conversation

@bitsandfoxes
Copy link
Contributor

We no longer need anything related to the old SmokeTester setup and the handling/fallback.

#skip-changelog

@github-actions
Copy link
Contributor

github-actions bot commented Mar 5, 2026

Semver Impact of This PR

🟢 Patch (bug fixes)

📋 Changelog Preview

This is how your changes will appear in the changelog.
Entries from this PR are highlighted with a left border (blockquote style).


This PR will not appear in the changelog.


🤖 This preview updates automatically when you update the PR.

Base automatically changed from feat/app-runner-webgl to main March 17, 2026 07:50
Copy link

@cursor cursor bot left a comment

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Autofix Details

Bugbot Autofix prepared a fix for the issue found in the latest run.

  • ✅ Fixed: Orphaned workflow file not deleted during rename
    • Deleted the orphaned smoke-test-run-webgl.yml file that was duplicated instead of renamed during the workflow cleanup.

Create PR

Or push these changes by commenting:

@cursor push 15cc715c9e
Preview (15cc715c9e)
diff --git a/.github/workflows/smoke-test-run-webgl.yml b/.github/workflows/smoke-test-run-webgl.yml
deleted file mode 100644
--- a/.github/workflows/smoke-test-run-webgl.yml
+++ /dev/null
@@ -1,54 +1,0 @@
-name: "IntegrationTest: Run WebGL"
-on:
-  workflow_call:
-    inputs:
-      unity-version:
-        required: true
-        type: string
-
-defaults:
-  run:
-    shell: pwsh
-
-jobs:
-  run:
-    name: WebGL ${{ inputs.unity-version }}
-    runs-on: ubuntu-latest
-    env:
-      SENTRY_TEST_DSN: ${{ secrets.SENTRY_TEST_DSN }}
-      SENTRY_AUTH_TOKEN: ${{ secrets.SENTRY_AUTH_TOKEN }}
-
-    steps:
-      - name: Checkout
-        uses: actions/checkout@11bd71901bbe5b1630ceea73d27597364c9af683 # v4
-
-      - name: Initialize app-runner submodule
-        run: git submodule update --init modules/app-runner
-        shell: bash
-
-      - name: Download test app artifact
-        uses: actions/download-artifact@d3f86a106a0bac45b974a628896c90dbdf5c8093 # v4
-        with:
-          name: testapp-webgl-compiled-${{ inputs.unity-version }}
-
-      - name: Extract test app
-        run: tar -xvzf test-app-webgl.tar.gz
-
-      - name: Install Selenium
-        run: pip3 install --upgrade selenium
-        shell: bash
-
-      - name: Run Integration Tests
-        timeout-minutes: 20
-        run: |
-          $env:SENTRY_WEBGL_BUILD_PATH = "samples/IntegrationTest/Build"
-          Invoke-Pester -Path test/IntegrationTest/Integration.Tests.WebGL.ps1 -CI
-
-      - name: Upload test results on failure
-        if: ${{ failure() }}
-        uses: actions/upload-artifact@ea165f8d65b6e75b540449e92b4886f43607fa02 # v4
-        with:
-          name: testapp-webgl-logs-${{ inputs.unity-version }}
-          path: |
-            test/IntegrationTest/results/
-          retention-days: 14
\ No newline at end of file

This Bugbot Autofix run was free. To enable autofix for future PRs, go to the Cursor dashboard.

Copy link

@cursor cursor bot left a comment

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Fix All in Cursor

Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

@bitsandfoxes bitsandfoxes merged commit 111a235 into main Mar 17, 2026
128 of 130 checks passed
@bitsandfoxes bitsandfoxes deleted the chore/cleanup-integration-test branch March 17, 2026 12:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants