Skip to content

feat(ncdd): add focused app test commands - #60

Merged
vitormattos merged 6 commits into
mainfrom
feat/ncdd-test-api
Oct 7, 2026
Merged

vitormattos merged 6 commits into
mainfrom
feat/ncdd-test-api

Conversation

@vitormattos

@vitormattos vitormattos commented Oct 7, 2026 •

Copy link
Copy Markdown
Member

Builds directly on the merged NCDD CLI foundation from #59 and adds only the first focused test interface.

Highlights:

  • bin/ncdd test --list
  • bin/ncdd test phpunit --app APP -- ...
  • bin/ncdd test behat --app APP -- ...
  • source code remains in the existing volumes/nextcloud/apps-extra/ tree
  • no .ncdd.yml
  • no second Compose stack or dev-worker image
  • no frontend runner or scenario mini-framework in this PR
  • test-runner arguments are passed as an argument array, without shell evaluation
  • app ids are validated before they become container paths
  • Bats covers the test command contract and CI runs ShellCheck over both CLI modules

The branch was rebuilt from current main after #59 merged so this PR can be reviewed independently against the accepted architecture. It remains draft for code review.

Signed-off-by: Vitor Mattos <1079143+vitormattos@users.noreply.github.com>
Signed-off-by: Vitor Mattos <1079143+vitormattos@users.noreply.github.com>
Signed-off-by: Vitor Mattos <1079143+vitormattos@users.noreply.github.com>
@vitormattos vitormattos changed the title feat(ncdd): add test API and reproducible scenarios feat(ncdd): add focused app test commands Oct 7, 2026
@vitormattos
vitormattos changed the base branch from feat/ncdd-cli-runtime to main October 7, 2026 15:52
Signed-off-by: Vitor Mattos <1079143+vitormattos@users.noreply.github.com>
Signed-off-by: Vitor Mattos <1079143+vitormattos@users.noreply.github.com>
Signed-off-by: Vitor Mattos <1079143+vitormattos@users.noreply.github.com>
@vitormattos
vitormattos marked this pull request as ready for review October 7, 2026 16:34
@vitormattos
vitormattos merged commit cfbc8b1 into main Oct 7, 2026
4 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