Skip to content

Comments

ci: Only run test server on macOS#6350

Merged
philipphofmann merged 3 commits intomainfrom
test/test-server-only-macos
Oct 7, 2025
Merged

ci: Only run test server on macOS#6350
philipphofmann merged 3 commits intomainfrom
test/test-server-only-macos

Conversation

@philipphofmann
Copy link
Member

@philipphofmann philipphofmann commented Oct 6, 2025

The test server tests on iOS are very flaky. Let's only run these on macOS, as they didn't fail there in the last two weeks.
Perhaps that's because the simulator has difficulty communicating with a running test server in CI.

Screenshot 2025-10-06 at 07 10 25

#skip-changelog

Closes #6351

The test server tests on iOS are very flaky. Let's only run these on
macOS.
@codecov
Copy link

codecov bot commented Oct 6, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 86.834%. Comparing base (7aaa0b6) to head (6cfc890).
⚠️ Report is 2 commits behind head on main.
✅ All tests successful. No failed tests found.

Additional details and impacted files

Impacted file tree graph

@@              Coverage Diff              @@
##              main     #6350       +/-   ##
=============================================
+ Coverage   86.818%   86.834%   +0.016%     
=============================================
  Files          438       438               
  Lines        37311     37311               
  Branches     17428     17429        +1     
=============================================
+ Hits         32393     32399        +6     
+ Misses        4874      4867        -7     
- Partials        44        45        +1     

see 6 files with indirect coverage changes


Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 7aaa0b6...6cfc890. Read the comment docs.

Copy link
Contributor

@noahsmartin noahsmartin left a comment

Choose a reason for hiding this comment

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

LGTM with the suggestion that we should leave it commented out with a link to an issue tracking it

@philipphofmann philipphofmann enabled auto-merge (squash) October 6, 2025 14:56
@philipphofmann philipphofmann merged commit 1d4dad9 into main Oct 7, 2025
132 of 134 checks passed
@philipphofmann philipphofmann deleted the test/test-server-only-macos branch October 7, 2025 04:08
philipphofmann added a commit that referenced this pull request Oct 8, 2025
The test server tests on iOS are very flaky. Let's only run these on
macOS.
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.

ci: Only run test server on macOS

3 participants