fix(e2e): Refactor Sections to match new policy

Author: rogeruiz

client/e2e/configuration.spec.ts

@@ -652,6 +652,13 @@ test.describe('Sections Validation and Error Lifecycle', () => {
     makeAxeBuilder,
   }) => {
     const sectionRow = page.getByRole('row').filter({ hasText: 'Results' });
+
+    // Ensure the section is included so the narrative select is visible
+    const includeCheckbox = sectionRow.getByRole('checkbox');
+    if (!(await includeCheckbox.isChecked())) {
+      await includeCheckbox.click();
+    }
+
     const narrativeSelect = sectionRow.getByRole('combobox');
     const codedDataSwitch = sectionRow.getByRole('switch');
 
@@ -662,6 +669,10 @@ test.describe('Sections Validation and Error Lifecycle', () => {
     // 1. Trigger Error: Switch to 'Keep original'
     await expect(codedDataSwitch).toBeChecked();
     await codedDataSwitch.click();
+    const resultsCheckbox = sectionRow.getByRole('checkbox');
+    if (!(await resultsCheckbox.isChecked())) {
+      await resultsCheckbox.click();
+    }
     const errorAlert = sectionRow.getByRole('alert');
     await expect(errorAlert).toBeVisible();
     await expect(errorAlert).toHaveText(
@@ -680,10 +691,7 @@ test.describe('Sections Validation and Error Lifecycle', () => {
     await expect(errorAlert).toBeVisible();
 
     // 4. Persistence via Internal Click
-    await page
-      .locator('[data-error-trigger]')
-      .first()
-      .click({ position: { x: 20, y: 20 } });
+    await sectionRow.getByRole('switch').click({ position: { x: 20, y: 20 } });
     await expect(errorAlert).toBeVisible();
 
     // 5. Dismiss via Input Change