install gcloud and run tests that need gsutil (#175)

jakemac53 · web-flow · commit e82d503b7d33 · 2025-06-11T10:44:58.000-07:00
Part of #173
diff --git a/.github/workflows/baseline.yaml b/.github/workflows/baseline.yaml
@@ -34,12 +34,13 @@ jobs:
       - uses: dart-lang/setup-dart@e51d8e571e22473a2ddebf0ef8a2123f0ab2c02c
         with:
           sdk: ${{ matrix.sdk }}
+      - name: 'Set up Cloud SDK'
+        uses: 'google-github-actions/setup-gcloud@77e7a554d41e2ee56fc945c52dfd3f33d12def9a'
 
       - run: dart pub get
 
       - run: dart analyze --fatal-infos
 
       - run: dart format --output=none --set-exit-if-changed .
 
-      # TODO: Install gsutil and un-skip tests that require it
-      - run: dart test -x requires_gsutil
+      - run: dart test
diff --git a/baseline/lib/baseline.dart b/baseline/lib/baseline.dart
@@ -108,11 +108,11 @@ Future<void> baselineBuilder(
 }
 
 Future<String> read(String url) {
-  return run('gsutil.py', ['cp', url, '-']);
+  return run('gsutil', ['cp', url, '-']);
 }
 
 Future<String> write(String url, String stdin, bool dryRun) {
-  return run('gsutil.py', ['cp', '-', url], stdin: stdin, dryRun: dryRun);
+  return run('gsutil', ['cp', '-', url], stdin: stdin, dryRun: dryRun);
 }
 
 Future<String> run(String command, List<String> arguments,

Original file line number	Diff line number	Diff line change
`@@ -108,11 +108,11 @@ Future<void> baselineBuilder(`
`108`	`108`	`}`
`109`	`109`
`110`	`110`	`Future<String> read(String url) {`
`111`		`- return run('gsutil.py', ['cp', url, '-']);`
	`111`	`+ return run('gsutil', ['cp', url, '-']);`
`112`	`112`	`}`
`113`	`113`
`114`	`114`	`Future<String> write(String url, String stdin, bool dryRun) {`
`115`		`- return run('gsutil.py', ['cp', '-', url], stdin: stdin, dryRun: dryRun);`
	`115`	`+ return run('gsutil', ['cp', '-', url], stdin: stdin, dryRun: dryRun);`
`116`	`116`	`}`
`117`	`117`
`118`	`118`	`Future<String> run(String command, List<String> arguments,`