Skip to content

Commit 6a54166

Browse files
remo5000ning-y
authored andcommitted
Modify mock role fetching (#176)
* Refactor role fetching No more mock tokens, only a mock role-fetching function that can be forced to return a certain role. * Format file
1 parent 78290f2 commit 6a54166

File tree

1 file changed

+5
-19
lines changed

1 file changed

+5
-19
lines changed

src/mocks/userAPI.ts

Lines changed: 5 additions & 19 deletions
Original file line numberDiff line numberDiff line change
@@ -5,30 +5,16 @@ export enum Roles {
55
}
66
export type Role = keyof typeof Roles
77

8-
/* Used to mock student access, i.e only attempt missions */
9-
export const MOCK_STUDENT_ACCESS_TOKEN = 'STUDENT_ACCESS_TOKEN'
10-
/* Used to mock trainer access, i.e only grade missions*/
11-
export const MOCK_TRAINER_ACCESS_TOKEN = 'TRAINER_ACCESS_TOKEN'
12-
/* Used to mock admin access */
13-
export const MOCK_ADMIN_ACCESS_TOKEN = 'ADMIN_ACCESS_TOKEN'
14-
158
/**
169
* Mock for fetching a role, given an access token. A null
17-
* value is returned for invalid tokens.
10+
* value is returned for invalid tokens. Fetching a particular
11+
* role can be simluated using an optional paramter.
1812
*
1913
* @param accessToken a valid access token for the cadet backend.
14+
* @param mockRole a role to mock retrieval for.
2015
*/
21-
export const mockFetchRole = (accessToken: string): Role | null => {
22-
switch (accessToken) {
23-
case MOCK_STUDENT_ACCESS_TOKEN:
24-
return Roles.student
25-
case MOCK_TRAINER_ACCESS_TOKEN:
26-
return Roles.trainer
27-
case MOCK_ADMIN_ACCESS_TOKEN:
28-
return Roles.admin
29-
default:
30-
return null
31-
}
16+
export const mockFetchRole = (accessToken: string, role: Role = Roles.trainer): Role | null => {
17+
return role
3218
}
3319

3420
/**

0 commit comments

Comments
 (0)