Skip to content

Commit

Permalink
chore: test-isolation remove old tests (GoogleCloudPlatform#3998)
Browse files Browse the repository at this point in the history
Removing old tests that are already consistently passing in our prod workflow with the new "custard" tooling.

Also made some small cleanups:

* Remove "Node.js" prefix for jobs since it's redundant
* Split region tags job from lint (like the previous ci.yaml)
* Documented some other tests that were not previously tested
  • Loading branch information
davidcavazos authored Feb 28, 2025
1 parent 5a1e566 commit 5989107
Show file tree
Hide file tree
Showing 93 changed files with 26 additions and 4,851 deletions.
4 changes: 4 additions & 0 deletions .github/config/nodejs-dev.jsonc
Original file line number Diff line number Diff line change
Expand Up @@ -66,10 +66,12 @@
"functions/firebase", // parent directory
"functions/helloworld", // parent directory
"functions/http", // parent directory
"functions/http/uploadFile", // no tests exist
"functions/log", // parent directory
"functions/pubsub", // parent directory
"memorystore/redis", // parent directory
"recaptcha_enterprise/demosite/app", // no tests exist

// These tests are already passing in prod, so skip them in dev.
"appengine/building-an-app/build",
"appengine/building-an-app/update",
Expand All @@ -80,6 +82,7 @@
"appengine/hello-world/standard",
"appengine/memcached",
"appengine/metadata/flexible_nodejs16_and_earlier",
"appengine/metadata/standard",
"appengine/pubsub",
"appengine/static-files",
"appengine/storage/flexible",
Expand Down Expand Up @@ -196,6 +199,7 @@
"security-center/snippets",
"service-directory/snippets",
"storage-control",
"talent",
"texttospeech",
"tpu",
"workflows/invoke-private-endpoint"
Expand Down
15 changes: 7 additions & 8 deletions .github/config/nodejs-prod.jsonc
Original file line number Diff line number Diff line change
Expand Up @@ -64,6 +64,7 @@
"functions/firebase", // parent directory
"functions/helloworld", // parent directory
"functions/http", // parent directory
"functions/http/uploadFile", // no tests exist
"functions/log", // parent directory
"functions/pubsub", // parent directory
"memorystore/redis", // parent directory
Expand All @@ -86,21 +87,19 @@
"dlp", // [ERR_REQUIRE_ESM]: require() of ES Module
"document-ai", // [ERR_REQUIRE_ESM]: require() of ES Module
"eventarc/audit-storage", // (untested) Environment Variable 'SERVICE_NAME' not found
"functions/billing", // Error: Request failed with status code 500
"functions/http/uploadFile", // npm error Missing script: "test"
"functions/imagemagick", // Error: A bucket name is needed to use Cloud Storage
"functions/ocr/app", // Error: Bucket not provided. Make sure you have a "bucket" property in your request
"functions/billing", // (untested) Error: Request failed with status code 500
"functions/imagemagick", // (untested) Error: A bucket name is needed to use Cloud Storage
"functions/ocr/app", // (untested) Error: Bucket not provided. Make sure you have a "bucket" property in your request
"functions/slack", // TypeError [ERR_INVALID_ARG_TYPE]: The "key" argument must be of type ... Received undefined
"functions/v2/imagemagick", // Error: A bucket name is needed to use Cloud Storage.
"functions/v2/imagemagick", // (untested) Error: A bucket name is needed to use Cloud Storage.
"healthcare/fhir", // Error: Cannot find module 'whatwg-url'
"iam/deny", // PERMISSION_DENIED: Permission iam.googleapis.com/denypolicies.create denied on resource cloudresourcemanager.googleapis.com/projects/long-door-651
"recaptcha_enterprise/snippets", // Cannot use import statement outside a module
"run/idp-sql", // Error: Invalid contents in the credentials file
"run/markdown-preview/editor", // Error: could not create an identity token: Cannot fetch ID token in this environment, use GCE or set the GOOGLE_APPLICATION_CREDENTIALS environment variable to a service account credentials JSON file
"run/idp-sql", // (untested) Error: Invalid contents in the credentials file
"run/markdown-preview/editor", // (untested) Error: could not create an identity token: Cannot fetch ID token in this environment, use GCE or set the GOOGLE_APPLICATION_CREDENTIALS environment variable to a service account credentials JSON file
"scheduler", // SyntaxError: Cannot use import statement outside a module
"speech", // AssertionError: expected 'Transcription: Okay, I\'m here.\n Hi…' to match /Terrific. It's on the way./
"storagetransfer", // CredentialsError: Missing credentials in config, if using AWS_CONFIG_FILE, set AWS_SDK_LOAD_CONFIG=1
"talent", // AssertionError: expected '' to match /Job summary/
"translate", // AssertionError: expected 'Languages:\n{ code: \'ab\', name: \'A…' to match /{ code: 'af', name: 'afrikáans' }/
"video-intelligence", // PERMISSION_DENIED: The caller does not have permission
"vision", // REDIS: Error: connect ECONNREFUSED 127.0.0.1:6379
Expand Down
52 changes: 0 additions & 52 deletions .github/workflows/appengine-building-an-app-build.yaml

This file was deleted.

52 changes: 0 additions & 52 deletions .github/workflows/appengine-building-an-app-update.yaml

This file was deleted.

52 changes: 0 additions & 52 deletions .github/workflows/appengine-datastore.yaml

This file was deleted.

52 changes: 0 additions & 52 deletions .github/workflows/appengine-endpoints.yaml

This file was deleted.

52 changes: 0 additions & 52 deletions .github/workflows/appengine-hello-world-flexible.yaml

This file was deleted.

52 changes: 0 additions & 52 deletions .github/workflows/appengine-hello-world-standard.yaml

This file was deleted.

Loading

0 comments on commit 5989107

Please sign in to comment.