Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

RUM-5650 chore: Refine Smoke Tests and add Swift 5.9 coverage #2053

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

ncreated
Copy link
Collaborator

What and why?

📦 This PR reorganizes and improves the coverage of smoke tests for our iOS SDK. It now includes coverage for all supported Swift versions while maintaining compatibility with various dependency managers.

How?

The smoke-test stage has been streamlined to 4 key jobs:

Screenshot 2024-09-13 at 11 50 12
  • Smoke Test (iOS) and Smoke Test (tvOS): These jobs test SDK integration through CocoaPods, Carthage, Xcode's SPM, and XCFrameworks. They run on a macos:sonoma runner with Xcode 15.4.

  • SPM Build (Swift 5.9): This job builds Package.swift for each supported platform using the macos:ventura runner with Xcode 15.2.

  • SPM Build (Swift 5.10): Similar to the above, but for Swift 5.10 using the macos:sonoma runner with Xcode 15.4.

The two "SPM Build" jobs now fully mirror SDK builds in Swift Package Index.

Review checklist

  • Feature or bugfix MUST have appropriate tests (unit, integration)
  • Make sure each commit and the PR mention the Issue number or JIRA reference
  • Add CHANGELOG entry for user facing changes

@ncreated ncreated requested review from a team as code owners September 13, 2024 09:56
@ncreated ncreated self-assigned this Sep 13, 2024
@ncreated ncreated changed the title RUM-5650 chore: Reorganize Smoke Tests; Add "SPM build" jobs for Swift 5.9 and 5.10 RUM-5650 chore: Refine Smoke Tests add Swift 5.9 coverage Sep 13, 2024
@ncreated ncreated changed the title RUM-5650 chore: Refine Smoke Tests add Swift 5.9 coverage RUM-5650 chore: Refine Smoke Tests and add Swift 5.9 coverage Sep 13, 2024
Comment on lines -265 to +235
- ./tools/runner-setup.sh --xcode "$XCODE" --watchOS --os "$OS" # temporary, waiting for AMI
- ./tools/runner-setup.sh --xcode "$XCODE" --iOS --tvOS --visionOS --watchOS
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

💡 Current macos:ventura AMI includes all these platforms, but there is no harm in listing them here - the script will just skip what is already installed.

@ncreated ncreated force-pushed the ncreated/RUM-5650/enhance-smoke-testing-on-ci branch from 8815206 to 511917a Compare September 13, 2024 12:29
@datadog-datadog-prod-us1
Copy link

Datadog Report

Branch report: ncreated/RUM-5650/enhance-smoke-testing-on-ci
Commit report: 511917a
Test service: dd-sdk-ios

✅ 0 Failed, 3468 Passed, 0 Skipped, 2m 17.75s Total Time
🔻 Test Sessions change in coverage: 4 decreased, 4 increased, 6 no change

🔻 Code Coverage Decreases vs Default Branch (4)

  • test DatadogInternalTests iOS 79.44% (-0.06%) - Details
  • test DatadogTraceTests iOS 54.25% (-0.02%) - Details
  • test DatadogCrashReportingTests iOS 26.77% (-0.02%) - Details
  • test DatadogSessionReplayTests iOS 30.63% (-0.01%) - Details

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