Change behavior of tests when req_mirror isn't available

Make them to be failed
line · Nov 21, 2016 · b47c680 · b47c680
1 parent 6c403f9
commit b47c680
Showing 1 changed file with 1 addition and 6 deletions.
diff --git a/tests/LINEBot/HTTPClient/CurlHTTPClientTest.php b/tests/LINEBot/HTTPClient/CurlHTTPClientTest.php
@@ -48,12 +48,7 @@ public static function setUpBeforeClass()
                 CurlHTTPClientTest::$reqMirrorPID = $out[0];
                 sleep(1); // XXX
             }
-
-            return;
         }
-
-        echo PHP_EOL, '[WARN] req_mirror does not exist. Please try to execute `make install-devtool`', PHP_EOL;
-        echo '[WARN] Skip CurlHTTPClientTest', PHP_EOL;
     }
 
     public static function tearDownAfterClass()
@@ -66,7 +61,7 @@ public static function tearDownAfterClass()
     protected function setUp()
     {
         if (!file_exists(CurlHTTPClientTest::$reqMirrorPath) || empty(CurlHTTPClientTest::$reqMirrorPID)) {
-            $this->markTestSkipped('req_mirror is not available');
+            $this->fail('req_mirror server is not available. Please try to execute `make install-devtool`');
         }
     }