Skip to content

Running SalesforceSDKCore unit tests when no libs changed#3992

Merged
wmathurin merged 5 commits intoforcedotcom:devfrom
wmathurin:dev
Feb 23, 2026
Merged

Running SalesforceSDKCore unit tests when no libs changed#3992
wmathurin merged 5 commits intoforcedotcom:devfrom
wmathurin:dev

Conversation

@wmathurin
Copy link
Contributor

@wmathurin wmathurin commented Feb 20, 2026

Previously we would get
Error when evaluating 'strategy' for job 'ios-pr'. .github/workflows/pr.yaml (Line: 104, Col: 14): Matrix vector 'lib' does not contain any values
If no libs were modified e.g. when changing just UI tests or github workflows

Previously we would get
Error when evaluating 'strategy' for job 'ios-pr'. .github/workflows/pr.yaml (Line: 104, Col: 14): Matrix vector 'lib' does not contain any values
If no libs were modified (e.g. when changing just UI tests)
@wmathurin wmathurin marked this pull request as ready for review February 20, 2026 23:39
@github-actions
Copy link

github-actions bot commented Feb 20, 2026

TestsPassed ☑️SkippedFailed ❌️
SalesforceSDKCore iOS ^18 Test Results591 ran590 ✅1 ❌
TestResult
SalesforceSDKCore iOS ^18 Test Results
NativeLoginManagerTests.testPasswordValidation()❌ failure

@github-actions
Copy link

github-actions bot commented Feb 20, 2026

TestsPassed ☑️SkippedFailed ❌️
SalesforceSDKCore iOS ^26 Test Results591 ran590 ✅1 ❌
TestResult
SalesforceSDKCore iOS ^26 Test Results
DomainDiscoveryCoordinatorTests.testMissingMyDomain()❌ failure

@codecov
Copy link

codecov bot commented Feb 21, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 68.36%. Comparing base (f6645b7) to head (7600035).
⚠️ Report is 6 commits behind head on dev.

Additional details and impacted files
@@            Coverage Diff             @@
##              dev    #3992      +/-   ##
==========================================
- Coverage   69.07%   68.36%   -0.71%     
==========================================
  Files         253      253              
  Lines       22260    22260              
==========================================
- Hits        15375    15217     -158     
- Misses       6885     7043     +158     
Components Coverage Δ
Analytics 70.78% <ø> (ø)
Common 69.76% <ø> (ø)
Core 61.88% <100.00%> (-1.12%) ⬇️
SmartStore 74.59% <ø> (ø)
MobileSync 87.41% <ø> (ø)
Files with missing lines Coverage Δ
...ore/Classes/OAuth/DomainDiscoveryCoordinator.swift 89.70% <100.00%> (ø)

... and 14 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@wmathurin
Copy link
Contributor Author

More PRs to follow.

@wmathurin wmathurin merged commit 7321a09 into forcedotcom:dev Feb 23, 2026
22 of 29 checks passed
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.

3 participants