Merge pull request #47 from LibreSign/bugfix/prevent-handle-string-as…

…-json Prevent handle string as json
LibreSign · Dec 26, 2023 · 30a99d0 · 30a99d0
2 parents 2a5762a + 2c6c66a
commit 30a99d0
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 0 deletions.
diff --git a/features/test.feature b/features/test.feature
@@ -37,6 +37,12 @@ Feature: Test this extension
       default
       """
 
+  Scenario: Test initial state with string
+    Then the response should contain the initial state "appid-string" with the following values:
+      """
+      "text as json string"
+      """
+
   Scenario: Test initial state with boolean
     Then the response should contain the initial state "appid-string" with the following values:
       """

diff --git a/src/NextcloudApiContext.php b/src/NextcloudApiContext.php
@@ -286,6 +286,10 @@ public function theResponseShouldContainTheInitialStateWithTheFollowingValues(st
 		if ($this->isJson($expected)) {
 			Assert::assertJsonStringEqualsJsonString($expected, $actual);
 		} else {
+			if (trim($actual, '"') === json_decode($actual) && str_starts_with($actual, '"')) {
+				$actual = trim($actual, '"');
+				$expected = trim($expected, '"');
+			}
 			Assert::assertEquals($expected, $actual);
 		}
 	}