Improve other tests

code-q-web-factory · Oct 3, 2023 · b484824 · b484824
1 parent 353588d
commit b484824
Show file tree

Hide file tree

Showing 2 changed files with 35 additions and 2 deletions.
diff --git a/Tests/Unit/ZoomApiAccessTokenFactoryTest.php b/Tests/Unit/ZoomApiAccessTokenFactoryTest.php
@@ -47,7 +47,7 @@ public static function getInvalidConfigurations(): array
      * @dataProvider getInvalidConfigurations
      * @test
      */
-    public function invalidConfigurationWillThrowException($invalidConfiguration): void
+    public function invalidConfigurationWillThrowException(array $invalidConfiguration): void
     {
         $this->expectException(ZoomApiException::class);
         $this->expectExceptionMessage('Please set a Zoom Account ID, Client ID and Secret for CodeQ.ZoomApi to be able to authenticate.');
@@ -81,6 +81,39 @@ public function unsuccessfulRequestThrowsZoomApiException(): void
         $factoryMock->createFromConfiguration();
     }
 
+    public static function getInvalidAccessTokenScopes(): array
+    {
+        return [
+            ['recording:read:admin,meeting:read:admin'],
+            ['user:read:admin,meeting:read:admin'],
+            ['user:read:admin,recording:read:admin'],
+        ];
+    }
+
+    /**
+     * @test
+     * @dataProvider getInvalidAccessTokenScopes
+     * @param string $scopes
+     *
+     * @return void
+     */
+    public function missingAccessTokenScopesThrowsZoomApiException(string $scopes): void
+    {
+        $this->expectException(ZoomApiException::class);
+        $this->expectExceptionMessage('Please ensure your Zoom app has the following scopes: user:read:admin, recording:read:admin, meeting:read:admin');
+
+
+        $handlerStack = HandlerStack::create(
+            new MockHandler([
+                new Response(200, [], json_encode(['access_token' => '1234567890', 'scope' => $scopes]))
+            ])
+        );
+        $factoryMock = $this->getFactory($handlerStack);
+
+
+        $factoryMock->createFromConfiguration();
+    }
+
     private function getFactory(HandlerStack $handlerStack = null): ZoomApiAccessTokenFactory|MockObject
     {
         $factory = $this->getAccessibleMock(ZoomApiAccessTokenFactory::class, ['buildClient'], [], '', false);

diff --git a/Tests/Unit/ZoomApiServiceTest.php b/Tests/Unit/ZoomApiServiceTest.php
@@ -238,7 +238,7 @@ public function getUpcomingMeetingsWithEmptyResponseThrowsException()
     }
 
     /**
-     * @param Client $client
+     * @param HandlerStack|null $handlerStack
      *
      * @return ZoomApiService|MockObject
      */