fixup! Adjust tests

Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
nextcloud · Feb 18, 2024 · 5ea7066 · 5ea7066
1 parent 19fb7cb
commit 5ea7066
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 2 deletions.
diff --git a/tests/Unit/Controller/ApiControllerTest.php b/tests/Unit/Controller/ApiControllerTest.php
@@ -386,6 +386,7 @@ public function dataTestCreateNewForm() {
 		return [
 			'forms' => ['expectedForm' => [
 				'id' => 7,
+				'state' => 0,
 				'hash' => 'formHash',
 				'title' => '',
 				'description' => '',

diff --git a/tests/Unit/Service/FormsServiceTest.php b/tests/Unit/Service/FormsServiceTest.php
@@ -170,6 +170,7 @@ public function dataGetForm() {
 			// Just the full form without submissions
 			'one-full-form' => [[
 				'id' => 42,
+				'state' => 0,
 				'hash' => 'abcdefg',
 				'title' => 'Form 1',
 				'description' => 'Description Text',
@@ -250,6 +251,7 @@ public function testGetForm(array $expected) {
 		// The form
 		$form = new Form();
 		$form->setId(42);
+		$form->setState(0); // default => 0 means active
 		$form->setHash('abcdefg');
 		$form->setTitle('Form 1');
 		$form->setDescription('Description Text');
@@ -341,13 +343,14 @@ public function dataGetPartialForm() {
 		return [
 			'onePartialOwnedForm' => [[
 				'id' => 42,
+				'state' => 0,
 				'hash' => 'abcdefg',
 				'title' => 'Form 1',
 				'expires' => 0,
 				'lastUpdated' => 123456789,
 				'permissions' => Constants::PERMISSION_ALL,
 				'submissionCount' => 123,
-				'partial' => true
+				'partial' => true,
 			]]
 		];
 	}
@@ -358,6 +361,7 @@ public function dataGetPartialForm() {
 	 */
 	public function testGetPartialForm(array $expected) {
 		$form = new Form();
+		$form->setState(0);
 		$form->setId(42);
 		$form->setHash('abcdefg');
 		$form->setTitle('Form 1');
@@ -384,7 +388,8 @@ public function dataGetPartialFormShared() {
 				'lastUpdated' => 123456789,
 				'permissions' => ['results', 'submit'],
 				'submissionCount' => 123,
-				'partial' => true
+				'state' => 0,
+				'partial' => true,
 			]]
 		];
 	}
@@ -397,6 +402,7 @@ public function dataGetPartialFormShared() {
 	public function testGetPartialFormShared(array $expected) {
 		$form = new Form();
 		$form->setId(42);
+		$form->setState(0);
 		$form->setHash('abcdefg');
 		$form->setTitle('Form 1');
 		$form->setOwnerId('otherUser');
@@ -428,6 +434,7 @@ public function dataGetPublicForm() {
 			// Bare form without questions, checking removed access & ownerId
 			'one-full-form' => [[
 				'id' => 42,
+				'state' => 0,
 				'hash' => 'abcdefg',
 				'title' => 'Form 1',
 				'description' => 'Description Text',
@@ -455,6 +462,7 @@ public function testGetPublicForm(array $expected) {
 		// The form
 		$form = new Form();
 		$form->setId(42);
+		$form->setState(0);
 		$form->setHash('abcdefg');
 		$form->setTitle('Form 1');
 		$form->setDescription('Description Text');