Skip to content

Add seed tests and CI workflow#68

Open
Jim8y wants to merge 1 commit into
neoorder:masterfrom
Jim8y:codex/ci-seed-tests
Open

Add seed tests and CI workflow#68
Jim8y wants to merge 1 commit into
neoorder:masterfrom
Jim8y:codex/ci-seed-tests

Conversation

@Jim8y

@Jim8y Jim8y commented Jun 22, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Add a focused xUnit seed test project for low-risk repository checks.
  • Add a GitHub Actions workflow that runs the seed tests on pull requests and master pushes.
  • Cover localization key parity and bundled raw JSON resource sanity.

Validation

  • dotnet test OneGateApp.Tests/OneGateApp.Tests.csproj --configuration Release --verbosity normal
  • dotnet build OneGateApp/OneGateApp.csproj -f net10.0-android -p:AndroidSdkDirectory=/opt/homebrew/share/android-commandlinetools -p:EmbedAssembliesIntoApk=true
  • dotnet build OneGateApp/OneGateApp.csproj -f net10.0-ios -p:RuntimeIdentifier=iossimulator-arm64 -p:CodesignKey=- -p:CodesignProvision=
  • iOS and Android simulator screenshots will be posted in a PR comment before marking ready for review.

Notes

  • Scope is intentionally limited to CI and seed tests.
  • No product UI, wallet, signing, or runtime behavior changes.

@Jim8y

Jim8y commented Jun 22, 2026

Copy link
Copy Markdown
Contributor Author

Simulator validation completed for this PR.

Validation run:

  • Seed tests passed: dotnet test OneGateApp.Tests/OneGateApp.Tests.csproj --configuration Release --verbosity normal (7 passed, 0 failed).
  • iOS build passed: dotnet build OneGateApp/OneGateApp.csproj -f net10.0-ios -p:RuntimeIdentifier=iossimulator-arm64 -p:CodesignKey=- -p:CodesignProvision= (3 existing NU1903 SQLite warnings, 0 errors).
  • Android build passed: RuntimeIdentifiers=android-arm64;android-x64 dotnet build OneGateApp/OneGateApp.csproj -f net10.0-android -p:AndroidSdkDirectory=/opt/homebrew/share/android-commandlinetools -p:EmbedAssembliesIntoApk=true (9 existing NU1903 SQLite warnings, 0 errors).
  • iOS Simulator: installed and launched on iPhone 17 Pro / iOS 26.5, launch returned PID 2996.
  • Android Emulator: installed and launched on emulator-5554, app PID 10797, crash buffer empty.

Screenshots:

iOS simulator

Android emulator

@Jim8y Jim8y marked this pull request as ready for review June 22, 2026 02:44
Copilot AI review requested due to automatic review settings June 22, 2026 02:44

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

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.Tests xUnit project targeting net10.0 with 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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants