Skip to content

fix(cli): scope test discovery to package - #331

Merged
akonwi merged 1 commit into
mainfrom
fix/test-discovery-package-boundaries
Jul 21, 2026
Merged

fix(cli): scope test discovery to package#331
akonwi merged 1 commit into
mainfrom
fix/test-discovery-package-boundaries

Conversation

@akonwi

@akonwi akonwi commented Jul 21, 2026

Copy link
Copy Markdown
Owner

Summary

  • stop ard test <directory> traversal at nested ard.toml package boundaries
  • keep the explicitly requested package root searchable
  • add a regression test covering a root test alongside a nested example project

Problem

Running ard test . in dram traversed into examples/hello, which has its own ard.toml. The CLI attempted to load that example as part of the root package and failed before discovering test/dram_test.ard.

Validation

  • cd compiler && go test . -run 'TestDiscoverTestFilesSkipsNestedProjects|TestTestCommand' -count=1
  • cd compiler && go test ./... -count=1
  • built the branch compiler and ran ard test . in ../dram: 27 passed, 0 failed, 0 panicked
  • git diff --check

@vercel

vercel Bot commented Jul 21, 2026

Copy link
Copy Markdown

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

Project Deployment Actions Updated (UTC)
ard-lang-dev Ready Ready Preview, Comment Jul 21, 2026 1:40am

@akonwi
akonwi merged commit 285ceae into main Jul 21, 2026
6 checks passed
@akonwi
akonwi deleted the fix/test-discovery-package-boundaries branch July 21, 2026 01:57
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