Skip to content

ci(ncdd): add reusable app test workflow - #61

Merged
vitormattos merged 4 commits into
mainfrom
feat/ncdd-automation-consumers
Oct 7, 2026
Merged

vitormattos merged 4 commits into
mainfrom
feat/ncdd-automation-consumers

Conversation

@vitormattos

@vitormattos vitormattos commented Oct 7, 2026 •

Copy link
Copy Markdown
Member

Rebuilt on the current main after #60 merged. This increment exposes the accepted NCDD contract to GitHub Actions consumers without reintroducing the discarded development stack.

Highlights:

  • reusable .github/workflows/test-nextcloud-app.yml
  • PHPUnit and Behat only, matching the current ncdd test API
  • app checkout under the existing volumes/nextcloud/apps-extra/<app-id> tree
  • ncdd logs for CI diagnostics
  • optional Composer dependency bootstrap in the GitHub-hosted workspace
  • caller template under templates/workflows/ncdd-tests.yml
  • actions/checkout pinned to the immutable v7.0.1 commit

Explicitly removed from the old PR scope:

  • .ncdd.yml
  • scenario runner
  • frontend/node-worker test dispatch
  • second Compose stack and dev-worker image
  • Dev Container/Codespaces template
  • duplicated Nextcloud version selection

Dev Containers/Codespaces should be evaluated separately after this reusable CI contract is exercised by a real app repository.

The PR remains draft while CI and code review complete.

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 ci(ncdd): add reusable workflow, Dev Container and Codespaces path ci(ncdd): add reusable app test workflow Oct 7, 2026
@vitormattos
vitormattos changed the base branch from feat/ncdd-test-api to main October 7, 2026 16:45
Signed-off-by: Vitor Mattos <1079143+vitormattos@users.noreply.github.com>
@vitormattos
vitormattos marked this pull request as ready for review October 7, 2026 17:03
@vitormattos
vitormattos merged commit cf91ffb 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