Merge pull request #58 from LibreSign/feature/validate-if-jq-exists

Validate if jq exists

psalm-baseline.xml

@@ -1,8 +1,9 @@
 <?xml version="1.0" encoding="UTF-8"?>
-<files psalm-version="5.22.2@d768d914152dbbf3486c36398802f74e80cfde48">
+<files psalm-version="5.23.1@8471a896ccea3526b26d082f4461eeea467f10a4">
   <file src="src/NextcloudApiContext.php">
     <ForbiddenCode>
       <code><![CDATA[`which jq`]]></code>
+      <code><![CDATA[`which jq`]]></code>
     </ForbiddenCode>
   </file>
 </files>

src/NextcloudApiContext.php

@@ -295,14 +295,13 @@ private function testAndGetActualValue(array $value, string $json): string {
 	 * @return mixed
 	 */
 	private function evalJsonQuery(string $jsonQuery, string $target) {
+		Assert::assertNotEmpty(`which jq`, 'Is necessary install the jq command to use jq');
 		$jq = \JsonQueryWrapper\JsonQueryFactory::createWith($target);
 		return $jq->run($jsonQuery);
 	}
 
 	private function validateAsJsonQuery(string $expected, string $actual): void {
-		if (!`which jq`) {
-			throw new \InvalidArgumentException('Is necessary install the jq command to use jq');
-		}
+		Assert::assertNotEmpty(`which jq`, 'Is necessary install the jq command to use jq');
 		$jq = \JsonQueryWrapper\JsonQueryFactory::createWith($actual);
 		$result = $jq->run($expected);
 		Assert::assertTrue($result, 'The jq "' . $expected . '" do not match with: ' . $actual);