Improve exception messages in ffi integration

vkhramtsov · Jan 19, 2021 · 091e14c · 091e14c
1 parent 60883c4
commit 091e14c
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/src/Bicycle/Tesseract/Bridge/FFI.php b/src/Bicycle/Tesseract/Bridge/FFI.php
@@ -39,15 +39,15 @@ public function __construct(Configuration $configuration)
         try {
             $libaryPath = $this->configuration->getSharedLibraryPath();
             if (empty($libaryPath)) {
-                throw new Exception\Exception('Problem with connecting library via FFI');
+                throw new Exception\Exception('Problem with connecting library via FFI: empty library path');
             }
             /** @var FFI\TesseractInterface ffiInstance */
             $this->ffiInstance = \FFI::cdef(
                 $definitions,
                 $libaryPath
             );
         } catch (\FFI\Exception $e) {
-            throw new Exception\Exception('Problem with connecting library via FFI');
+            throw new Exception\Exception(sprintf('Problem with connecting library via FFI: %s', $e->getMessage()));
         }
     }
 

diff --git a/tests/Bicycle/Tesseract/Bridge/FFITest.php b/tests/Bicycle/Tesseract/Bridge/FFITest.php
@@ -59,7 +59,7 @@ public function testIncorrectConfiguration(): void
     {
         $this->isFFIEnabled();
         $this->expectException(Exception::class);
-        $this->expectExceptionMessage('Problem with connecting library via FFI');
+        $this->expectExceptionMessage('Problem with connecting library via FFI: empty library path');
         $configuration = new Configuration([]);
         new FFI($configuration);
     }