test: guarantee the logo-less member dialog case through the e2e overlay

[hivecommons-hive]

Test Improvement

tests/e2e/member-directory.spec.js holds the only browser coverage of the
member profile dialog's logo-less arm
(src/components/MemberDirectory/MemberProfile.js:35-41). It selected the
member driving that case by reading data/members.json off disk and
test.skipping when every member carried a logo.

That file is regenerated from the CNCF landscape by npm run generate:members,
so the condition is upstream state rather than a source edit: at main =
7ab301e only two of 101 members lack a logo (DiDi,
Subaru Corporation). A landscape refresh that gives both of them one retires
the case as a skip — and no gate reports a skip, so a green suite would
stop covering a shipped UI arm without saying so.

This PR reads the members through loadSiteData('members.json') from
tests/tools/e2e-data-fixtures.cjs, as tests/e2e/data-fixtures.spec.js and
tests/e2e/data-variants.spec.js already do. The committed overlay in
tests/e2e/fixtures/data/members.json appends coverage-fixture-org with
"logo": null, so under E2E_COVERAGE=1 a logo-less member is a property of
the build; its absence is then asserted as a broken fixture rather than
skipped. The plain End-to-end tests job applies no overlay
(loadSiteData returns the file unchanged there) and keeps the existing skip,
because in that job the case really does depend on upstream data.

No behaviour of the dialog is restated: the case still compares the dialog's
large initials against the card's server-rendered ones, and a real logo-less
member is still preferred when the data has one.

Verification

• npm run test:unit at this head: 1966 passed, 0 failed (node v26.10.0,
  TZ=UTC).
• npx prettier --check tests/e2e/member-directory.spec.js: clean.
• npx playwright test --list tests/e2e/member-directory.spec.js: all six
  cases enumerate.
• Overlay behaviour checked directly at 7ab301e:
  loadSiteData('members.json', {}) → 101 members, 2 without a logo;
  loadSiteData('members.json', {E2E_COVERAGE: '1'}) → 102 members, 3 without
  a logo, the third being Coverage Fixture Organization.
• The browser run itself was not executed locally: this environment has no
  libglib-2.0.so.0 and no root to install Playwright's system dependencies,
  so chrome-headless-shell exits 127 before any spec runs. The
  End-to-end coverage job is the check that exercises it.

Files claimed

tests/e2e/member-directory.spec.js only. Disjoint from
tests/e2e/fixtures/data-variants/members.json (#1104),
tests/tools/e2e-data-fixtures.cjs and tests/e2e-data-fixtures.test.mjs
(#1110, #1113), tests/e2e/dialog-non-dismissing.spec.js (#1120), and every
workflow file.

Related Issue

Closes #1148

— hive: agent=quality backend=copilot model=claude-opus-5 copilot=1.0.88

[hivecommons-hive]

Important

Held for human review by the hive's ACMM level gate.

This PR was opened by the "quality" agent while Hive policy required a human checkpoint for that agent. Non-outreach agents are held at ACMM L3–L5; the outreach agent is always held because it publishes project-facing communication.

Hive will keep the hold label until a human removes it. Operators can make a deliberate one-off release during an ACMM level change with release_level_holds=true, but level changes never release this hold automatically.