Fix tests for listable conversations

Fix number detection in participantType tests

The guests app is now enabled.
The spreed app is whitelisted for guest account users.
A password is now set for the guest account users to be able to log in properly.

Added system message tests for listable and read-only attributes.

Added tests scenarios for permission checks and room type checks before
setting listable attribute.

Added integration tests for /listed-rooms endpoint
Covers permissions to search and also with and without search terms.

Signed-off-by: Vincent Petry <vincent@nextcloud.com>

2 files changed, 6 insertions, 0 deletions

diff --git a/tests/php/Controller/SignalingControllerTest.php b/tests/php/Controller/SignalingControllerTest.php
index 4f3644e14..2dc9453db 100644
--- a/tests/php/Controller/SignalingControllerTest.php
+++ b/tests/php/Controller/SignalingControllerTest.php
@@ -37,6 +37,7 @@ use OCA\Talk\Service\ParticipantService;
 use OCA\Talk\Service\SessionService;
 use OCA\Talk\Signaling\Messages;
 use OCA\Talk\TalkSession;
+use OCP\App\IAppManager;
 use OCP\AppFramework\Utility\ITimeFactory;
 use OCP\EventDispatcher\IEventDispatcher;
 use OCP\Http\Client\IClientService;
@@ -889,6 +890,7 @@ class SignalingControllerTest extends \Test\TestCase {
 			$dbConnection,
 			\OC::$server->getConfig(),
 			$this->createMock(Config::class),
+			\OC::$server->get(IAppManager::class),
 			\OC::$server->get(AttendeeMapper::class),
 			\OC::$server->get(SessionMapper::class),
 			$participantService,
diff --git a/tests/php/Signaling/BackendNotifierTest.php b/tests/php/Signaling/BackendNotifierTest.php
index bfc6e3d6e..57226b743 100644
--- a/tests/php/Signaling/BackendNotifierTest.php
+++ b/tests/php/Signaling/BackendNotifierTest.php
@@ -35,6 +35,7 @@ use OCA\Talk\Service\ParticipantService;
 use OCA\Talk\Signaling\BackendNotifier;
 use OCA\Talk\TalkSession;
 use OCA\Talk\Webinary;
+use OCP\App\IAppManager;
 use OCP\AppFramework\Utility\ITimeFactory;
 use OCP\EventDispatcher\IEventDispatcher;
 use OCP\Http\Client\IClientService;
@@ -141,6 +142,7 @@ class BackendNotifierTest extends \Test\TestCase {
 			$dbConnection,
 			$config,
 			$this->config,
+			\OC::$server->get(IAppManager::class),
 			\OC::$server->get(AttendeeMapper::class),
 			\OC::$server->get(SessionMapper::class),
 			$this->participantService,
@@ -313,6 +315,7 @@ class BackendNotifierTest extends \Test\TestCase {
 					'lobby-state' => Webinary::LOBBY_NONE,
 					'lobby-timer' => null,
 					'read-only' => Room::READ_WRITE,
+					'listable' => Room::LISTABLE_NONE,
 					'active-since' => null,
 					'sip-enabled' => 0,
 				],
@@ -434,6 +437,7 @@ class BackendNotifierTest extends \Test\TestCase {
 					'listable' => Room::LISTABLE_ALL,
 					'active-since' => null,
 					'sip-enabled' => 0,
+					'description' => '',
 				],
 			],
 		]);