Skip to content

task/FCT2-22065 Revise CICD pipelines in line with new branching strategy (Phase 1) - #249

Merged
lilachdavis merged 2 commits into
mainfrom
development
Sep 23, 2026
Merged

lilachdavis merged 2 commits into
mainfrom
development

Conversation

@lilachdavis

Copy link
Copy Markdown
Contributor

No description provided.

lilachdavis and others added 2 commits September 23, 2026 14:48
* tracking of events using journey id (#234)

* tracking of events using journey id

* pr comments

* Story/FCT2-22040 - Added a TelemetryLogger Service for ui (#233)

* added a TelemetryLogger Service for ui

* clean up and refactor and added unit tests

* updated the test

* Task/FCT2-19358 integrate e2e test suite into cicd (#237)

* telemetry tracking properties (#240)

* telemetry tracking properties

* pr comment fixes

* Story/FCT2-22041 : implemented ui tracking page views and few custom events (#239)

* implemented ui tracking page views and few custom events

* added journeyId and areaOrDivisionText for caseregistration request

* fixed the pull request comments

* minor refactor

* update pipelines with development branch triggers

* add src/tests directory to backend PR pipeline trigeer paths

* update parameter empty string value option

* story/FCT2-22092- Unauthorised page (#241)

* handled 401 error on initial page load

* added unit tests

* fixed the pr comments

* minor refactor

* test(e2e): assert the Repealed tag on offence search results (#245)

The tag itself shipped in #229; this covers it against the real
/api/v1/offences response, where previously only the MSW integration
suite asserted it.

Expectations come from the search response rather than hardcoded codes:
the offences are split into repealed (an "effective to" date is present)
and active, and the rendered table is checked against that split, so the
test holds as the reference data changes. It fails loudly if a search
returns only one kind, rather than passing vacuously.

Covers all three acceptance criteria - a tag for every offence with an
"effective to" date, no tag for any without one, and the tag sitting in
the Description column beneath the description text, carrying the GOV.UK
warning styling that distinguishes it.

Each was falsified before being kept: treating active offences as
repealed fails the tag count (expected 19, received 1), looking for the
tag in the Statute column fails on visibility, and expecting the
description without the tag appended fails the cell text.

* fix stage dependency

---------

Co-authored-by: Rhys Bridges <bridgesrhys@gmail.com>
Co-authored-by: Renjith Abby <renjithabby@gmail.com>
Co-authored-by: kmshrajcps <KameshRaj.Rajendran@cps.gov.uk>
@sonarqubecloud

sonarqubecloud Bot commented Sep 23, 2026 •

Copy link
Copy Markdown

@HasanCPS HasanCPS left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@lilachdavis
lilachdavis merged commit cd66561 into main Sep 23, 2026
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants