Skip collection API checks in test (all) job

[kenyonj]

Summary

• Skip the expensive collection rename/existence API checks (~300 GraphQL calls) when running the test (all) job
• The all job triggers for non-collection file changes (Gemfile, Rakefile, etc.) where these checks are redundant
• Collection renames are already handled by the hourly collections-renames workflow and the test (collections) job

Changes

• test/collections_test.rb: Add skip guard when SKIP_COLLECTION_API_CHECKS env var is set
• .github/workflows/test.yml: Set SKIP_COLLECTION_API_CHECKS=1 for the all matrix type

Impact

Reduces test (all) runtime from ~16 minutes to ~2 minutes, unblocking dependabot PRs and infrastructure changes.

[Copilot]

Pull request overview

This PR optimizes CI test runtime by skipping expensive GitHub API checks (~300 GraphQL calls) when running the test (all) job. The all job handles non-collection file changes (Gemfile, Rakefile, infrastructure files) where collection rename/existence checks are redundant. These checks are already handled by the hourly collections-renames workflow and the test (collections) job.

Changes:

• Add conditional skip logic to collection API checks test based on SKIP_COLLECTION_API_CHECKS environment variable
• Configure workflow to set SKIP_COLLECTION_API_CHECKS=1 for the all matrix test type

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File	Description
test/collections_test.rb	Add skip guard at the beginning of the rename/removal detection test when SKIP_COLLECTION_API_CHECKS is set
.github/workflows/test.yml	Set SKIP_COLLECTION_API_CHECKS environment variable to '1' for the 'all' test matrix type

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[ahpook]

For the record, there's a problem: the tests have changed between the time the PR was opened and now, as the collections test was split out into 4 parallel jobs with different names. But the PRs which were opened when the collections test was a named item are still "expecting" that test to run and pass, and won't clear from the merge queue until this no-longer-existent test runs. So I'm bypassing the merge queue for all of them.