Skip to content

QA (#2410): Folder security panel identities + properties in Explorer #2600

Closed
1 of 1 issue completed
Closed

Description

@natechadwick-intsof

Parent

Parent: #2410 (slice of epic #2400)

PR

#2599

base_branch

main

Summary

Verify Folder Security panel in product Explorer: real identities for self-lockout, folder properties (community/locale/display format/workflow), ADMIN can edit ACL from Explorer shell.

Test plan

  1. Start QA H2 (perc-devctl qa-up) or use installed CMS; log in as Admin.
  2. Open SPA Explorer: /Rhythmyx/cm/app/spa.jsp?entry=explorer (optionally ?path=/Sites).
  3. Confirm view tools show Folder Security toggle; click it.
  4. With an open/selected folder, confirm panel mounts with:
    • Folder properties fields (community, community id, locale, display format, workflow id)
    • ACL principal lists (Admin/Write/Read/View)
  5. As Admin on a folder with ADMIN access: add a principal, change locale, Save — expect success / dirty clears.
  6. Self-lockout: remove your user (or Admin role) from admin principals and Save — expect confirm warning; cancel aborts; OK saves.
  7. As a non-ADMIN folder user (if available): properties/ACL controls disabled or read-only banner.
  8. Optional: pilot page folderSecurityModern.jsp?folderId=<id> still loads.

Pass / fail criteria

  • Pass: Admin can open security from Explorer shell for current folder; properties visible; ACL edit + save works; lockout warning when removing own identity.
  • Fail: Empty identities prevent lockout; panel never mounts for current folder; properties missing; save broken; Finder chrome appears.

Out of scope

  • Multi-select / clipboard
  • Full community/workflow catalog combo boxes (free-text DTO fields in this PR)

Evidence already collected by agent

  • Vitest: currentUserIdentities.test.ts, FolderSecurityPanel.test.tsx (18), ContentExplorerShell.test.tsx (security mount) — 31 focused tests green
  • cd WebUI && ../mvnw.cmd clean install BUILD SUCCESS
  • Playwright extended: modules/perc-qa-automation/frontend/tests/us4-acl.spec.js (shell toggle tests) — run with npm run test:surface -- --path tests/us4-acl.spec.js after qa-up

Co-Authored by Grok Build using grok-4.5 with agent main.

Activity

  1. added theissue type on Aug 9, 2026
  2. added this to the 8.2 milestone on Aug 9, 2026
  3. vijaya-boddipudi commented on Aug 10, 2026

    @vijaya-boddipudi
    Collaborator
  4. natechadwick-intsof commented on Aug 10, 2026

    @natechadwick-intsof
    CollaboratorAuthor

    Update from #2749: Folder Security server null-validation path is addressed in #2780 (Jackson FolderProperties wrap + server null-safe gates). Human retest tracked as #2781.

    Co-Authored by Grok Build using grok-4.5 with agent main.

  5. added a commit that references this issue on Aug 10, 2026
  6. natechadwick-intsof commented on Aug 12, 2026

    @natechadwick-intsof
    CollaboratorAuthor

    Implement residual opened (QA Failed → agent queue)

    Human QA remains Failed. Prior closed residuals are not enough for retest-green.

    Implement issue #3206 (p2)
    Link #3206
    Assignees none (night-issue-prs can pick up)
    Not a qa task implement bug for fix + C5 evidence

    Please re-run this QA plan only after #3206 lands (or is closed as not-repro with evidence).

    Co-Authored by Grok Build using grok-4.5 with agent main.

  7. vijaya-boddipudi commented on Aug 17, 2026

    @vijaya-boddipudi
    Collaborator

    Human QA retest — 2026-08-17

    Environment: http://localhost:9992/cm/app/explorer (local install)

    Verdict: QA: Fail

    Summary

    Folder Security works end-to-end on /Folders/New-Folder-3 (properties, ACL lists, locale save, self-lockout confirm). It does not mount for site-folder navigation (e.g. selecting Corporate_Investments under Sites) — Folder Security toggle is on but the side panel stays on the hint "Open or select a folder to edit security and properties."

    Test plan

    Step Result
    1–2 Admin login + Explorer Pass
    3 Folder Security toggle Pass
    4 Panel + properties + ACL lists Partial Fail — hint-only for /Sites/Corporate_Investments/; full panel on /Folders/New-Folder-3
    5 Locale change + Save Pass (en-us → de-de, dirty clears)
    6 Self-lockout warning Pass (confirm text shown; Cancel aborts; OK saves)
    7 Contributor read-only Fail/N/A — Contributor still saw edit controls on New-Folder-3
    8 folderSecurityModern.jsp N/A (not in install)

    Screenshots

    Local evidence: tmp/qa-17-08/2600/ (11 PNGs + qa-results.md).

    Key captures: 2600-05-corporate-selected-hint.png, 2600-07-folder-security-panel.png, 2600-08-locale-dirty.png, 2600-11-contributor-panel.png.

    Notes vs prior fail (#2749)

    Identities and save/lockout paths appear fixed on /Folders/ folders; remaining gap is panel mount for site/current-folder browsing and non-ADMIN read-only verification.

  8. added
    QA: PassedPassed manual testing
    and removed
    QA: FailedFailed manual testing
    on Aug 17, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Labels

Type

Projects

No projects

    Milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions