test: cover init modules and pipeline - #401
Conversation
🦋 Changeset detectedLatest commit: 4862d57 The changes in this PR will be included in the next version bump. This PR includes changesets to release 1 package
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
👷 Deploy request for compose-diamonds pending review.Visit the deploys page to approve it
|
There was a problem hiding this comment.
Hey @0x76agabond, i really like how you reorganized the adapters. It`s way cleaner.
For the test, i didn't review all of them but i trust you on this. You can merge when you feel it's ready.
On a sidenote, if we want a quick insight on PR, we could do a coverage comment action similar to our Solidity one but for the CLI. What do you think?
Yeah, I like it. I think the coverage comment would be useful. |
Great idea! |
Summary
Add focused tests for the modules used by
compose initand an integration test for the complete init pipeline.Changes Made
InitPipelinetest harness using temporary directories and a fixture framework adapter.130CLI tests, TypeScript build, and lint successfully.Checklist
Before submitting this PR, please ensure:
forge fmt- Not applicable; no Solidity files were changed.130CLI tests pass.npm run cli@test,npm run cli@build, andnpm run cli@lintpass.Make sure to follow the [contributing](https://compose.diamonds/docs/contribution/how-to-contribute) guidelines.
Additional Notes
Framework installation and real project compilation remain covered by the existing Foundry and Hardhat smoke-test steps in the CLI workflow. The Vitest pipeline tests use a fixture adapter so the regular test suite remains fast and deterministic.