Skip to content

Commit

Permalink
Fix test invocation once more
Browse files Browse the repository at this point in the history
--testsuite is still old comma-separated format for multiple
  • Loading branch information
matthiasmullie committed Apr 8, 2024
1 parent 6f3c483 commit 25b5fc6
Showing 1 changed file with 4 additions and 5 deletions.
9 changes: 4 additions & 5 deletions makefile
Original file line number Diff line number Diff line change
Expand Up @@ -37,14 +37,13 @@ test:
# make test - tests all adapters on latest PHP version
# make test PHP=8.3 ADAPTER=Memcached - tests Memcached on PHP 8.3
VOLUMES="";\
ADAPTER=($$(echo $(ADAPTER) | tr ',' "\n"));\
GROUP=($$(echo $(GROUP) | tr ',' "\n"));\
for VOLUME in $$(echo "$(VOLUME_BINDS)" | tr "," "\n"); do VOLUMES="$$VOLUMES -v $$(pwd)/$$VOLUME:/var/www/$$VOLUME"; done;\
test "$(PHP)" && TEST_CONTAINER=php-$(PHP) || TEST_CONTAINER=php;\
DEPENDENT_CONTAINERS="$(filter-out apc flysystem memorystore sqlite, $(shell echo $(ADAPTER) | tr 'A-Z' 'a-z'))";\
RELEVANT_CONTAINERS="$$TEST_CONTAINER $(filter-out apc flysystem memorystore sqlite, $(shell echo $(ADAPTER) | tr 'A-Z' 'a-z'))";\
DEPENDENT_CONTAINERS="$(filter-out apc flysystem memorystore sqlite, $(shell echo $(ADAPTER) | tr 'A-Z,' 'a-z '))";\
RELEVANT_CONTAINERS="$$TEST_CONTAINER $(filter-out apc flysystem memorystore sqlite, $(shell echo $(ADAPTER) | tr 'A-Z,' 'a-z '))";\
docker-compose up --no-deps --wait -d $$DEPENDENT_CONTAINERS;\
docker-compose run --no-deps $$VOLUMES $$TEST_CONTAINER env XDEBUG_MODE=coverage vendor/bin/phpunit $${GROUP[@]/#/--group } $${ADAPTER[@]/#/--testsuite } --coverage-clover build/coverage-$(PHP)-$(shell echo $(ADAPTER) | tr ' ' '-').clover;\
GROUP_ARRAY=($$(echo "$(GROUP)" | tr "," "\n"));\
docker-compose run --no-deps $$VOLUMES $$TEST_CONTAINER env XDEBUG_MODE=coverage vendor/bin/phpunit $${GROUP_ARRAY[@]/#/--group } --testsuite $(ADAPTER) --coverage-clover build/coverage-$(PHP)-$(ADAPTER).clover;\
TEST_STATUS=$$?;\
docker-compose stop -t0 $$RELEVANT_CONTAINERS;\
exit $$TEST_STATUS
Expand Down

0 comments on commit 25b5fc6

Please sign in to comment.