gsadmin/pulse
1
0
Fork 0
mirror of https://github.com/rcourtman/Pulse.git synced 2026-09-10 02:25:56 +00:00
Code Issues Packages Projects Releases Wiki Activity

Merge commit '2705098aac192074d44dd227bcfa0f8fd19c6211'

Browse Source 
Change-source: pulse-maintainer
This commit is contained in:
pulse-triage[bot]
2026-09-05 20:07:06 +01:00
parent 95a0faec93 2705098aac
commit ccbc543115
2 changed files with 15 additions and 9 deletions
Download Patch File Download Diff File Expand all files Collapse all files
frontend-modern/src/components/Settings/__tests__/settingsArchitecture.test.ts
+3 -1
View File
@@ -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(
frontend-modern/src/components/Settings/__tests__/settingsPanelRegistryContext.test.ts
+12 -8
View File
@@ -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('');
});