Skip to content

[codex] Expand integration acceptance coverage#672

Merged
AdamEssenmacher merged 8 commits into
developmentfrom
codex/integration-coverage-expansion
May 23, 2026
Merged

[codex] Expand integration acceptance coverage#672
AdamEssenmacher merged 8 commits into
developmentfrom
codex/integration-coverage-expansion

Conversation

@AdamEssenmacher

Copy link
Copy Markdown
Collaborator

Summary

  • Split large Auth, Firestore, Functions, Storage, Remote Config, and Performance fixtures into focused partial fixtures.
  • Add App Check, Bundled, Cloud Messaging, Firestore, Functions, Storage, Installations, Crashlytics, and Performance acceptance coverage.
  • Add coverage metadata auditing and Cloud Functions helpers for custom token, auth context, structured error, and regional callable tests.
  • Remove obsolete fixture files replaced by focused split files.

Validation

  • git diff --check origin/development..HEAD
  • scripts/check-integration-coverage.rb
  • npm --prefix tests/cloud-functions/functions run build
  • dotnet format Plugin.Firebase.sln --verify-no-changes --include $(git diff --name-only origin/development..HEAD | tr "\n" " ")
  • dotnet build tests/Plugin.Firebase.IntegrationTests/Plugin.Firebase.IntegrationTests.csproj -f net9.0-android -c Debug --no-restore
  • dotnet build tests/Plugin.Firebase.IntegrationTests/Plugin.Firebase.IntegrationTests.csproj -f net9.0-ios -c Debug --no-restore

Note: full solution format verification still reports pre-existing formatting issues outside this stack, so only the stack diff was format-verified.

@AdamEssenmacher AdamEssenmacher force-pushed the codex/integration-harness-ci branch from 274cc72 to d15bc69 Compare May 4, 2026 04:41
@AdamEssenmacher AdamEssenmacher force-pushed the codex/integration-coverage-expansion branch from cc32375 to 231d2a1 Compare May 4, 2026 04:43
@AdamEssenmacher AdamEssenmacher force-pushed the codex/integration-harness-ci branch from d15bc69 to 95802a2 Compare May 4, 2026 04:51
@AdamEssenmacher AdamEssenmacher force-pushed the codex/integration-coverage-expansion branch from 231d2a1 to 8ed2256 Compare May 4, 2026 04:51
@AdamEssenmacher AdamEssenmacher force-pushed the codex/integration-harness-ci branch from 95802a2 to 7cfc4f7 Compare May 4, 2026 05:03
@AdamEssenmacher AdamEssenmacher force-pushed the codex/integration-coverage-expansion branch from 8ed2256 to bb6bd90 Compare May 4, 2026 05:03
@AdamEssenmacher AdamEssenmacher force-pushed the codex/integration-coverage-expansion branch from bb6bd90 to fdfba5e Compare May 4, 2026 05:28
@AdamEssenmacher AdamEssenmacher force-pushed the codex/integration-harness-ci branch from 7cfc4f7 to 41c078e Compare May 5, 2026 23:07
Base automatically changed from codex/integration-harness-ci to development May 21, 2026 20:43
…ion-coverage-expansion

# Conflicts:
#	.github/workflows/integration-emulators.yml
#	src/Analytics/Platforms/Android/FirebaseAnalyticsImplementation.cs
#	src/Analytics/Platforms/iOS/FirebaseAnalyticsImplementation.cs
#	src/Analytics/Shared/IFirebaseAnalytics.cs
#	tests/Plugin.Firebase.IntegrationTests/Analytics/AnalyticsFixture.cs
#	tests/Plugin.Firebase.IntegrationTests/RemoteConfig/RemoteConfigFixture.cs
@AdamEssenmacher AdamEssenmacher marked this pull request as ready for review May 21, 2026 21:06
@AdamEssenmacher

Copy link
Copy Markdown
Collaborator Author

@Kapusch this is the big one I mentioned. It's just a ton of device test expansions and refactoring that should let us move forward faster with a much bigger safety net.

@Kapusch Kapusch left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

I left some comments, but overall good to me

Comment thread tests/Plugin.Firebase.IntegrationTests/Firestore/SimpleItem.cs
Comment thread tests/Plugin.Firebase.IntegrationTests/Firestore/SimpleItem.cs
Comment thread tests/Plugin.Firebase.IntegrationTests/README.md
@AdamEssenmacher AdamEssenmacher merged commit c8a103a into development May 23, 2026
3 checks passed
@AdamEssenmacher AdamEssenmacher deleted the codex/integration-coverage-expansion branch May 23, 2026 00:14
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