Skip to content
Merged
Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
26 commits
Select commit Hold shift + click to select a range
481a336
Add technical planning for VIBE-241: Cookie Policy
github-actions[bot] Nov 21, 2025
c27d5b3
Add technical planning documentation for VIBE-241
github-actions[bot] Nov 21, 2025
1dbb803
Add technical planning for VIBE-241
github-actions[bot] Nov 22, 2025
a773f2d
Add technical planning documentation for VIBE-241
github-actions[bot] Nov 22, 2025
eca9b87
docs: Add technical plan for VIBE-241 - Cookie Policy
github-actions[bot] Nov 22, 2025
ce25bb1
Add technical planning for VIBE-241 Cookie Policy
github-actions[bot] Nov 22, 2025
355b26e
Add cookies policy page and content in English and Welsh
NatashaAlker Dec 1, 2025
f0ad35d
Update page footer
NatashaAlker Dec 1, 2025
1499340
Add analytic and performance cookies
NatashaAlker Dec 1, 2025
be7d512
Add tests
NatashaAlker Dec 1, 2025
e6d4f6b
Update task list
NatashaAlker Dec 1, 2025
ebefd22
Merge branch 'master' into feature/VIBE-241-cookie-policy
NatashaAlker Dec 1, 2025
7a7f24e
Claude's code review
NatashaAlker Dec 1, 2025
78b5c42
Merge branch 'feature/VIBE-241-cookie-policy' of github.com:hmcts/cat…
NatashaAlker Dec 1, 2025
a4be933
Fix some e2e test failures
NatashaAlker Dec 1, 2025
170c3d7
Fix faiing e2e test for cookie page table contents
NatashaAlker Dec 3, 2025
cd90152
Remove non functional welsh url and make language toggling consistent…
NatashaAlker Dec 3, 2025
9aba200
Fix failing e2e tests
NatashaAlker Dec 3, 2025
40f7c51
Update cookie-policy url and fix failing e2e test
NatashaAlker Dec 3, 2025
72a5bc3
Remove duplicate docs
NatashaAlker Dec 5, 2025
3635eff
Merge branch 'master' into feature/VIBE-241-cookie-policy
NatashaAlker Dec 5, 2025
b0a29d8
Fiz dependency scan issue
NatashaAlker Dec 5, 2025
6988194
refactor: consolidate cookie policy E2E tests into journey flows
ashwini-mv Dec 23, 2025
74e8f81
Merge master into feature/VIBE-241-cookie-policy
junaidiqbalmoj Dec 24, 2025
e7ba7da
Merge branch 'master' into feature/VIBE-241-cookie-policy
junaidiqbalmoj Dec 24, 2025
3daa3f4
fix end to end test
junaidiqbalmoj Dec 24, 2025
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Merge master into feature/VIBE-241-cookie-policy
Resolved conflicts:
- e2e-tests/tests/page-structure.spec.ts: Updated footer link tests to use /cookie-preferences instead of /cookie-policy
- yarn.lock: Regenerated to include latest dependencies

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>
  • Loading branch information
