Skip to content

Added unit tests workflow#596

Draft
WilcoLouwerse wants to merge 143 commits intodevelopmentfrom
feature/CONNECTOR-371/unit-tests-workflow-new
Draft

Added unit tests workflow#596
WilcoLouwerse wants to merge 143 commits intodevelopmentfrom
feature/CONNECTOR-371/unit-tests-workflow-new

Conversation

@WilcoLouwerse
Copy link
Contributor

No description provided.

@WilcoLouwerse WilcoLouwerse force-pushed the feature/CONNECTOR-371/unit-tests-workflow-new branch from b35736f to 4474f2e Compare October 7, 2025 11:49
@WilcoLouwerse WilcoLouwerse force-pushed the feature/CONNECTOR-371/unit-tests-workflow-new branch 2 times, most recently from 92a26dd to 1c79ec5 Compare October 7, 2025 14:02
@WilcoLouwerse WilcoLouwerse force-pushed the feature/CONNECTOR-371/unit-tests-workflow-new branch from 1c79ec5 to 1405e48 Compare October 7, 2025 14:03
- ensure dev tools are in require-dev
- update composer.lock via `composer update -W phpunit/phpunit friendsofphp/php-cs-fixer vimeo/psalm`
- unblocks CI “composer install” on the runner
…urate testing

- Removed hardcoded nextcloud-dev-php83:latest from global env section
- Added dynamic image selection step in tests job based on matrix.php-version
- Tests job now uses ghcr.io/juliusknorr/nextcloud-dev-php82:latest for PHP 8.2
- Tests job now uses ghcr.io/juliusknorr/nextcloud-dev-php83:latest for PHP 8.3
- Quality job uses ghcr.io/juliusknorr/nextcloud-dev-php83:latest for PHP 8.3
- All images now use ghcr.io/juliusknorr/ prefix for consistency
- Updated COMPREHENSIVE_DOCUMENTATION.md to version 1.51 with changelog entry

This ensures each test runs against a Nextcloud container with the exact PHP version being tested, improving test accuracy and eliminating version mismatches.
@rjzondervan rjzondervan marked this pull request as draft March 20, 2026 10:43
@rjzondervan
Copy link
Member

PR converted to draft to serve as inspiration for refactor

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.

2 participants