CCM-13478: Initial commit with various improvements

Author: ScottFullerton-NHSE

.github/actions/build-docs/action.yml

@@ -27,8 +27,8 @@ runs:
       uses: ruby/[email protected]
       with:
         ruby-version: "3.4.7" # Not needed with a .ruby-version file
-        bundler-cache: false # runs 'bundle install' and caches installed gems automatically
-        #cache-version: 0 # Increment this number if you need to re-download cached gems
+        bundler-cache: true # Enable automatic gem caching (saves ~60-90s per build)
+        cache-version: 0 # Increment this number if you need to re-download cached gems
         working-directory: "./docs"
     - name: Setup Pages
       id: pages

.github/actions/node-install/action.yaml

@@ -16,6 +16,8 @@ runs:
       uses: actions/setup-node@v6
       with:
         node-version: '${{ inputs.node-version }}'
+        cache: 'npm'
+        cache-dependency-path: '**/package-lock.json'
 
     - name: "Configure npm for GitHub Packages"
       shell: bash

.github/workflows/stage-2-test.yaml

@@ -80,6 +80,8 @@ jobs:
         uses: actions/setup-python@v6
         with:
           python-version: ${{ inputs.python_version }}
+          cache: 'pip'
+          cache-dependency-path: '**/requirements*.txt'
       - name: "Run unit test suite"
         run: |
           make test-unit
@@ -117,6 +119,8 @@ jobs:
         uses: actions/setup-python@v6
         with:
           python-version: ${{ inputs.python_version }}
+          cache: 'pip'
+          cache-dependency-path: '**/requirements*.txt'
       - uses: ./.github/actions/node-install
         with:
           node-version: ${{ inputs.nodejs_version }}
@@ -143,22 +147,6 @@ jobs:
       - name: "Run typecheck"
         run: |
           make test-typecheck
-  test-coverage:
-    name: "Test coverage"
-    needs: [test-unit]
-    runs-on: ubuntu-latest
-    timeout-minutes: 5
-    permissions:
-      contents: read
-    steps:
-      - name: "Checkout code"
-        uses: actions/checkout@v5
-      - name: "Run test coverage check"
-        run: |
-          make test-coverage
-      - name: "Save the coverage check result"
-        run: |
-          echo "Nothing to save"
   perform-static-analysis:
     name: "Perform static analysis"
     needs: [test-unit]

.github/workflows/stage-3-build.yaml

@@ -33,8 +33,42 @@ on:
         type: string
 
 jobs:
+  detect-doc-changes:
+    name: "Detect Documentation Changes"
+    runs-on: ubuntu-latest
+    outputs:
+      docs_changed: ${{ steps.filter.outputs.docs }}
+      schemas_changed: ${{ steps.filter.outputs.schemas }}
+      eventcatalog_changed: ${{ steps.filter.outputs.eventcatalog }}
+    steps:
+      - name: "Checkout code"
+        uses: actions/checkout@v5
+
+      - name: "Check for relevant changes"
+        uses: dorny/paths-filter@v3
+        id: filter
+        with:
+          filters: |
+            docs:
+              - 'docs/**'
+              - '**.md'
+            schemas:
+              - 'schemas/**'
+              - 'src/cloudevents/**'
+              - 'src/cloudeventjekylldocs/**'
+            eventcatalog:
+              - 'src/eventcatalog/**'
+              - 'src/asyncapigenerator/**'
+              - 'src/eventcatalogasyncapiimporter/**'
+
   artefact-jekyll-docs:
     name: "Build Docs"
+    needs: [detect-doc-changes]
+    # Only build docs if relevant files changed
+    if: |
+      needs.detect-doc-changes.outputs.docs_changed == 'true' ||
+      needs.detect-doc-changes.outputs.schemas_changed == 'true' ||
+      needs.detect-doc-changes.outputs.eventcatalog_changed == 'true'
     runs-on: ubuntu-latest
     timeout-minutes: 15
     steps:

tests/playwright/digital-letters-component-tests/core-notify.component.spec.ts

@@ -35,6 +35,8 @@ const baseEvent: Omit<PDMResourceAvailable, 'id' | 'data'> = {
 };
 
 test.describe('Digital Letters - Core Notify', () => {
+  test.describe.configure({ mode: 'parallel' });
+
   test.beforeAll(async () => {
     await purgeQueue(CORE_NOTIFIER_DLQ_NAME);
     test.setTimeout(250_000);

tests/playwright/digital-letters-component-tests/pdm-poll.component.spec.ts

@@ -41,6 +41,8 @@ const unavailableEvent = {
 };
 
 test.describe('PDM Poll', () => {
+  test.describe.configure({ mode: 'parallel' });
+
   test.beforeAll(async () => {
     test.setTimeout(250_000);
     await purgeQueue(PDM_POLL_DLQ_NAME);

tests/playwright/digital-letters-component-tests/pdm-uploader.component.spec.ts

@@ -51,6 +51,8 @@ const baseEvent = {
 };
 
 test.describe('Digital Letters - Upload to PDM', () => {
+  test.describe.configure({ mode: 'parallel' });
+
   test.beforeAll(async () => {
     await purgeQueue(PDM_UPLOADER_DLQ_NAME);
     test.setTimeout(250_000);