Skip to content

Run the client crates' tests in an iOS simulator in CI - #108

Merged
LucaCappelletti94 merged 6 commits into
mainfrom
feat/ios-crate-tests
Sep 29, 2026
Merged

LucaCappelletti94 merged 6 commits into
mainfrom
feat/ios-crate-tests

Conversation

@LucaCappelletti94

Copy link
Copy Markdown
Owner

The crates the iOS demo links had their tests run only on Linux. A new macOS job builds the tests of connetto-core, connetto-auth-session, connetto-file-core, connetto-file-client, connetto-client with native-auth and connetto-dioxus for the iOS simulator and runs them inside one, as diesel-sqlite-session does.

About 55 of those tests start Postgres, OpenFGA and the mock identity provider in containers, and the runner has no Docker. Each fixture now copies a stopped template cluster and starts its own server on the copy, beside the OpenFGA and identity provider the workflow already starts for the simulator proof, so every test keeps a cluster to itself. The workflow makes the template with initdb before the simulator runs, because a program started inside the simulator cannot look up the host's user and initdb needs to, while the server itself starts there fine. With the variables unset, every fixture keeps its container.

Rehearsing the job found that the harness named two temporary directories made in the same millisecond alike, which let two fixtures share one. Names now carry a counter. On a Mac with Docker unreachable from the simulator, all six crates passed there, 449 tests with 3 ignored, without a single container. The job keeps no build cache, since main's caches have little room left, so this run also measures a cold build.

@coderabbitai

coderabbitai Bot commented Sep 29, 2026 •

Copy link
Copy Markdown

Important

  • 🔍 Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

⚙️ Run configuration

Configuration used: Repository: LucaCappelletti94/coderabbit/.coderabbit.yaml

Review profile: ASSERTIVE

Plan: Advanced

Run ID: 00a9d378-4eff-4051-a1b2-cd39f17ff416


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@codecov

codecov Bot commented Sep 29, 2026 •

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 86.30137% with 30 lines in your changes missing coverage. Please review.
✅ Project coverage is 84.64%. Comparing base (370be90) to head (dc5e395).
⚠️ Report is 2 commits behind head on main.

Files with missing lines Patch % Lines
crates/connetto-test-harness/src/native.rs 89.54% 7 Missing and 9 partials ⚠️
crates/connetto-test-harness/src/lib.rs 78.78% 11 Missing and 3 partials ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #108      +/-   ##
==========================================
+ Coverage   84.62%   84.64%   +0.01%     
==========================================
  Files         130      131       +1     
  Lines       31240    31447     +207     
  Branches    31240    31447     +207     
==========================================
+ Hits        26436    26617     +181     
- Misses       3283     3297      +14     
- Partials     1521     1533      +12     
Flag Coverage Δ
client 62.49% <20.76%> (-0.25%) ⬇️
rest 58.35% <85.84%> (+0.53%) ⬆️
server 52.82% <20.76%> (-0.19%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

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

@sonarqubecloud

Copy link
Copy Markdown

@LucaCappelletti94
LucaCappelletti94 merged commit 22a0b40 into main Sep 29, 2026
111 of 112 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.

1 participant