Skip to content

ci(flutter): skip integration tests on forks without auth token#3860

Draft
buenaflor wants to merge 2 commits into
mainfrom
fix/flutter-integration-test-fork-skip
Draft

ci(flutter): skip integration tests on forks without auth token#3860
buenaflor wants to merge 2 commits into
mainfrom
fix/flutter-integration-test-fork-skip

Conversation

@buenaflor

@buenaflor buenaflor commented Jul 6, 2026

Copy link
Copy Markdown
Contributor

Problem

In flutter_test.yml, the SENTRY_AUTH_TOKEN_E2E secret is unavailable on fork PRs (GitHub doesn't pass secrets to forks). Without a guard, the e2e/captureException test runs with an empty token, calls sentry.io/api/... for verification, and always fails.

All other integration tests use a fakeDsn and don't need the token — they should still run on forks.

Change

Added skip: authToken.isEmpty to the captureException testWidgets call inside the group('e2e', ...) block in integration_test.dart.

authToken is already a const String.fromEnvironment('SENTRY_AUTH_TOKEN_E2E'), so this is a compile-time skip with no runtime overhead. Everything else in the integration test suite continues to run on forks as-is.

Co-Authored-By: sentry-junior[bot] <264270552+sentry-junior[bot]@users.noreply.github.com>
@github-actions

github-actions Bot commented Jul 6, 2026

Copy link
Copy Markdown
Contributor

Semver Impact of This PR

None (no version bump detected)

📋 Changelog Preview

This is how your changes will appear in the changelog.
Entries from this PR are highlighted with a left border (blockquote style).


Features

  • Add feature flags to hub span by denrase in #3806

Internal Changes

  • (flutter) Skip integration tests on forks without auth token by buenaflor in #3860

🤖 This preview updates automatically when you update the PR.

Co-Authored-By: sentry-junior[bot] <264270552+sentry-junior[bot]@users.noreply.github.com>
@codecov

codecov Bot commented Jul 6, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 91.50%. Comparing base (c652e19) to head (74298fb).
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #3860      +/-   ##
==========================================
+ Coverage   87.35%   91.50%   +4.15%     
==========================================
  Files         338      105     -233     
  Lines       12283     3803    -8480     
==========================================
- Hits        10730     3480    -7250     
+ Misses       1553      323    -1230     
Flag Coverage Δ
sentry ?
sentry_dio ?
sentry_drift ?
sentry_file ?
sentry_firebase_remote_config 100.00% <ø> (ø)
sentry_flutter 91.10% <ø> (ø)
sentry_grpc ?
sentry_hive ?
sentry_isar ?
sentry_link ?
sentry_logging ?
sentry_sqflite ?
sentry_supabase 97.27% <ø> (ø)

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.

@github-actions

github-actions Bot commented Jul 6, 2026

Copy link
Copy Markdown
Contributor

iOS Performance metrics 🚀

  Plain With Sentry Diff
Startup time 1253.14 ms 1254.58 ms 1.45 ms
Size 5.83 MiB 6.28 MiB 460.41 KiB

Baseline results on branch: main

Startup times

Revision Plain With Sentry Diff
0f0a415 1228.28 ms 1233.51 ms 5.23 ms
5367726 1211.00 ms 1216.73 ms 5.73 ms
f80073a 1249.75 ms 1243.52 ms -6.23 ms
d15266b 1247.42 ms 1251.29 ms 3.87 ms
2f63d89 1251.67 ms 1263.94 ms 12.27 ms
bfabaf2 1251.72 ms 1253.38 ms 1.67 ms
8af916c 1246.66 ms 1240.79 ms -5.87 ms
9441966 1239.78 ms 1254.00 ms 14.22 ms
54acf91 1257.65 ms 1277.96 ms 20.31 ms
396cb30 1265.06 ms 1265.73 ms 0.67 ms

App size

Revision Plain With Sentry Diff
0f0a415 5.83 MiB 6.28 MiB 458.16 KiB
5367726 5.73 MiB 6.17 MiB 455.87 KiB
f80073a 5.73 MiB 6.18 MiB 464.26 KiB
d15266b 5.83 MiB 6.28 MiB 462.36 KiB
2f63d89 5.65 MiB 6.09 MiB 446.25 KiB
bfabaf2 5.53 MiB 6.01 MiB 487.95 KiB
8af916c 5.73 MiB 6.18 MiB 463.84 KiB
9441966 5.83 MiB 6.28 MiB 459.62 KiB
54acf91 20.70 MiB 22.46 MiB 1.75 MiB
396cb30 5.53 MiB 6.01 MiB 488.17 KiB

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