Skip to content

test: cover init modules and pipeline - #401

Open
0x76agabond wants to merge 3 commits into
Perfect-Abstractions:mainfrom
0x76agabond:test/compose-init
Open

0x76agabond wants to merge 3 commits into
Perfect-Abstractions:mainfrom
0x76agabond:test/compose-init

Conversation

@0x76agabond

Copy link
Copy Markdown
Collaborator

Summary

Add focused tests for the modules used by compose init and an integration test for the complete init pipeline.

Changes Made

  • Added focused tests for config, init, preflight, project directory, scaffolding, Diamond generation, deploy generation, and test generation modules.
  • Added an InitPipeline test harness using temporary directories and a fixture framework adapter.
  • Added pipeline coverage for bare Foundry, bare Hardhat, and minimal ERC-20 projects.
  • Updated the CLI workflow to run Vitest in CI.
  • Verified 130 CLI tests, TypeScript build, and lint successfully.

Checklist

Before submitting this PR, please ensure:

  • Code follows the Solidity feature ban - Not applicable; no Solidity production code was changed.
  • Code follows Design Principles - Tests follow the existing module and adapter boundaries.
  • Code matches the codebase style - Test files follow the existing CLI test conventions.
  • Code is formatted with forge fmt - Not applicable; no Solidity files were changed.
  • Existing tests pass - All 130 CLI tests pass.
  • New tests are optional - New tests are included.
  • All tests pass - npm run cli@test, npm run cli@build, and npm run cli@lint pass.
  • Documentation updated - Not applicable; no user-facing behavior changed.
  • Changesets - No changeset is required because this PR only adds tests and CI coverage.

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.

@changeset-bot

changeset-bot Bot commented Sep 17, 2026

Copy link
Copy Markdown

⚠️ No Changeset found

Latest commit: d0b6334

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@netlify

netlify Bot commented Sep 17, 2026

Copy link
Copy Markdown

👷 Deploy request for compose-diamonds pending review.

Visit the deploys page to approve it

Name Link
🔨 Latest commit d0b6334

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.

1 participant