Skip to content

test(project-list): recursive root-walk and cwd integration tests - #37

Merged
mateodelnorte merged 1 commit into
mainfrom
feat/project-list-root-walk
Feb 21, 2026
Merged

mateodelnorte merged 1 commit into
mainfrom
feat/project-list-root-walk

Conversation

@mateodelnorte

Copy link
Copy Markdown
Contributor

Summary

Test plan

  • bats tests/project_list.bats — 16/16 passing (2 new)

🤖 Generated with Claude Code

Two new bats tests:
- Recursive from nested meta-repo shows root tree (not just local)
- JSON output includes absolute cwd field

Implements [[tasks/meta-65]]

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@coderabbitai

coderabbitai Bot commented Feb 21, 2026

Copy link
Copy Markdown

Warning

Rate limit exceeded

@mateodelnorte has exceeded the limit for the number of commits that can be reviewed per hour. Please wait 0 minutes and 47 seconds before requesting another review.

⌛ How to resolve this issue?

After the wait time has elapsed, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout.

Please see our FAQ for further information.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch feat/project-list-root-walk

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

@greptile-apps

greptile-apps Bot commented Feb 21, 2026

Copy link
Copy Markdown

Greptile Summary

Adds two integration tests for meta project list enhancements that verify recursive root-walk behavior and JSON cwd field presence.

Test 1: Recursive from nested meta-repo shows root tree

  • Verifies that running meta --recursive project list --json from inside a child meta-repo (frontend/) correctly outputs the root ancestor's project tree (frontend, backend, shared) rather than just the local child's projects
  • Validates that the cwd field is present and contains an absolute path

Test 2: JSON includes cwd field

  • Ensures meta project list --json output includes a cwd field with an absolute path

Both tests follow existing patterns in the file, use proper test isolation with setup/teardown, and validate JSON output using Python assertions. The tests depend on functionality from gitkb/meta_core#5 and gitkb/meta_project_cli#7.

Confidence Score: 5/5

  • This PR is safe to merge with no risk
  • The PR adds only test coverage with no production code changes. The tests are well-structured, follow existing patterns in the file, use proper setup/teardown, and have clear assertions. No logical errors, syntax issues, or security concerns present.
  • No files require special attention

Important Files Changed

Filename Overview
tests/project_list.bats Adds two well-structured integration tests verifying recursive root-walk behavior and cwd field presence in JSON output

Last reviewed commit: be20a04

@mateodelnorte
mateodelnorte merged commit 525c7c6 into main Feb 21, 2026
14 of 16 checks passed
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