Implement test with bash_unit

Author: mistraloz

.gitignore

@@ -1,2 +1 @@
-.idea
-tmp
+.idea

Makefile

@@ -16,16 +16,18 @@ test-quick:  ## Test 8.0 and 8.1 quickly
 test-8.1:  ## Test php8.1 build only
 	VERSION=8.1 VARIANT=cli $(MAKE) _test-version
 	VERSION=8.1 VARIANT=apache $(MAKE) _test-version
+	VERSION=8.1 VARIANT=fpm $(MAKE) _test-version
 
 test-8.0:  ## Test php8.0 build only
 	VERSION=8.0 VARIANT=cli $(MAKE) _test-version
 	VERSION=8.0 VARIANT=apache $(MAKE) _test-version
+	VERSION=8.0 VARIANT=fpm $(MAKE) _test-version
 
 _test-version: _test-prerequisites ## Test php build for VERSION="" and VARIANT=""
 	docker buildx bake --load \
 	    --set "*.platform=$$(uname -p)" \
-		php$${VERSION//.}-cli
-	PHP_VERSION="$(VERSION)" BRANCH=v4 VARIANT=cli ./test-image.sh || (notify-send -u critical "Tests failed ($(VERSION)-$(VARIANT))" && exit 1)
+		php$${VERSION//.}-$(VARIANT)-all
+	PHP_VERSION="$(VERSION)" BRANCH=v4 VARIANT=$(VARIANT) ./tests-suite/bash_unit -f tap ./tests-suite/*.sh || (notify-send -u critical "Tests failed ($(VERSION)-$(VARIANT))" && exit 1)
 	notify-send -u critical "Tests passed with success ($(VERSION)-$(VARIANT))"
 
 clean: ## Clean dangles image after build

test-image.sh

@@ -0,0 +1,25 @@
+# Test suite
+
+These tests use bash_unit to allow a clean visualisation of tests. 
+This file describe some features used here. 
+
+## `./config` file
+
+This file allow the test suite to have some variables / command already set. You can customize.
+
+## Setup/teardown
+
+Like any regular test unit process, you can play command at initialisation of the test, and other at this end. It's used to generate fake directories or start docker commands. The teardown allow to clean directory/started container but if any fatal error happens (or `ctrl+c`), the tear down if not played. So you can have unwanted container started (you need to clean them manually, but it's not require to clean to re-execute because port/name/directory are random and uniq for each test suite).
+
+## Pipe to `/dev/null`
+
+Lot of command display result, error, warning. All are forward to /dev/null when it's not relevant (only the exit code is used `$?`). It's maybe usefull to provide a full log on these cases if it's possible to find a solution for that. 
+
+## Uncommon command `tail -n +1`
+
+When there is comparaison with grep, it's required to prefix by ` | tail -n +1`. It's do nothing at all (only display from the first line to the end) but avoid this warning "write /dev/stdout: broken pipe" if command make too long time for response (required for arm64 testing).
+
+## `[[ $VARIANT != apache* ]]`
+
+Some part of the test is not depend on variant of image. Per example here, apache. It's maybe useful to find another way to just ignore the test instead of `exit 0` the full file (bash unit do not manage that currently). 
+

tests-suite/README.md

@@ -1,8 +1,10 @@
 ARG BRANCH
 ARG BRANCH_VARIANT
 ARG PHP_VERSION
+ARG REPO
+ARG TAG_PREFIX
 
-FROM thecodingmachine/php:${PHP_VERSION}-${BRANCH}-${BRANCH_VARIANT}
+FROM ${REPO}:${TAG_PREFIX}${PHP_VERSION}-${BRANCH}-${BRANCH_VARIANT}
 
 ENV PHP_EXTENSION_GD=1