Do not accept empty OCRmyPDF results and fix log #79

Signed-off-by: Robin Windey <[email protected]>

Author: R0Wi

lib/OcrProcessors/PdfOcrProcessor.php

@@ -50,15 +50,21 @@ public function ocrFile(string $fileContent): string {
 		$stdErr = $this->command->getStdErr();
 		$exitCode = $this->command->getExitCode();
 
-		if ($success) {
-			if ($stdErr !== '' || $errorOutput !== '') {
-				// Log warning if ocrmypdf wrote a warning to the stderr
-				$this->logger->warning('OCRmyPDF succeeded with warning(s): {stdErr}, {errorOutput}', [$stdErr, $errorOutput]);
-			}
+		if (!$success) {
+			throw new OcrNotPossibleException('OCRmyPDF exited abnormally with exit-code ' . $exitCode . '. Message: ' . $errorOutput . ' ' . $stdErr);
+		}
+
+		if ($stdErr !== '' || $errorOutput !== '') {
+			// Log warning if ocrmypdf wrote a warning to the stderr
+			$this->logger->warning('OCRmyPDF succeeded with warning(s): {stdErr}, {errorOutput}', [$stdErr, $errorOutput]);
+		}
+
+		$ocrOutput = $this->command->getOutput();
 
-			return $this->command->getOutput();
+		if (!$ocrOutput) {
+			throw new OcrNotPossibleException('OCRmyPDF did not produce any output');
 		}
 
-		throw new OcrNotPossibleException('OCRmyPDF exited abnormally with exit-code ' . $exitCode . '. Message: ' . $errorOutput . ' ' . $stdErr);
+		return $ocrOutput;
 	}
 }

tests/Unit/OcrProcessors/PdfOcrProcessorTest.php

@@ -111,6 +111,9 @@ public function testLogsWarningIfOcrMyPdfSucceedsWithWarningOutput() {
 		$this->command->expects($this->once())
 			->method('getStdErr')
 			->willReturn('stdErr');
+		$this->command->expects($this->once())
+			->method('getOutput')
+			->willReturn('someOcrFileContent');
 		$this->logger->expects($this->once())
 			->method('warning')
 			->with(
@@ -125,4 +128,33 @@ public function testLogsWarningIfOcrMyPdfSucceedsWithWarningOutput() {
 		$processor = new PdfOcrProcessor($this->command, $this->logger);
 		$processor->ocrFile('someContent');
 	}
+
+	public function testThrowsErrorIfOcrFileWasEmpty() {
+		$this->command->expects($this->once())
+			->method('execute')
+			->willReturn(true);
+		$this->command->expects($this->once())
+			->method('getError')
+			->willReturn('error');
+		$this->command->expects($this->once())
+			->method('getStdErr')
+			->willReturn('stdErr');
+		$this->command->expects($this->once())
+			->method('getOutput')
+			->willReturn('');
+
+	
+		$thrown = false;
+		$processor = new PdfOcrProcessor($this->command, $this->logger);
+
+		try {
+			$processor->ocrFile('someContent');
+		} catch (\Throwable $t) {
+			$thrown = true;
+			$this->assertInstanceOf(OcrNotPossibleException::class, $t);
+			$this->assertEquals('OCRmyPDF did not produce any output', $t->getMessage());
+		}
+		
+		$this->assertTrue($thrown);
+	}
 }