fix(ci): remove --no-pub from integration tests; use dart analyze instead of flutter analyze

Integration tests build native Linux app via CMake which requires pub get side effects
(plugin registrant file generation) — --no-pub broke the CMake step.

Switch flutter analyze to dart analyze --fatal-infos to eliminate the flutter wrapper's
non-deterministic state writes to /root/.dartServer/, which were preventing action cache
hits on the analyze step.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
This commit is contained in:
Thomas SharedInbox
2026-05-19 23:57:25 +02:00
co-authored by Claude Sonnet 4.6
parent 7fce683375
commit e0a99d7d63
+2 -2
View File
@@ -377,7 +377,7 @@ func (m *Ci) TestBackend(ctx context.Context) (string, error) {
func (m *Ci) TestIntegration(ctx context.Context) (string, error) {
return m.WithStalwart(m.setup(m.integrationSrc())).
WithEnvVariable("LIBGL_ALWAYS_SOFTWARE", "1").
WithExec([]string{"xvfb-run", "-s", "-screen 0 1280x720x24", "flutter", "test", "--no-pub", "integration_test/", "-d", "linux"}).
WithExec([]string{"xvfb-run", "-s", "-screen 0 1280x720x24", "flutter", "test", "integration_test/", "-d", "linux"}).
Stdout(ctx)
}
@@ -403,7 +403,7 @@ func (m *Ci) Check(ctx context.Context) (string, error) {
return "Format check failed", err
}
analyze, err := checkSetup.WithExec([]string{"flutter", "analyze", "--no-pub"}).Stdout(ctx)
analyze, err := checkSetup.WithExec([]string{"dart", "analyze", "--fatal-infos"}).Stdout(ctx)
if err != nil {
return analyze, err
}