From 375a8f48425ab669c63dcad4aa33fb40e8456ebb Mon Sep 17 00:00:00 2001
From: wangsijie <wangsijie@silverhand.io>
Date: Wed, 11 Jan 2023 16:13:35 +0800
Subject: [PATCH] test: add IT for scopes (#2895)

---
 packages/integration-tests/src/api/scope.ts   | 38 +++++++++
 .../src/tests/api/resource.scope.test.ts      | 77 +++++++++++++++++++
 packages/integration-tests/src/utils.ts       |  1 +
 3 files changed, 116 insertions(+)
 create mode 100644 packages/integration-tests/src/api/scope.ts
 create mode 100644 packages/integration-tests/src/tests/api/resource.scope.test.ts

diff --git a/packages/integration-tests/src/api/scope.ts b/packages/integration-tests/src/api/scope.ts
new file mode 100644
index 000000000..6116ac12b
--- /dev/null
+++ b/packages/integration-tests/src/api/scope.ts
@@ -0,0 +1,38 @@
+import type { Scope, CreateScope } from '@logto/schemas';
+import type { OptionsOfTextResponseBody } from 'got';
+
+import { generateScopeName } from '#src/utils.js';
+
+import { authedAdminApi } from './api.js';
+
+export const getScopes = (resourceId: string, options?: OptionsOfTextResponseBody) =>
+  authedAdminApi.get(`resources/${resourceId}/scopes`, options).json<Scope[]>();
+
+export const createScope = (resourceId: string, name?: string) => {
+  const scopeName = name ?? generateScopeName();
+
+  return authedAdminApi
+    .post(`resources/${resourceId}/scopes`, {
+      json: {
+        name: scopeName,
+        description: scopeName,
+      },
+    })
+    .json<Scope>();
+};
+
+export const updateScope = (
+  resourceId: string,
+  scopeId: string,
+  payload: Partial<Omit<CreateScope, 'id' | 'resourceId'>>
+) =>
+  authedAdminApi
+    .patch(`resources/${resourceId}/scopes/${scopeId}`, {
+      json: {
+        ...payload,
+      },
+    })
+    .json<Scope>();
+
+export const deleteScope = (resourceId: string, scopeId: string) =>
+  authedAdminApi.delete(`resources/${resourceId}/scopes/${scopeId}`);
diff --git a/packages/integration-tests/src/tests/api/resource.scope.test.ts b/packages/integration-tests/src/tests/api/resource.scope.test.ts
new file mode 100644
index 000000000..938d6a070
--- /dev/null
+++ b/packages/integration-tests/src/tests/api/resource.scope.test.ts
@@ -0,0 +1,77 @@
+import { managementResource, managementResourceScope } from '@logto/schemas';
+import { HTTPError } from 'got';
+
+import { createResource } from '#src/api/index.js';
+import { createScope, deleteScope, getScopes, updateScope } from '#src/api/scope.js';
+import { generateScopeName } from '#src/utils.js';
+
+describe('admin console api resources', () => {
+  it('should get management api resource scopes successfully', async () => {
+    const scopes = await getScopes(managementResource.id);
+
+    expect(scopes[0]).toMatchObject(managementResourceScope);
+  });
+
+  it('should create scope successfully', async () => {
+    const resource = await createResource();
+    const scopeName = generateScopeName();
+    const createdScope = await createScope(resource.id, scopeName);
+
+    expect(createdScope.name).toBe(scopeName);
+
+    const scopes = await getScopes(resource.id);
+
+    expect(scopes.some(({ name }) => name === scopeName)).toBeTruthy();
+  });
+
+  it('should fail when create scope with name conflict', async () => {
+    const resource = await createResource();
+    const createdScope = await createScope(resource.id);
+    const response = await createScope(resource.id, createdScope.name).catch(
+      (error: unknown) => error
+    );
+    expect(response instanceof HTTPError && response.response.statusCode === 422).toBe(true);
+  });
+
+  it('should update scope successfully', async () => {
+    const resource = await createResource();
+    const scope = await createScope(resource.id);
+
+    expect(scope).toBeTruthy();
+
+    const newScopeName = `new_${scope.name}`;
+    const newScopeDescription = `new_${scope.description ?? ''}`;
+
+    const updatesScope = await updateScope(resource.id, scope.id, {
+      name: newScopeName,
+      description: newScopeDescription,
+    });
+
+    expect(updatesScope.id).toBe(scope.id);
+    expect(updatesScope.name).toBe(newScopeName);
+    expect(updatesScope.description).toBe(newScopeDescription);
+  });
+
+  it('should fail when update scope with name conflict', async () => {
+    const resource = await createResource();
+    const createdScope = await createScope(resource.id);
+    const createdScope2 = await createScope(resource.id);
+    const response = await updateScope(resource.id, createdScope2.id, {
+      name: createdScope.name,
+    }).catch((error: unknown) => error);
+    expect(response instanceof HTTPError && response.response.statusCode === 422).toBe(true);
+  });
+
+  it('should delete scope successfully', async () => {
+    const resource = await createResource();
+    const scope = await createScope(resource.id);
+
+    expect(scope).toBeTruthy();
+
+    await deleteScope(resource.id, scope.id);
+
+    const scopes = await getScopes(resource.id);
+
+    expect(scopes.some(({ name }) => name === scope.name)).toBeFalsy();
+  });
+});
diff --git a/packages/integration-tests/src/utils.ts b/packages/integration-tests/src/utils.ts
index c14acebd4..68e366749 100644
--- a/packages/integration-tests/src/utils.ts
+++ b/packages/integration-tests/src/utils.ts
@@ -6,6 +6,7 @@ export const generatePassword = () => `pwd_${crypto.randomUUID()}`;
 export const generateResourceName = () => `res_${crypto.randomUUID()}`;
 export const generateResourceIndicator = () => `https://${crypto.randomUUID()}.logto.io`;
 export const generateEmail = () => `${crypto.randomUUID().toLowerCase()}@logto.io`;
+export const generateScopeName = () => `sc:${crypto.randomUUID()}`;
 
 export const generatePhone = () => {
   const array = new Uint32Array(1);