Add seed tests and CI workflow#68
Conversation
|
Simulator validation completed for this PR. Validation run:
Screenshots: |
There was a problem hiding this comment.
Pull request overview
Adds a small xUnit “seed” test suite and a GitHub Actions workflow to run it in CI, providing low-risk validation of repository resources (localized RESX parity and bundled raw JSON sanity) without affecting app runtime behavior.
Changes:
- Added
OneGateApp.TestsxUnit project targetingnet10.0with resource-focused tests. - Added tests to validate localized RESX key parity and validate/spot-check bundled Raw JSON resources.
- Added a GitHub Actions workflow to run the seed tests on PRs and pushes to
master.
Reviewed changes
Copilot reviewed 5 out of 5 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| OneGateApp.Tests/TestPaths.cs | Adds repository-root discovery helper used by tests to locate app resources reliably. |
| OneGateApp.Tests/ResourceParityTests.cs | Validates that each localized Strings.*.resx exposes the same set of keys as the neutral Strings.resx. |
| OneGateApp.Tests/RawResourceTests.cs | Validates raw JSON files parse correctly and checks a few required fields / expected protocol config values. |
| OneGateApp.Tests/OneGateApp.Tests.csproj | Introduces the seed test project and required test dependencies (xUnit, test SDK, coverlet collector). |
| .github/workflows/dotnet-tests.yml | Adds CI workflow to execute the seed tests on PRs and pushes to master. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
erikzhang
left a comment
There was a problem hiding this comment.
You need to add the test project to .slnx too.
|
Addressed Erik's review request:
Validation on branch head
Notes: no runtime code changed in this follow-up; existing simulator screenshots remain in the earlier validation comment. Builds still report existing SQLitePCLRaw NU1903 warnings and existing iOS duplicate asset warnings, with no errors. |
|
Addressed Erik's review feedback in Change:
Validation:
|
|
Current head already addresses the open project-reference feedback. Checked in this pass:
No additional commit was needed in this pass. |


Summary
Validation
Notes