test: cover php resolver and setup composer detection

Author: MekDrop

tests/01-setup.bats

@@ -9,17 +9,18 @@ setup() {
   export GITHUB_RUN_ID="99"
   export GITHUB_RUN_ATTEMPT="3"
   export GITHUB_ACTION_PATH="$BATS_TEST_DIRNAME/.."
+  export TEST_WORK_DIR="$TEST_TMP_DIR/workspace"
   export GITHUB_ENV="$TEST_TMP_DIR/github.env"
-  mkdir -p "$RUNNER_TEMP"
+  mkdir -p "$RUNNER_TEMP" "$TEST_WORK_DIR"
   : > "$GITHUB_ENV"
 }
 
 teardown() {
   teardown_test_tmp_dir
 }
 
-@test "setup exports variables and selects engine php version" {
-  run bash "$BATS_TEST_DIRNAME/../bin/setup.sh" "clean/phpdoc-md"
+@test "setup exports variables and falls back to engine php version" {
+  run bash -c "cd \"$TEST_WORK_DIR\" && bash \"$BATS_TEST_DIRNAME/../bin/setup.sh\" \"clean/phpdoc-md\""
   [ "$status" -eq 0 ]
 
   run grep -F "NEEDED_PHP_VERSION=8.1" "$GITHUB_ENV"
@@ -29,8 +30,24 @@ teardown() {
   [ -d "$NEW_WIKI_PATH" ]
 }
 
+@test "setup uses composer.json php constraint when present" {
+  cat > "$TEST_WORK_DIR/composer.json" <<'EOF'
+{
+  "require": {
+    "php": "^8.2"
+  }
+}
+EOF
+
+  run bash -c "cd \"$TEST_WORK_DIR\" && bash \"$BATS_TEST_DIRNAME/../bin/setup.sh\" \"clean/phpdoc-md\""
+  [ "$status" -eq 0 ]
+
+  run grep -F "NEEDED_PHP_VERSION=8.2" "$GITHUB_ENV"
+  [ "$status" -eq 0 ]
+}
+
 @test "setup fails on unknown engine" {
-  run bash "$BATS_TEST_DIRNAME/../bin/setup.sh" "unknown/engine"
+  run bash -c "cd \"$TEST_WORK_DIR\" && bash \"$BATS_TEST_DIRNAME/../bin/setup.sh\" \"unknown/engine\""
   [ "$status" -eq 1 ]
   [[ "$output" == *"ERROR: unknown engine"* ]]
 }

tests/09-resolve-php-version.bats

@@ -0,0 +1,64 @@
+#!/usr/bin/env bats
+
+load "$BATS_TEST_DIRNAME/helpers.bash"
+
+setup() {
+  setup_test_tmp_dir
+  export RESOLVER_SCRIPT="$BATS_TEST_DIRNAME/../bin/resolve-php-version.sh"
+}
+
+teardown() {
+  teardown_test_tmp_dir
+}
+
+@test "resolve-php-version uses engine defaults when composer files are missing" {
+  run bash -c "cd \"$TEST_TMP_DIR\" && bash \"$RESOLVER_SCRIPT\" \"clean/phpdoc-md\""
+  [ "$status" -eq 0 ]
+  [ "$output" = "8.1" ]
+
+  run bash -c "cd \"$TEST_TMP_DIR\" && bash \"$RESOLVER_SCRIPT\" \"evert/phpdoc-md\""
+  [ "$status" -eq 0 ]
+  [ "$output" = "7.4" ]
+}
+
+@test "resolve-php-version reads composer.json require.php first" {
+  cat > "$TEST_TMP_DIR/composer.json" <<'EOF'
+{
+  "require": {
+    "php": "^8.3 || ^8.4"
+  }
+}
+EOF
+
+  run bash -c "cd \"$TEST_TMP_DIR\" && bash \"$RESOLVER_SCRIPT\" \"clean/phpdoc-md\""
+  [ "$status" -eq 0 ]
+  [ "$output" = "8.3" ]
+}
+
+@test "resolve-php-version reads composer.lock when composer.json has no php requirement" {
+  cat > "$TEST_TMP_DIR/composer.json" <<'EOF'
+{
+  "require": {
+    "monolog/monolog": "^3.0"
+  }
+}
+EOF
+
+  cat > "$TEST_TMP_DIR/composer.lock" <<'EOF'
+{
+  "platform": {
+    "php": ">=8.2"
+  }
+}
+EOF
+
+  run bash -c "cd \"$TEST_TMP_DIR\" && bash \"$RESOLVER_SCRIPT\" \"clean/phpdoc-md\""
+  [ "$status" -eq 0 ]
+  [ "$output" = "8.2" ]
+}
+
+@test "resolve-php-version fails for unknown engine" {
+  run bash -c "cd \"$TEST_TMP_DIR\" && bash \"$RESOLVER_SCRIPT\" \"unknown/engine\""
+  [ "$status" -eq 1 ]
+  [[ "$output" == *"ERROR: unknown engine"* ]]
+}