removed -n flag, deprecated -C option [WIP]

Author: dg

.github/workflows/tests.yml

@@ -24,7 +24,7 @@ jobs:
             - run: composer install --no-progress --prefer-dist
             - run: src/tester --info
               shell: bash
-            - run: src/tester tests -s -C
+            - run: src/tester tests -s
               shell: bash
             - if: failure()
               uses: actions/upload-artifact@v4
@@ -59,7 +59,7 @@ jobs:
                   coverage: none
 
             - run: composer install --no-progress --prefer-dist
-            - run: src/tester -p phpdbg tests -s -C --coverage ./coverage.xml --coverage-src ./src
+            - run: src/tester -p phpdbg tests -s --coverage ./coverage.xml --coverage-src ./src
             - run: wget https://github.com/php-coveralls/php-coveralls/releases/download/v2.4.3/php-coveralls.phar
             - env:
                   COVERALLS_REPO_TOKEN: ${{ secrets.GITHUB_TOKEN }}

CLAUDE.md

@@ -16,10 +16,10 @@ Nette Tester is a lightweight, standalone PHP testing framework designed for sim
 
 ```bash
 # or directly:
-src/tester tests -s -C
+src/tester tests -s
 
 # Run specific test file
-src/tester tests/Framework/Assert.phpt -s -C
+src/tester tests/Framework/Assert.phpt -s
 
 # or simply
 php tests/Framework/Assert.phpt
@@ -806,10 +806,7 @@ Assert::fail('Error in connection');
 
 ### PHP INI Handling
 
-Tester runs PHP processes with `-n` flag (no php.ini loaded):
-- Ensures consistent test environment
-- System extensions from `/etc/php/conf.d/*.ini` are NOT loaded
-- Use `-C` flag to load system php.ini
+Tester runs PHP processes with the system php.ini:
 - Use `-c path/to/php.ini` for custom php.ini
 - Use `-d key=value` for individual INI settings
 

readme.md

@@ -85,7 +85,7 @@ Now we run tests from command-line using the `tester` command:
 |_   _/ __)( __/_   _/ __)| _ )
   |_| \___ /___) |_| \___ |_|_\  v2.6
 
-PHP 8.2.0 | php -n | 8 threads
+PHP 8.2.0 | php | 8 threads
 .
 OK (1 tests, 0 skipped, 0.0 seconds)
 ```
@@ -217,8 +217,8 @@ Usage:
 
 Options:
     -p <path>                    Specify PHP interpreter to run (default: php).
-    -c <path>                    Look for php.ini file (or look in directory) <path>.
-    -C                           Use system-wide php.ini.
+    -c <path>                    Use custom php.ini, ignore system configuration.
+    -C                           With -c, include system configuration as well.
     -l | --log <path>            Write log to file <path>.
     -d <key=value>...            Define INI entry 'key' with value 'val'.
     -s                           Show information about skipped tests.

src/Runner/CliTester.php

@@ -108,8 +108,8 @@ private function loadOptions(): CommandLine
 
 				Options:
 				    -p <path>                    Specify PHP interpreter to run (default: php).
-				    -c <path>                    Look for php.ini file (or look in directory) <path>.
-				    -C                           Use system-wide php.ini.
+				    -c <path>                    Use custom php.ini, ignore system configuration.
+				    -C                           With -c, include system configuration as well.
 				    -d <key=value>...            Define INI entry 'key' with value 'value'.
 				    -s                           Show information about skipped tests.
 				    --stop-on-fail               Stop execution upon the first failure.
@@ -189,11 +189,12 @@ private function loadOptions(): CommandLine
 
 	private function createPhpInterpreter(): void
 	{
-		$args = $this->options['-C'] ? [] : ['-n'];
+		$args = [];
 		if ($this->options['-c']) {
+			if (!$this->options['-C']) {
+				$args[] = '-n';
+			}
 			array_push($args, '-c', $this->options['-c']);
-		} elseif (!$this->options['--info'] && !$this->options['-C']) {
-			echo "Note: No php.ini is used.\n";
 		}
 
 		if (in_array($this->stdoutFormat, ['tap', 'junit'], strict: true)) {

tests/Runner/PhpInterpreter.phpt

@@ -33,3 +33,10 @@ if (extension_loaded('pcov')) {
 }
 
 Assert::count($count, $engines);
+
+
+// createInterpreter() uses same php.ini as parent
+if (!$interpreter->isCgi()) {
+	$output = shell_exec($interpreter->withArguments(['-r echo php_ini_loaded_file();'])->getCommandLine());
+	Assert::same(php_ini_loaded_file(), $output);
+}

tests/bootstrap.php

@@ -14,12 +14,9 @@
 
 function createInterpreter(): PhpInterpreter
 {
-	$args = strlen((string) php_ini_scanned_files())
-		? []
-		: ['-n'];
-
-	if (php_ini_loaded_file()) {
-		array_push($args, '-c', php_ini_loaded_file());
+	$args = [];
+	if ($file = php_ini_loaded_file()) {
+		array_push($args, '-n', '-c', $file);
 	}
 
 	return new PhpInterpreter(PHP_BINARY, $args);