Skip to content

Commit

Permalink
name and unit test updated
Browse files Browse the repository at this point in the history
  • Loading branch information
atedros-ch committed Feb 19, 2024
1 parent 14a57d8 commit bdfeb58
Show file tree
Hide file tree
Showing 2 changed files with 11 additions and 11 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -3,16 +3,16 @@ import { validationResult } from "express-validator";
import * as config from "../../../config";
import { FormattedValidationErrors, formatValidationError } from "../../../validation/validation";
import { selectLang, addLangToUrl, getLocalesService, getLocaleInfo } from "../../../utils/localise";
import { SOLE_TRADER_SECTOR_YOU_WORK_IN, LIMITED_NAME_REGISTERED_WITH_AML, LIMITED_WHAT_IS_YOUR_ROLE } from "../../../types/pageURL";
import { SOLE_TRADER_SECTOR_YOU_WORK_IN, LIMITED_NAME_REGISTERED_WITH_AML, LIMITED_WHAT_IS_YOUR_ROLE, BASE_URL } from "../../../types/pageURL";

export const get = async (req: Request, res: Response, next: NextFunction) => {
const lang = selectLang(req.query.lang);
const locales = getLocalesService();
res.render(config.LIMITED_NAME_REGISTERED_WITH_AML, {
previousPage: addLangToUrl(LIMITED_WHAT_IS_YOUR_ROLE, lang),
title: "Which sector do you work in?",
previousPage: addLangToUrl(BASE_URL + LIMITED_WHAT_IS_YOUR_ROLE, lang),
title: "Which name is registered with your AML supervisory body?",
...getLocaleInfo(locales, lang),
currentUrl: LIMITED_NAME_REGISTERED_WITH_AML
currentUrl: BASE_URL + LIMITED_NAME_REGISTERED_WITH_AML
});
};

Expand All @@ -24,14 +24,14 @@ export const post = async (req: Request, res: Response, next: NextFunction) => {
if (!errorList.isEmpty()) {
const pageProperties = getPageProperties(formatValidationError(errorList.array(), lang));
res.status(400).render(config.LIMITED_NAME_REGISTERED_WITH_AML, {
previousPage: addLangToUrl(LIMITED_WHAT_IS_YOUR_ROLE, lang),
previousPage: addLangToUrl(BASE_URL + LIMITED_WHAT_IS_YOUR_ROLE, lang),
title: "Which name is registered with your AML supervisory body?",
...getLocaleInfo(locales, lang),
currentUrl: LIMITED_NAME_REGISTERED_WITH_AML,
currentUrl: BASE_URL + LIMITED_NAME_REGISTERED_WITH_AML,
...pageProperties
});
} else {
const nextPageUrl = addLangToUrl(SOLE_TRADER_SECTOR_YOU_WORK_IN, lang);
const nextPageUrl = addLangToUrl(BASE_URL + SOLE_TRADER_SECTOR_YOU_WORK_IN, lang);
res.redirect(nextPageUrl);
}
} catch (error) {
Expand Down
Original file line number Diff line number Diff line change
@@ -1,23 +1,23 @@
import supertest from "supertest";
import app from "../../../main/app";
import { LIMITED_NAME_REGISTERED_WITH_AML } from "../../../main/types/pageURL";
import { LIMITED_NAME_REGISTERED_WITH_AML, BASE_URL } from "../../../main/types/pageURL";
const router = supertest(app);

describe("GET" + LIMITED_NAME_REGISTERED_WITH_AML, () => {
it("should return status 200", async () => {
await router.get(LIMITED_NAME_REGISTERED_WITH_AML).expect(200);
await router.get(BASE_URL + LIMITED_NAME_REGISTERED_WITH_AML).expect(200);
});
});

// Test when radio btn selected, will return 302 after redirecting to the next page.
describe("POST" + LIMITED_NAME_REGISTERED_WITH_AML, () => {
it("should return status 302 after redirect", async () => {
await router.post(LIMITED_NAME_REGISTERED_WITH_AML).send({ nameRegisteredWithAml: "NAME_OF_THE_BUSINESS" }).expect(302);
await router.post(BASE_URL + LIMITED_NAME_REGISTERED_WITH_AML).send({ nameRegisteredWithAml: "NAME_OF_THE_BUSINESS" }).expect(302);
});
});
// Test for no radio btn value selected, will return 400.
describe("POST" + LIMITED_NAME_REGISTERED_WITH_AML, () => {
it("should return status 400 after incorrect data entered", async () => {
await router.post(LIMITED_NAME_REGISTERED_WITH_AML).send({ nameRegisteredWithAml: "" }).expect(400);
await router.post(BASE_URL + LIMITED_NAME_REGISTERED_WITH_AML).send({ nameRegisteredWithAml: "" }).expect(400);
});
});

0 comments on commit bdfeb58

Please sign in to comment.