chore: Add test workflow for swift log #6945
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,138 @@ | ||
| name: Test Integrations | ||
| on: | ||
| push: | ||
| branches: | ||
| - main | ||
| - release/** | ||
|
|
||
| pull_request: | ||
| types: [opened, synchronize, reopened, labeled] | ||
|
|
||
| # Concurrency configuration: | ||
| # - We use workflow-specific concurrency groups to allow independent test runs across different workflows | ||
| # while preventing multiple runs of the same test suite on the same branch/commit. | ||
| # - For pull requests, we cancel in-progress runs when new commits are pushed to save CI resources | ||
| # and provide faster feedback on the latest changes. | ||
| # - For main branch pushes and scheduled runs, we never cancel in-progress runs to ensure the complete | ||
| # test suite always finishes, maintaining the integrity of our main branch quality gates. | ||
| concurrency: | ||
| group: ${{ github.workflow }}-${{ github.ref }} | ||
| cancel-in-progress: ${{ github.event_name == 'pull_request' }} | ||
|
|
||
| jobs: | ||
| ready-to-merge-gate: | ||
| name: Ready-to-merge gate | ||
| uses: ./.github/workflows/ready-to-merge-workflow.yml | ||
|
|
||
| # This job detects if the PR contains changes that require running unit tests. | ||
| # If yes, the job will output a flag that will be used by the next job to run the unit tests. | ||
| # If no, the job will output a flag that will be used by the next job to skip running the unit tests. | ||
| # At the end of this workflow, we run a check that validates that either all unit tests passed or were | ||
| # called unit-tests-required-check. | ||
| files-changed: | ||
| name: Detect File Changes | ||
| runs-on: ubuntu-latest | ||
| needs: ready-to-merge-gate | ||
| # Map a step output to a job output | ||
| outputs: | ||
| run_swift_log_for_prs: ${{ steps.changes.outputs.run_swift_log_for_prs }} | ||
| steps: | ||
| - uses: actions/checkout@v6 | ||
| - name: Get changed files | ||
| id: changes | ||
| uses: dorny/paths-filter@de90cc6fb38fc0963ad72b210f1f284cd68cea36 # v3.0.2 | ||
| with: | ||
| token: ${{ github.token }} | ||
| filters: .github/file-filters.yml | ||
|
|
||
| # TODO: All tests here are validating against the released SPM package, so we need to build the XCFramework first. | ||
| spm-integrations-tests: | ||
| name: "SPM Tests for Swift Log" | ||
| needs: files-changed | ||
| if: github.event_name != 'pull_request' || needs.files-changed.outputs.run_swift_log_for_prs == 'true' | ||
| runs-on: macos-15 | ||
| timeout-minutes: 30 | ||
| steps: | ||
| - uses: actions/checkout@v6 | ||
| - name: Select Xcode version | ||
| env: | ||
| XCODE_VERSION: 16.4 | ||
| run: ./scripts/ci-select-xcode.sh "$XCODE_VERSION" | ||
| - name: Prepare Package.swift | ||
| working-directory: 3rd-party-integrations/SwiftLog | ||
| shell: bash | ||
| run: | | ||
| ../../scripts/prepare-package.sh \ | ||
| --package-file Package.swift \ | ||
| --update-path-to-sentry-cocoa true | ||
| - name: Run SPM Tests | ||
| working-directory: 3rd-party-integrations/SwiftLog | ||
| run: swift test | ||
|
|
||
| test-integration: | ||
| name: Unit Tests for Swift Log | ||
| if: github.event_name != 'pull_request' || needs.files-changed.outputs.run_swift_log_for_prs == 'true' | ||
| needs: files-changed | ||
| uses: ./.github/workflows/unit-test-common.yml | ||
| with: | ||
| name: SwiftLog | ||
| scheme: SentrySwiftLog | ||
| working_directory: 3rd-party-integrations/SwiftLog | ||
| path_to_scripts_folder: ../../scripts | ||
| xcode: ${{ matrix.xcode }} | ||
| test-destination-os: ${{ matrix.test-destination-os }} | ||
| device: ${{ matrix.device }} | ||
| platform: ${{ matrix.platform }} | ||
| runs-on: ${{ matrix.macos_version }} | ||
| timeout: 10 | ||
| update_path_to_sentry_cocoa: true | ||
| test_spm_package: true | ||
| strategy: | ||
| fail-fast: false | ||
| matrix: | ||
| include: | ||
| - name: iOS 18 | ||
| xcode: "16.4" | ||
| macos_version: macos-15 | ||
| test-destination-os: "18.4" | ||
| platform: "iOS" | ||
| device: "iPhone 16 Pro" | ||
|
|
||
| - name: iOS 26 | ||
| macos_version: macos-26 | ||
| xcode: "26.1.1" | ||
| test-destination-os: "26.1" | ||
| platform: "iOS" | ||
| device: "iPhone 17 Pro" | ||
|
|
||
| - name: macOS 15 | ||
| macos_version: macos-15 | ||
| xcode: "16.4" | ||
| test-destination-os: "latest" | ||
| platform: "macOS" | ||
|
|
||
| - name: macOS 26 | ||
| macos_version: macos-26 | ||
| xcode: "26.1.1" | ||
| test-destination-os: "26.1" | ||
| platform: "macOS" | ||
|
|
||
| unit-tests-required-check: | ||
| needs: | ||
| [ | ||
| ready-to-merge-gate, | ||
| files-changed, | ||
| spm-integrations-tests, | ||
| test-integration, | ||
| ] | ||
cursor[bot] marked this conversation as resolved.
Show resolved
Hide resolved
|
||
| name: Swift Log Tests - Required Check | ||
| # This is necessary since a failed/skipped dependent job would cause this job to be skipped | ||
| if: always() | ||
| runs-on: ubuntu-latest | ||
| steps: | ||
| # If any jobs we depend on fails gets cancelled or times out, this job will fail. | ||
| # Skipped jobs are not considered failures. | ||
| - name: Check for failures | ||
| if: contains(needs.*.result, 'failure') || contains(needs.*.result, 'cancelled') | ||
| run: | | ||
| echo "One of the unit test jobs has failed." && exit 1 | ||
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Bug: Missing secrets inheritance for codecov token access
The
test-integrationjob is missingsecrets: inheritwhen calling theunit-test-common.ymlreusable workflow. Other callers liketest.ymlandfast-pr-checks.ymlinclude this directive. Without it,unit-test-common.ymlcannot accesssecrets.CODECOV_TOKEN, causing the codecov upload steps to fail or behave unexpectedly.