@hmcts/media-viewer 4.2.33 → 4.2.34

package/README.md

@@ -116,10 +116,11 @@ For browser-level proof that the standalone viewer is using AAT-backed services,
 yarn test:local:aat
 ```
 
-This opens `http://localhost:3000/#/media-viewer`, loads
-`/documents/<MV_SMOKE_PDF_DOCUMENT_ID>/binary`, and waits for the rendered PDF viewer
-and first page. If `MV_SMOKE_PDF_DOCUMENT_ID` is blank, the smoke uses the demo app's
-default AAT PDF document id.
+This executes the Functional Playwright contracts against the local standalone Media
+Viewer and AAT-backed proxy configuration while keeping the browser at
+`http://localhost:3000/`. It does not create CCD cases or DM Store documents; use
+the opt-in external-service contracts for those live service probes. The lightweight
+route and health check remains available as `yarn smoke:local:aat`.
 
 ### 5. Run Playwright tests
 Media Viewer is starting its Playwright migration with the same runner and
@@ -132,17 +133,17 @@ Current Playwright lanes:
 | Lane | Config/project | Command | Scope |
 | --- | --- | --- | --- |
 | Standalone smoke | `playwright.config.ts`, project `smoke` | `yarn test:playwright:smoke` or `yarn test:smoke` | One readiness contract: loads a standalone PDF and proves the rendered viewer, first page and canvas are usable. |
