Fix testProcessEnvIsValid and skip signal test

amphp · Dec 5, 2017 · 2591a9c · 2591a9c
1 parent 63ef6dd
commit 2591a9c
Showing 1 changed file with 6 additions and 2 deletions.
diff --git a/test/ProcessTest.php b/test/ProcessTest.php
@@ -71,6 +71,10 @@ public function testCommandArray() {
     }
 
     public function testProcessCanTerminate() {
+        if (\DIRECTORY_SEPARATOR === "\\") {
+            $this->markTestSkipped("Signals are not supported on Windows");
+        }
+
         Loop::run(function () {
             $process = new Process(self::CMD_PROCESS);
             $process->start();
@@ -137,11 +141,11 @@ public function testGetStderrIsCustomized() {
 
     public function testProcessEnvIsValid() {
         $process = new Process(self::CMD_PROCESS, null, [
-            'env_value'
+            'env_name' => 'env_value'
         ]);
         $process->start();
         $promise = $process->join();
-        $this->assertSame('env_value', $process->getEnv()[0]);
+        $this->assertSame('env_value', $process->getEnv()['env_name']);
     }
 
     /**