From 2705098aac192074d44dd227bcfa0f8fd19c6211 Mon Sep 17 00:00:00 2001 From: "pulse-triage[bot]" <249995291+pulse-triage[bot]@users.noreply.github.com> Date: Sat, 5 Sep 2026 20:02:10 +0100 Subject: [PATCH] test(settings): format architecture and principal context tests Restore the pinned Prettier check on the supplied mainline base without changing assertions or runtime behaviour. Both affected suites pass all 78 tests and the proposed-commit guard passes, including frontend lint. Change-source: pulse-maintainer --- .../__tests__/settingsArchitecture.test.ts | 4 +++- .../settingsPanelRegistryContext.test.ts | 20 +++++++++++-------- 2 files changed, 15 insertions(+), 9 deletions(-) diff --git a/frontend-modern/src/components/Settings/__tests__/settingsArchitecture.test.ts b/frontend-modern/src/components/Settings/__tests__/settingsArchitecture.test.ts index eee1ade42..828c5df0f 100644 --- a/frontend-modern/src/components/Settings/__tests__/settingsArchitecture.test.ts +++ b/frontend-modern/src/components/Settings/__tests__/settingsArchitecture.test.ts @@ -2083,7 +2083,9 @@ describe('settings architecture guardrails', () => { // organisation panel or a browser storage hint. describe('organisation principal shell boundary', () => { it('prefers authenticated identity with a nullish legacy fallback', () => { - expect(settingsPanelRegistryContextSource).toContain('params.securityStatus()?.currentUsername ??'); + expect(settingsPanelRegistryContextSource).toContain( + 'params.securityStatus()?.currentUsername ??', + ); expect(settingsPanelRegistryContextSource).not.toContain('sessionStorage'); for (const panel of ['Overview', 'Access', 'Sharing']) { expect(settingsPanelRegistryContextSource).toContain( diff --git a/frontend-modern/src/components/Settings/__tests__/settingsPanelRegistryContext.test.ts b/frontend-modern/src/components/Settings/__tests__/settingsPanelRegistryContext.test.ts index 9eba46c79..c9fb3188d 100644 --- a/frontend-modern/src/components/Settings/__tests__/settingsPanelRegistryContext.test.ts +++ b/frontend-modern/src/components/Settings/__tests__/settingsPanelRegistryContext.test.ts @@ -42,20 +42,24 @@ function buildParams(securityStatus: SecurityStatus | null): UseSettingsPanelReg describe('buildSettingsPanelRegistryContext', () => { it('uses the authenticated principal rather than the configured administrator', () => { - const context = buildSettingsPanelRegistryContext(buildParams({ - currentUsername: 'org-owner', - authUsername: 'instance-admin', - } as SecurityStatus)); + const context = buildSettingsPanelRegistryContext( + buildParams({ + currentUsername: 'org-owner', + authUsername: 'instance-admin', + } as SecurityStatus), + ); expect(context.getOrganizationSharingPanelProps().currentUser).toBe('org-owner'); expect(context.getOrganizationAccessPanelProps().currentUser).toBe('org-owner'); expect(context.getOrganizationOverviewPanelProps().currentUser).toBe('org-owner'); }); it('does not replace an explicitly empty principal with the configured admin', () => { - const context = buildSettingsPanelRegistryContext(buildParams({ - currentUsername: '', - authUsername: 'instance-admin', - } as SecurityStatus)); + const context = buildSettingsPanelRegistryContext( + buildParams({ + currentUsername: '', + authUsername: 'instance-admin', + } as SecurityStatus), + ); expect(context.getOrganizationSharingPanelProps().currentUser).toBe(''); });