Unit and template tests

[Arjun544]

Summary by CodeRabbit

• Chores
  • Adjusted test distribution logic in the continuous integration pipeline to optimize test execution efficiency.

[vercel]

The latest updates on your projects. Learn more about Vercel for GitHub.

Project	Deployment	Actions	Updated (UTC)
flutter-init	Ready	Preview, Comment	May 23, 2026 4:57am

[coderabbitai]

📝 Walkthrough

Walkthrough

The pull request modifies the test matrix preparation in the Tier 3 GitHub Actions workflow, changing the total chunk count calculation from ALL_COMBINATIONS to PRIMARY_COMBINATIONS. This adjustment directly affects the number of parallel test jobs generated in Layer 2 execution.

Changes

Test matrix configuration

Layer / File(s)	Summary
Matrix total calculation for Tier 3 .github/workflows/test-tier3.yml	The TOTAL environment variable now counts PRIMARY_COMBINATIONS.length instead of ALL_COMBINATIONS.length, affecting how many parallel matrix jobs are generated for Tier 3 testing.

🎯 1 (Trivial) | ⏱️ ~2 minutes

🐰 One line tweaked with care,
The matrix hops with lighter flair,
PRIMARY_COMBINATIONS takes the stage,
Tier 3 tests turn a cleaner page! ✨

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Title check	⚠️ Warning	The title 'Unit and template tests' is vague and does not reflect the actual change, which is modifying the TOTAL calculation in a GitHub workflow by using PRIMARY_COMBINATIONS.length instead of ALL_COMBINATIONS.length.	Update the title to accurately describe the specific change, such as 'Use PRIMARY_COMBINATIONS for matrix chunk calculation in test-tier3 workflow' or similar.

✅ Passed checks (4 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch unit-and-template-tests

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[coderabbitai]

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents

Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In @.github/workflows/test-tier3.yml:
- Line 67: The CI step computes TOTAL from PRIMARY_COMBINATIONS but the runner
and validator use ALL_COMBINATIONS, causing mismatched chunk sizing and skipped
tail combos; update the bun command that sets TOTAL to import and use
ALL_COMBINATIONS.length instead of PRIMARY_COMBINATIONS.length so the matrix
chunk sizing aligns with the actual execution/validation set (reference symbols:
TOTAL, PRIMARY_COMBINATIONS, ALL_COMBINATIONS, and the matrix chunking step).

🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

• Push a commit to this branch (recommended)
• Create a new PR with the fixes

---

ℹ️ Review info

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 674d42b5-7bb6-4132-93fd-4efc82ada629

📥 Commits

Reviewing files that changed from the base of the PR and between 41163a8 and 6774a45.

📒 Files selected for processing (1)

• .github/workflows/test-tier3.yml

[coderabbitai]

⚠️ Potential issue | 🟠 Major | ⚡ Quick win

Align matrix chunk sizing with the same combo set used at execution time.

Line 67 computes TOTAL from PRIMARY_COMBINATIONS, but Layer 2 executes and validates ranges against ALL_COMBINATIONS (tests/e2e/run-matrix.ts:38-44, tests/e2e/validate-combo.ts:34-42). This mismatch can silently skip tail combinations when lengths diverge, reducing Tier 3 coverage.

Suggested patch

-          TOTAL=$(bun -e "import { PRIMARY_COMBINATIONS } from './tests/utils/matrix.config'; console.log(PRIMARY_COMBINATIONS.length)")
+          TOTAL=$(bun -e "import { ALL_COMBINATIONS } from './tests/utils/matrix.config'; console.log(ALL_COMBINATIONS.length)")

📝 Committable suggestion

‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.

Suggested change

	TOTAL=$(bun -e "import { PRIMARY_COMBINATIONS } from './tests/utils/matrix.config'; console.log(PRIMARY_COMBINATIONS.length)")
	TOTAL=$(bun -e "import { ALL_COMBINATIONS } from './tests/utils/matrix.config'; console.log(ALL_COMBINATIONS.length)")

🤖 Prompt for AI Agents

Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In @.github/workflows/test-tier3.yml at line 67, The CI step computes TOTAL from
PRIMARY_COMBINATIONS but the runner and validator use ALL_COMBINATIONS, causing
mismatched chunk sizing and skipped tail combos; update the bun command that
sets TOTAL to import and use ALL_COMBINATIONS.length instead of
PRIMARY_COMBINATIONS.length so the matrix chunk sizing aligns with the actual
execution/validation set (reference symbols: TOTAL, PRIMARY_COMBINATIONS,
ALL_COMBINATIONS, and the matrix chunking step).