List all process

Signed-off-by: Vitor Mattos <vitor@php.rio>
LibreSign · May 29, 2023 · 1ab25ca · 1ab25ca
1 parent b7f5e3b
commit 1ab25ca
Show file tree

Hide file tree

Showing 5 changed files with 30 additions and 12 deletions.
diff --git a/.github/workflows/behat.yml b/.github/workflows/behat.yml
@@ -25,4 +25,8 @@ jobs:
         run: composer install
 
       - name: Run behat
-        run: vendor/bin/behat
+        run: |
+          export BEHAT_RUN_AS=$(ls -ld behat.yml | awk '{print $3}')
+          export BEHAT_VERBOSE="$RUNNER_DEBUG"
+          env
+          vendor/bin/behat
diff --git a/README.md b/README.md
@@ -26,7 +26,7 @@ default:
 
 | config  | default       | Environment    | Description                                        |
 | ------- | ------------- | -------------- | -------------------------------------------------- |
-| verbose | false         | none           | Enables/disables verbose mode                      |
+| verbose | false         | BEHAT_VERBOSE  | Enables/disables verbose mode                      |
 | rootDir | /var/www/html | BEHAT_HOST     | Specifies http root dir                            |
 | host    | localhost     | BEHAT_ROOT_DIR | Host domain or IP                                  |
 | runAs   |               | BEHAT_RUN_AS   | The username to be used to run the built-in server |

diff --git a/src/RunServerListener.php b/src/RunServerListener.php
@@ -114,7 +114,10 @@ public function start(): void
         }
 
         if (!$this->isRunning()) {
-            throw new ServerException('Failed to start server. Is something already running on port ' . self::$port . '?');
+            throw new ServerException(
+                'Failed to start server. Is something already running on port ' . self::$port . "?\n" .
+                'Full command: ' . $fullCmd
+            );
         }
 
         register_shutdown_function(function () {
@@ -136,6 +139,7 @@ public function isRunning(): bool
         }
 
         exec(sprintf('ps %d', $this->pid), $result);
+        var_dump($result);
 
         return count($result) > 1;
     }
@@ -162,7 +166,7 @@ public function killZombies(): void
             'grep "php -S ' . self::$host . '"|' .
             'grep -v grep|' .
             'sed -e "s/^[[:space:]]*//"|cut -d" " -f1';
-        $pids = trim(exec($cmd));
+        $pids = trim(shell_exec($cmd));
         $pids = explode("\n", $pids);
         foreach ($pids as $pid) {
             if ($pid) {

diff --git a/src/Server.php b/src/Server.php
@@ -83,16 +83,19 @@ public function configure(ArrayNodeDefinition $builder): void
     /** @inheritDoc */
     public function load(ContainerBuilder $container, array $config): void
     {
-        $verbose = $this->getVerboseLevel($container, $config);
         $rootDir = $this->getRootDir($config);
+        $host = $this->getHost($config);
+        $runAs = $this->getRunAs($config);
+        $verbose = $this->getVerboseLevel($container, $config);
         if (is_numeric($verbose)) {
             $output = $container->get('cli.output');
             if ($output instanceof OutputInterface) {
                 $output->writeln('<info>Root dir: ' . $rootDir . '</info>');
+                $output->writeln('<info>Verbose: ' . $verbose . '</info>');
+                $output->writeln('<info>Host: ' . $host . '</info>');
+                $output->writeln('<info>RunAs: ' . $runAs . '</info>');
             }
         }
-        $host = $this->getHost($config);
-        $runAs = $this->getRunAs($config);
         $definition = (new Definition('PhpBuiltin\RunServerListener'))
             ->addTag('event_dispatcher.subscriber')
             ->setArguments([$verbose, $rootDir, $host, $runAs])
@@ -150,6 +153,10 @@ private function getVerboseLevel(ContainerBuilder $container, array $config): ?i
             $verbose = $input->getParameterOption('-v');
             return strlen($verbose);
         }
+        $runAs = getenv('BEHAT_VERBOSE');
+        if (is_numeric($runAs)) {
+            return (int) $runAs;
+        }
         return $config['verbose'] ? 0 : null;
     }
 

diff --git a/tests/psalm-baseline.xml b/tests/psalm-baseline.xml
@@ -1,8 +1,11 @@
 <?xml version="1.0" encoding="UTF-8"?>
-<files psalm-version="5.12.0@f90118cdeacd0088e7215e64c0c99ceca819e176">
-  <file src="src/Server.php">
-    <UndefinedInterfaceMethod>
-      <code>scalarNode</code>
-    </UndefinedInterfaceMethod>
+<files psalm-version="5.4.0@62db5d4f6a7ae0a20f7cc5a4952d730272fc0863">
+  <file src="src/RunServerListener.php">
+    <ForbiddenCode occurrences="1">
+      <code>shell_exec($cmd)</code>
+    </ForbiddenCode>
+    <PossiblyNullArgument occurrences="1">
+      <code>shell_exec($cmd)</code>
+    </PossiblyNullArgument>
   </file>
 </files>