Make fail_loud apply to renderer exceptions

Author: ryan.kelley

src/Limenius/ReactRenderer/Renderer/ExternalServerReactRenderer.php

@@ -4,6 +4,7 @@
 
 use Limenius\ReactRenderer\Context\ContextProviderInterface;
 use Psr\Log\LoggerInterface;
+use Psr\Log\LogLevel;
 
 /**
  * Class ExternalServerReactRenderer
@@ -55,23 +56,43 @@ public function setServerSocketPath($serverSocketPath)
      */
     public function render($componentName, $propsString, $uuid, $registeredStores = array(), $trace)
     {
-        if (strpos($this->serverSocketPath, '://') === false) {
-            $this->serverSocketPath = 'unix://'.$this->serverSocketPath;
-        }
+        try {
+            if (\strpos($this->serverSocketPath, '://') === false) {
+                $this->serverSocketPath = 'unix://'.$this->serverSocketPath;
+            }
 
-        if (!$sock = stream_socket_client($this->serverSocketPath, $errno, $errstr)) {
-            throw new \RuntimeException($errstr);
-        }
-        stream_socket_sendto($sock, $this->wrap($componentName, $propsString, $uuid, $registeredStores, $trace)."\0");
+            if (!$sock = \stream_socket_client($this->serverSocketPath, $errno, $errstr)) {
+                throw new \RuntimeException($errstr);
+            }
+            \stream_socket_sendto(
+                $sock,
+                $this->wrap($componentName, $propsString, $uuid, $registeredStores, $trace)."\0"
+            );
+
+            $contents = '';
 
-        $contents = '';
+            while (!\feof($sock)) {
+                $contents .= \fread($sock, 8192);
+            }
+            \fclose($sock);
+
+            $result = \json_decode($contents, true);
+        } catch (\Throwable $t) {
+            if ($this->failLoud) {
+                throw $t;
+            }
+
+            if ($this->logger) {
+                $this->logger->log(LogLevel::ERROR, $t->getMessage(), ['exception' => $t]);
+            }
 
-        while (!feof($sock)) {
-            $contents .= fread($sock, 8192);
+            return [
+                'evaluated'     => '',
+                'consoleReplay' => '',
+                'hasErrors'     => true,
+            ];
         }
-        fclose($sock);
 
-        $result = json_decode($contents, true);
         if ($result['hasErrors']) {
             $this->logErrors($result['consoleReplayScript']);
             if ($this->failLoud) {

src/Limenius/ReactRenderer/Renderer/PhpExecJsReactRenderer.php

@@ -2,10 +2,11 @@
 
 namespace Limenius\ReactRenderer\Renderer;
 
-use Nacmartin\PhpExecJs\PhpExecJs;
-use Psr\Log\LoggerInterface;
 use Limenius\ReactRenderer\Context\ContextProviderInterface;
+use Nacmartin\PhpExecJs\PhpExecJs;
 use Psr\Cache\CacheItemPoolInterface;
+use Psr\Log\LoggerInterface;
+use Psr\Log\LogLevel;
 
 /**
  * Class PhpExecJsReactRenderer
@@ -93,15 +94,38 @@ public function setServerBundlePath($serverBundlePath)
      */
     public function render($componentName, $propsString, $uuid, $registeredStores = array(), $trace)
     {
-        $this->ensurePhpExecJsIsBuilt();
-        if ($this->needToSetContext) {
-            if ($this->phpExecJs->supportsCache()) {
-                $this->phpExecJs->setCache($this->cache);
+        try {
+            $this->ensurePhpExecJsIsBuilt();
+            if ($this->needToSetContext) {
+                if ($this->phpExecJs->supportsCache()) {
+                    $this->phpExecJs->setCache($this->cache);
+                }
+                $this->phpExecJs->createContext(
+                    $this->consolePolyfill()."\n".$this->timerPolyfills($trace)."\n".$this->loadServerBundle(),
+                    $this->cacheKey
+                );
+                $this->needToSetContext = false;
+            }
+            $result = \json_decode(
+                $this->phpExecJs->evalJs($this->wrap($componentName, $propsString, $uuid, $registeredStores, $trace)),
+                true
+            );
+        } catch (\Throwable $t) {
+            if ($this->failLoud) {
+                throw $t;
+            }
+
+            if ($this->logger) {
+                $this->logger->log(LogLevel::ERROR, $t->getMessage(), ['exception' => $t]);
             }
-            $this->phpExecJs->createContext($this->consolePolyfill()."\n".$this->timerPolyfills($trace)."\n".$this->loadServerBundle(), $this->cacheKey);
-            $this->needToSetContext = false;
+
+            return [
+                'evaluated'     => '',
+                'consoleReplay' => '',
+                'hasErrors'     => true,
+            ];
         }
-        $result = json_decode($this->phpExecJs->evalJs($this->wrap($componentName, $propsString, $uuid, $registeredStores, $trace)), true);
+
         if ($result['hasErrors']) {
             $this->logErrors($result['consoleReplayScript']);
             if ($this->failLoud) {

tests/Limenius/ReactRenderer/Tests/Renderer/PhpExecJsReactRendererTest.php

@@ -7,6 +7,7 @@
 use Psr\Log\LoggerInterface;
 use Nacmartin\PhpExecJs\PhpExecJs;
 use PHPUnit\Framework\TestCase;
+use Psr\Log\LogLevel;
 
 /**
  * Class PhpExecJsReactRendererTest
@@ -121,4 +122,51 @@ public function testFailLoud()
         $this->renderer->setPhpExecJs($phpExecJs);
         $this->renderer->render('MyApp', 'props', 1, null, true);
     }
+
+    /**
+     * @testdox failLoud true bubbles thrown exceptions
+     */
+    public function testFailLoudBubblesThrownException()
+    {
+        $err = new \Exception('test exception');
+        $this->phpExecJs->method('createContext')->willThrowException($err);
+        $this->renderer = new PhpExecJsReactRenderer(__DIR__.'/Fixtures/server-bundle.js', true, $this->contextProvider, $this->logger);
+        $this->renderer->setPhpExecJs($this->phpExecJs);
+
+        $this->expectExceptionObject($err);
+        $this->renderer->render('MyApp', 'props', 1, null, true);
+    }
+
+    /**
+     * @testdox failLoud false returns empty error result on exception
+     */
+    public function testFailQuietReturnsEmptyErrorResultOnException()
+    {
+        $this->phpExecJs->method('createContext')->willThrowException(new \Exception('test exception'));
+
+        $this->assertEquals(
+            [
+                'evaluated'     => '',
+                'consoleReplay' => '',
+                'hasErrors'     => true,
+            ],
+            $this->renderer->render('MyApp', 'props', 1, null, true)
+        );
+    }
+
+    /**
+     * @testdox failLoud false logs thrown exceptions
+     */
+    public function testFailQuietLogsThrownExceptions()
+    {
+        $err = new \Exception('test exception');
+        $this->phpExecJs->method('createContext')->willThrowException($err);
+
+        $this->logger
+            ->expects($this->exactly(1))
+            ->method('log')
+            ->with(LogLevel::ERROR, 'test exception', ['exception' => $err]);
+
+        $this->renderer->render('MyApp', 'props', 1, null, true);
+    }
 }