Skip to content

SFRestAPI stress test#4003

Draft
bbirman wants to merge 1 commit intoforcedotcom:devfrom
bbirman:rest-stress-test
Draft

SFRestAPI stress test#4003
bbirman wants to merge 1 commit intoforcedotcom:devfrom
bbirman:rest-stress-test

Conversation

@bbirman
Copy link
Member

@bbirman bbirman commented Mar 6, 2026

No description provided.

@github-actions
Copy link

github-actions bot commented Mar 6, 2026

TestsPassed ☑️SkippedFailed ❌️
SalesforceSDKCore iOS ^18 Test Results595 ran591 ✅4 ❌
TestResult
SalesforceSDKCore iOS ^18 Test Results
DomainDiscoveryCoordinatorTests.testSpecialCharactersInLoginHint()❌ failure
RestClientTests.testSendURLRequestHTTPError() (failure 1/2)❌ failure
RestClientTests.testSendURLRequestHTTPError() (failure 2/2)❌ failure
SFRestAPITokenRefreshStressTests.testHundredsOfConcurrentRequestsWithOccasional401() (failure 1/2)❌ failure
SFRestAPITokenRefreshStressTests.testHundredsOfConcurrentRequestsWithOccasional401() (failure 2/2)❌ failure
SFSDKAuthConfigUtilTests.testGetAuthConfig❌ failure

@github-actions
Copy link

github-actions bot commented Mar 7, 2026

TestsPassed ☑️SkippedFailed ❌️
SalesforceSDKCore iOS ^26 Test Results595 ran591 ✅4 ❌
TestResult
SalesforceSDKCore iOS ^26 Test Results
DomainDiscoveryCoordinatorTests.testNonCallbackURL()❌ failure
RestClientTests.testSendURLRequestHTTPError() (failure 1/2)❌ failure
RestClientTests.testSendURLRequestHTTPError() (failure 2/2)❌ failure
SFRestAPITokenRefreshStressTests.testHundredsOfConcurrentRequestsWithOccasional401()❌ failure
SFSDKAuthConfigUtilTests.testBrowserBasedLoginEnabled❌ failure

@codecov
Copy link

codecov bot commented Mar 7, 2026

Codecov Report

❌ Patch coverage is 66.66667% with 2 lines in your changes missing coverage. Please review.
✅ Project coverage is 64.64%. Comparing base (39ff53d) to head (b0a3bcb).

Files with missing lines Patch % Lines
...Core/SalesforceSDKCore/Classes/RestAPI/SFNetwork.m 66.66% 2 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##              dev    #4003      +/-   ##
==========================================
- Coverage   68.44%   64.64%   -3.80%     
==========================================
  Files         253      253              
  Lines       22289    22295       +6     
==========================================
- Hits        15255    14413     -842     
- Misses       7034     7882     +848     
Components Coverage Δ
Analytics 70.78% <ø> (ø)
Common 69.76% <ø> (ø)
Core 56.07% <66.66%> (-5.95%) ⬇️
SmartStore 74.59% <ø> (ø)
MobileSync 87.41% <ø> (ø)
Files with missing lines Coverage Δ
...Core/SalesforceSDKCore/Classes/RestAPI/SFNetwork.m 92.39% <66.66%> (-1.80%) ⬇️

... and 31 files with indirect coverage changes

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

Copy link
Contributor

@wmathurin wmathurin left a comment

Choose a reason for hiding this comment

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

Are you able to repro the race condition from last week with the new test?

@bbirman
Copy link
Member Author

bbirman commented Mar 9, 2026

I think so 👍 it fails without #4001 and passes with it.

One issue I'd need to fix is that the Swift tests run in parallel and this one should be isolated so it doesn't interfere with other tests

@wmathurin
Copy link
Contributor

I think so 👍 it fails without #4001 and passes with it.

One issue I'd need to fix is that the Swift tests run in parallel and this one should be isolated so it doesn't interfere with other tests

Maybe with another test plan ??

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