-| Migrated functional | `playwright.config.ts`, project `functional` | `yarn test:playwright:functional` | 56 behaviour tests across 12 explicit feature files; one bookmarks reorder test is intentionally skipped pending [EXUI-5097](https://tools.hmcts.net/jira/browse/EXUI-5097). See [`playwright_tests/functional/README.md`](playwright_tests/functional/README.md). |
+| Migrated functional | `playwright.config.ts`, project `functional` | `yarn test:playwright:functional` | 74 fixture-backed browser contracts across 13 feature files, including separate failed PDF/image rendered-state diagnostics. Two additional image-annotation create contracts are discoverable, ticketed against [EXUI-5124](https://tools.hmcts.net/jira/browse/EXUI-5124), and excluded from the default selection because the current product does not persist an image draw-box annotation. See [`playwright_tests/functional/README.md`](playwright_tests/functional/README.md). |
+| External service diagnostics | `playwright.config.ts`, opt-in project `external-service-contracts` | `yarn test:playwright:external-service-contracts` | Optional live AAT CCD/DM Store/annotation probes for a deliberate environment investigation. The default command executes 6 non-defect service contracts; four CCD browser-route contracts tagged against [EXUI-5122](https://tools.hmcts.net/jira/browse/EXUI-5122) and [EXUI-5123](https://tools.hmcts.net/jira/browse/EXUI-5123) remain discoverable but are excluded by default. Use `PLAYWRIGHT_INCLUDE_KNOWN_DEFECTS=true` to discover and execute all 10. They are never part of normal PR assurance. |
 | Viewer support | `playwright.config.ts`, project `support` | `yarn test:playwright:support` | Proves the reusable PDF, image and unsupported-media fixtures, component objects and response diagnostics. |
 
 The current migration slice is deliberately separated from smoke: smoke proves
-the application is ready, functional proves user-facing viewer behaviour, and
-support proves the reusable automation contracts. There is no API or mocked
-integration project in this slice because the migrated contracts are browser
-rendering and interaction behaviour owned by the standalone viewer. Every
-Playwright Odhín report also includes a capability inventory showing covered,
-partially covered and legacy-only Media Viewer areas, with the remaining
-assurance gap for each capability.
+the application is ready, functional proves user-facing viewer behaviour and
+its deterministic Viewer/service request-response contracts, and support proves
+the reusable automation contracts. External service diagnostics
+are opt-in and never gate a standalone Media Viewer change. Every Playwright Odhín report includes a capability
+inventory that states whether each contract is selected by default or is a
+discoverable, ticketed product-defect contract.
 
 The Playwright config runs tests fully in parallel with seven workers by
 default. Set `FUNCTIONAL_TESTS_WORKERS` to a positive integer (up to 64) for an
@@ -261,8 +262,10 @@ Migration boundaries:
 - Put new native Playwright specs under `playwright_tests/`.
 - Keep screen interactions and reusable locators in page objects under
   `playwright_tests/pages/`; keep assertions visible in specs.
-- Keep legacy Protractor and CodeceptJS coverage until replacement coverage and
-  Jenkins evidence are agreed.
+- Historical CodeceptJS scenarios are retained as source traceability only;
+  their executable pipeline routing is retired once the mapped Playwright
+  contract is selected by default or is represented by a discoverable,
+  ticketed product-defect contract.
 - Add stable report output paths for every new Playwright lane so Jenkins can
   publish Odhín and JUnit and archive failure diagnostics without bespoke stage
   logic.
@@ -270,66 +273,25 @@ Migration boundaries:
   an error page, blank page, wrong route or service-down page as a valid ready
   signal.
 
-### 6. Create isolated AAT test documents
-For mutation-heavy functional tests, do not share one document across parallel tests.
-Create fresh AAT DM Store documents through the local API proxy while `yarn start:aat`
-is running:
+### 6. Run local Playwright lanes
+`yarn test:local:aat` is the standalone smoke lane; it does not create a CCD
+case or DM Store document. Run the fixture-backed Functional lane locally with:
 
 ```
-yarn local-aat:documents -- --pdf-count 7 --image-count 1 --output .local-aat-documents.env
+yarn test:playwright:functional
 ```
 
-This writes:
-- `MV_SMOKE_PDF_DOCUMENT_ID`
-- `MV_SMOKE_IMAGE_DOCUMENT_ID`
-- `MV_FUNCTIONAL_PDF_DOCUMENT_IDS`
-- `MV_FUNCTIONAL_IMAGE_DOCUMENT_IDS`
+The default local suite is standalone: it needs neither AAT credentials nor
+shared CCD, DM Store or annotation-service state. External service diagnostics
+are deliberately opt-in and must not be used as normal migration assurance. The
+external command runs 6 contracts by default; `PLAYWRIGHT_INCLUDE_KNOWN_DEFECTS=true`
+runs the full 10-contract inventory, including the four ticketed CCD browser defects.
+Known product-defect contracts are not skipped: they remain discoverable and
+run only when explicitly selected:
 
-The upload path mirrors em-showcase: multipart `files`, `classification=PUBLIC`,
-and civil/probate metadata are posted to `/documents`.
-
-### 7. Run isolated local functional tests
-With `yarn start:aat` still running, execute the functional groups with separate
-documents and separate reports:
-
-```
-yarn test:functional:local:isolated
-```
-
-By default this creates fresh documents, runs up to three feature files at a time,
-and writes reports under `functional-output/local-isolated/`. Override with:
-- `MV_LOCAL_PARALLEL_MAX_JOBS=1` to run the same isolated groups serially
-- `MV_CREATE_LOCAL_AAT_DOCS=false` to reuse IDs from `.local-aat-documents.env`
-- `E2E_PARALLEL_OUTPUT_ROOT=<path>` to change report location
-
-Each feature writes its own report directory, including:
-- `mv-e2e-result.html`
-- `mv-e2e-result.json`
-- `result.xml`
-
-The validated local AAT sequence is:
-
-```
-yarn check:aat-config
-yarn start:aat
-yarn smoke:local:aat
-yarn test:local:aat
-yarn test:functional:local:isolated
 ```
-
-Expected `test:functional:local:isolated` feature groups:
-- `redact`
-- `imageViewerAnnotationsAndComments`
-
-For the strongest isolation proof, make each scenario upload and use its own document:
-
+PLAYWRIGHT_INCLUDE_KNOWN_DEFECTS=true yarn test:playwright:functional -- --grep @defect-EXUI-5124
 ```
-yarn test:functional:local:self-contained
-```
-
-This is slower because it creates a fresh AAT DM Store document for each scenario, but
-it is the best local check when diagnosing interference between bookmarks,
-annotations, comments, and redactions.
 
 ### Useful overrides
 Most developers should use the defaults from `.env.example`. Override only when you are deliberately testing a different endpoint or registered client setting.

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@hmcts/media-viewer",
-  "version": "4.2.33",
+  "version": "4.2.34",
   "description": "Media Viewer",
   "keywords": [
     "Angular",