Skip to content

feat: add reusable acceptance test workflow#49

Open
phil-davis wants to merge 1 commit intomainfrom
acceptance-workflow
Open

feat: add reusable acceptance test workflow#49
phil-davis wants to merge 1 commit intomainfrom
acceptance-workflow

Conversation

@phil-davis
Copy link
Copy Markdown
Contributor

@phil-davis phil-davis commented Apr 17, 2026

This is the "basic" flow that sets up Apache, installs a single or dual-server environment with 1 or 2 owncloud core instances running with the app-under-test set up and enabled. That allows ordinary tests and federated tests to run and pass.

I have tried to pass things in env variables etc, in the same way as was done in the other PRs today that secure the flow from command injection, validate inputs... That will need reviewing to find any holes that I have missed.

In future (next week), the acceptance reusable workflow will need to be enhanced for apps that use more external environment. For example:

  • provide a mail server
  • provide an LDAP environment
  • provide S3 storage (or other back-end storage types), for testing apps that support different storage
  • allow running with https (set up certificates...) - we might want to be able to run acceptance tests using https

@phil-davis phil-davis force-pushed the acceptance-workflow branch from 127abe7 to c3ad95d Compare April 17, 2026 13:02
@phil-davis phil-davis marked this pull request as ready for review April 17, 2026 13:10
@phil-davis
Copy link
Copy Markdown
Contributor Author

@DeepDiver1975 @jvillafanez this works for activity, admin_audit and impersonate apps.
It probably has enough flexibility for lots of the "ordinary" apps.

Please review. Specially for security loopholes...

And I think that we could merge and use this, and then I can expand its features/options next week to cover more of the "special" apps (things that send email, LDAP, files_primary_s3....). Let me know which "external feature" apps are most important to support first.

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