Skip to content

Refactor monolithic CLI index_test into specialized files#2106

Merged
rafal-hawrylak merged 1 commit intomainfrom
refactor_cli_tests
Mar 10, 2026
Merged

Refactor monolithic CLI index_test into specialized files#2106
rafal-hawrylak merged 1 commit intomainfrom
refactor_cli_tests

Conversation

@rafal-hawrylak
Copy link
Collaborator

Decomposes the large cli/index_test.ts into multiple files (help, init, project, compile, run, e2e) to improve maintainability and test isolation.

@rafal-hawrylak rafal-hawrylak force-pushed the refactor_cli_tests branch 3 times, most recently from 468286b to 6e2793d Compare March 9, 2026 10:44
Copy link
Collaborator

@apilaskowski apilaskowski left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

+1 from me on this change

Lets have second positive review before submitting.

@rafal-hawrylak rafal-hawrylak force-pushed the refactor_cli_tests branch 2 times, most recently from 9c086e3 to 39ae864 Compare March 9, 2026 11:05
@rafal-hawrylak rafal-hawrylak enabled auto-merge (squash) March 9, 2026 11:19
* refactor: decompose monolithic cli/index_test.ts into modular components

- Extract shared test utilities into cli/index_test_base.ts

- Create dedicated test files for help, init, project, compile, and run commands

- Update cli/BUILD to reflect the new modular test structure
@rafal-hawrylak rafal-hawrylak merged commit 270cafd into main Mar 10, 2026
6 checks passed
@rafal-hawrylak rafal-hawrylak deleted the refactor_cli_tests branch March 10, 2026 00:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants