Improve tests to actually verify environment variables are set

Co-authored-by: swissspidy <841956+swissspidy@users.noreply.github.com>

Author: Copilot

features/package-install.feature

@@ -1263,7 +1263,7 @@ Feature: Install WP-CLI packages
       """
     And STDOUT should be empty
 
-  Scenario: Install a package with --no-interaction flag
+  Scenario: Install a package with --no-interaction flag sets environment variables
     Given an empty directory
     And a composer.json file:
       """
@@ -1284,11 +1284,25 @@ Feature: Install WP-CLI packages
       """
       {
         "name": "wp-cli/restful",
-        "description": "Test package for no-interaction flag"
+        "description": "Test package for no-interaction flag",
+        "scripts": {
+          "post-install-cmd": [
+            "@php -r \"echo 'GIT_TERMINAL_PROMPT=' . getenv('GIT_TERMINAL_PROMPT') . PHP_EOL;\"",
+            "@php -r \"echo 'GIT_SSH_COMMAND=' . getenv('GIT_SSH_COMMAND') . PHP_EOL;\""
+          ]
+        }
       }
       """
     When I run `WP_CLI_PACKAGES_DIR=. wp package install wp-cli/restful --no-interaction`
     Then STDOUT should contain:
+      """
+      GIT_TERMINAL_PROMPT=0
+      """
+    And STDOUT should contain:
+      """
+      GIT_SSH_COMMAND=ssh -o BatchMode=yes
+      """
+    And STDOUT should contain:
       """
       Success: Package installed
       """

features/package-update.feature

@@ -195,22 +195,51 @@ Feature: Update WP-CLI packages
       """
     And the return code should be 1
 
-  Scenario: Update packages with --no-interaction flag
+  Scenario: Update packages with --no-interaction flag sets environment variables
     Given an empty directory
-
-    When I run `wp package install danielbachhuber/wp-cli-reset-post-date-command`
-    Then STDOUT should contain:
-      """
-      Success: Package installed.
+    And a composer.json file:
+      """
+      {
+        "repositories": {
+          "test" : {
+            "type": "path",
+            "url": "./dummy-package/"
+          },
+          "wp-cli": {
+            "type": "composer",
+            "url": "https://wp-cli.org/package-index/"
+          }
+        },
+        "require": {
+          "wp-cli/restful": "*"
+        }
+      }
+      """
+    And a dummy-package/composer.json file:
+      """
+      {
+        "name": "wp-cli/restful",
+        "version": "1.0.0",
+        "description": "Test package for no-interaction flag",
+        "scripts": {
+          "post-update-cmd": [
+            "@php -r \"echo 'GIT_TERMINAL_PROMPT=' . getenv('GIT_TERMINAL_PROMPT') . PHP_EOL;\"",
+            "@php -r \"echo 'GIT_SSH_COMMAND=' . getenv('GIT_SSH_COMMAND') . PHP_EOL;\""
+          ]
+        }
+      }
+      """
+
+    When I run `WP_CLI_PACKAGES_DIR=. wp package update --no-interaction`
+    Then STDOUT should contain:
+      """
+      GIT_TERMINAL_PROMPT=0
       """
-
-    When I run `wp package update --no-interaction`
-    Then STDOUT should contain:
+    And STDOUT should contain:
       """
-      Using Composer to update packages...
+      GIT_SSH_COMMAND=ssh -o BatchMode=yes
       """
     And STDOUT should contain:
       """
       Packages updated.
       """
-    And STDERR should be empty

features/package.feature

@@ -209,23 +209,53 @@ Feature: Manage WP-CLI packages
       {NO_SUCH_PACKAGE_COMPOSER_JSON}
       """
 
-  Scenario: Uninstall a package with --no-interaction flag
+  Scenario: Uninstall a package with --no-interaction flag sets environment variables
     Given an empty directory
+    And a composer.json file:
+      """
+      {
+        "repositories": {
+          "test" : {
+            "type": "path",
+            "url": "./dummy-package/"
+          },
+          "wp-cli": {
+            "type": "composer",
+            "url": "https://wp-cli.org/package-index/"
+          }
+        },
+        "require": {
+          "wp-cli/restful": "*"
+        }
+      }
+      """
+    And a dummy-package/composer.json file:
+      """
+      {
+        "name": "wp-cli/restful",
+        "version": "1.0.0",
+        "description": "Test package for no-interaction flag",
+        "scripts": {
+          "pre-uninstall-cmd": [
+            "@php -r \"echo 'GIT_TERMINAL_PROMPT=' . getenv('GIT_TERMINAL_PROMPT') . PHP_EOL;\"",
+            "@php -r \"echo 'GIT_SSH_COMMAND=' . getenv('GIT_SSH_COMMAND') . PHP_EOL;\""
+          ]
+        }
+      }
+      """
 
-    When I run `wp package install runcommand/hook`
-    Then STDERR should be empty
-
-    When I run `wp package uninstall runcommand/hook --no-interaction`
-    Then STDERR should be empty
+    When I run `WP_CLI_PACKAGES_DIR=. wp package uninstall wp-cli/restful --no-interaction`
+    Then STDOUT should contain:
+      """
+      GIT_TERMINAL_PROMPT=0
+      """
     And STDOUT should contain:
       """
-      Success: Uninstalled package.
+      GIT_SSH_COMMAND=ssh -o BatchMode=yes
       """
-
-    When I run `wp package list`
-    Then STDOUT should not contain:
+    And STDOUT should contain:
       """
-      runcommand/hook
+      Success: Uninstalled package.
       """
 
   Scenario: List packages with --skip-update-check flag