feat(cb2-8066): add permissions for get tech records v2 (#79)

dvsa · May 22, 2023 · d490a25 · d490a25
1 parent 1b50f1a
commit d490a25
Show file tree

Hide file tree

Showing 2 changed files with 14 additions and 6 deletions.
diff --git a/src/functions/functionalConfig.ts b/src/functions/functionalConfig.ts
@@ -25,6 +25,10 @@ export const functionConfig: { [key: string]: NonEmptyArray<IApiAccess> } = {
  verbs: ["GET", "OPTIONS"],
  path: "vehicles/*",
  },
+ {
+ verbs: ["GET", "OPTIONS"],
+ path: "v2/vehicles/*",
+ },
  {
  verbs: ["GET", "OPTIONS"],
  path: "reference/*",
@@ -35,6 +39,10 @@ export const functionConfig: { [key: string]: NonEmptyArray<IApiAccess> } = {
  verbs: ["GET", "OPTIONS"],
  path: "vehicles/*",
  },
+ {
+ verbs: ["GET", "OPTIONS"],
+ path: "v2/vehicles/*",
+ },
  ],
  "TechRecord.Archive": [
  {
@@ -142,12 +150,12 @@ export const functionConfig: { [key: string]: NonEmptyArray<IApiAccess> } = {
  {
  verbs: ["GET", "OPTIONS"],
  path: "reference/*",
- }
+ },
  ],
  "ReferenceData.Amend": [
  {
  verbs: ["GET", "OPTIONS", "PUT", "POST", "DELETE"],
  path: "reference/*",
- }
+ },
  ],
 };
diff --git a/tests/unit/functions/authoriser.unitTest.ts b/tests/unit/functions/authoriser.unitTest.ts
@@ -20,7 +20,7 @@ describe("authorizer() unit tests", () => {
  });
 
  it("should fail on non-2xx HTTP status", async () => {
- (getValidJwt as jest.Mock) = jest.fn().mockRejectedValue({ statusCode: 418, body: "I'm a teapot", options: { url: "http://example.org" }, response: {} as IncomingMessage});
+ (getValidJwt as jest.Mock) = jest.fn().mockRejectedValue({ statusCode: 418, body: "I'm a teapot", options: { url: "http://example.org" }, response: {} as IncomingMessage });
 
  await expectUnauthorised(event);
  });
@@ -107,7 +107,7 @@ describe("authorizer() unit tests", () => {
 
  expect(returnValue.principalId).toEqual(jwtJson.payload.sub);
 
- expect(returnValue.policyDocument.Statement.length).toEqual(2);
+ expect(returnValue.policyDocument.Statement.length).toEqual(4);
  expect(returnValue.policyDocument.Statement).toContainEqual({
  Effect: "Allow",
  Action: "execute-api:Invoke",
@@ -122,7 +122,7 @@ describe("authorizer() unit tests", () => {
  const returnValue: APIGatewayAuthorizerResult = await authorizer(event, exampleContext());
 
  expect(returnValue.principalId).toEqual(jwtJson.payload.sub);
- expect(returnValue.policyDocument.Statement.length).toEqual(6);
+ expect(returnValue.policyDocument.Statement.length).toEqual(8);
  });
 
  it("should return an accurate policy based on functional roles", async () => {
@@ -131,7 +131,7 @@ describe("authorizer() unit tests", () => {
  const returnValue: APIGatewayAuthorizerResult = await authorizer(event, exampleContext());
 
  expect(returnValue.principalId).toEqual(jwtJson.payload.sub);
- expect(returnValue.policyDocument.Statement.length).toEqual(6);
+ expect(returnValue.policyDocument.Statement.length).toEqual(8);
 
  const post: { Action: string; Effect: string; Resource: string } = returnValue.policyDocument.Statement[0] as unknown as { Action: string; Effect: string; Resource: string };
  expect(post.Effect).toEqual("Allow");