junaidiqbalmoj and claude committed Dec 24, 2025
commit 74e8f817b431edf30288eaec7d105f6e06546054
30 changes: 15 additions & 15 deletions e2e-tests/tests/page-structure.spec.ts
Original file line number Diff line number Diff line change
Expand Up @@ -74,34 +74,34 @@ test.describe("Page Structure - VIBE-149", () => {
await page.goto("/");

// AC8: All 8 footer links present
// Note: The footer has 8 links
const footerLinksOpenInSameTab = [
// Verify we have 8 footer meta links
const footerMetaLinks = page.locator(".govuk-footer__meta .govuk-footer__inline-list-item");
await expect(footerMetaLinks).toHaveCount(8);

// Links that should open in same tab
const sameTablLinks = [
{ text: "Help", href: "https://www.gov.uk/help" },
{ text: "Privacy policy", href: "https://www.gov.uk/help/privacy-notice" },
{ text: "Accessibility statement", href: "/accessibility-statement" },
{ text: "Cookies", href: "/cookie-preferences" },
{ text: "Contact us", href: "https://www.gov.uk/contact" },
{ text: "Terms and conditions", href: "https://www.gov.uk/help/terms-conditions" },
{ text: "Welsh", href: "https://www.gov.uk/cymraeg" },
{ text: "Government Digital Service", href: "https://www.gov.uk/government/organisations/government-digital-service" }
];

// Verify we have 8 footer meta links
const footerMetaLinks = page.locator(".govuk-footer__meta .govuk-footer__inline-list-item");
await expect(footerMetaLinks).toHaveCount(8);

// Verify links that open in same tab
for (const link of footerLinksOpenInSameTab) {
// Verify links that should open in same tab
for (const link of sameTablLinks) {
const footerLink = page.locator(`.govuk-footer__link[href="${link.href}"]`).first();
await expect(footerLink).toBeVisible({ timeout: 5000 });
const targetAttr = await footerLink.getAttribute("target");
expect(targetAttr).not.toBe("_blank");
}

// Verify Cookies link opens in new tab
const cookiesLink = page.locator('.govuk-footer__link[href="/cookie-policy"]').first();
await expect(cookiesLink).toBeVisible({ timeout: 5000 });
await expect(cookiesLink).toHaveAttribute("target", "_blank");
await expect(cookiesLink).toHaveAttribute("rel", "noopener noreferrer");
// Accessibility statement should open in new tab
const accessibilityLink = page.locator('.govuk-footer__link[href="/accessibility-statement"]').first();
await expect(accessibilityLink).toBeVisible({ timeout: 5000 });
await expect(accessibilityLink).toHaveAttribute("target", "_blank");
await expect(accessibilityLink).toHaveAttribute("rel", "noopener noreferrer");
});

test("should display Crown copyright link", async ({ page }) => {
Expand Down Expand Up @@ -160,7 +160,7 @@ test.describe("Page Structure - VIBE-149", () => {
await expect(signInLink).toHaveText("Mewngofnodi");

// Verify footer links are in Welsh
const cookiesLink = page.locator('.govuk-footer__link[href="/cookie-policy"]');
const cookiesLink = page.locator('.govuk-footer__link[href="/cookie-preferences"]');
await expect(cookiesLink).toHaveText("Cwcis");

// Click language toggle to switch back to English
Expand Down
41 changes: 39 additions & 2 deletions yarn.lock
Original file line number Diff line number Diff line change
Expand Up @@ -503,10 +503,17 @@ __metadata:
languageName: node
linkType: hard

"@biomejs/cli-linux-arm64@npm:2.3.10, @biomejs/cli-linux-arm64@npm:^2.3.6":
"@biomejs/cli-linux-arm64@npm:2.3.10":
version: 2.3.10
resolution: "@biomejs/cli-linux-arm64@npm:2.3.10"
checksum: 10c0/5467cf817b9384c713b64934e1a4c76e18f8a9cee7aec3d7d38b43982b65e3cb3a06820b1dd995ca55f85f64bf182bf76f8d8f9c85910aa96710e6ff0b479774
conditions: os=linux & cpu=arm64 & libc=glibc
languageName: node
linkType: hard

"@biomejs/cli-linux-arm64@npm:^2.3.6":
version: 2.3.8
resolution: "@biomejs/cli-linux-arm64@npm:2.3.8"
checksum: 10c0/1f10e8c05be93baf297ce77efb1e47e31008600f67af38886cb0c3a5b928d281c86d399fdb29caa5b0ae0216a68c7c49097c55af26e6e163c06f2b6969dfd7aa
conditions: os=linux & cpu=arm64 & libc=glibc
languageName: node
linkType: hard
Expand Down Expand Up @@ -5533,6 +5540,18 @@ __metadata:
languageName: node
linkType: hard

"jszip@npm:^3.10.1":
version: 3.10.1
resolution: "jszip@npm:3.10.1"
dependencies:
lie: "npm:~3.3.0"
pako: "npm:~1.0.2"
readable-stream: "npm:~2.3.6"
setimmediate: "npm:^1.0.5"
checksum: 10c0/58e01ec9c4960383fb8b38dd5f67b83ccc1ec215bf74c8a5b32f42b6e5fb79fada5176842a11409c4051b5b94275044851814a31076bf49e1be218d3ef57c863
languageName: node
linkType: hard

"jwa@npm:^1.4.2":
version: 1.4.2
resolution: "jwa@npm:1.4.2"
Expand Down Expand Up @@ -5568,6 +5587,24 @@ __metadata:
languageName: node
linkType: hard

"lazystream@npm:^1.0.0":
version: 1.0.1
resolution: "lazystream@npm:1.0.1"
dependencies:
readable-stream: "npm:^2.0.5"
checksum: 10c0/ea4e509a5226ecfcc303ba6782cc269be8867d372b9bcbd625c88955df1987ea1a20da4643bf9270336415a398d33531ebf0d5f0d393b9283dc7c98bfcbd7b69
languageName: node
linkType: hard

"lie@npm:~3.3.0":
version: 3.3.0
resolution: "lie@npm:3.3.0"
dependencies:
immediate: "npm:~3.0.5"
checksum: 10c0/56dd113091978f82f9dc5081769c6f3b947852ecf9feccaf83e14a123bc630c2301439ce6182521e5fbafbde88e88ac38314327a4e0493a1bea7e0699a7af808
languageName: node
linkType: hard

"limiter@npm:^1.1.5":
version: 1.1.5
resolution: "limiter@npm:1.1.5"
Expand Down
Loading
You are viewing a condensed version of this merge commit. You can view the full changes here.