Skip to content

refactor(test-harness): remove parseDefaultValue and CEM-related logic from SSR renderer#7532

Merged
radium-v merged 2 commits into
mainfrom
users/radium-v/test-harness-remove-cem-workaround
May 22, 2026
Merged

refactor(test-harness): remove parseDefaultValue and CEM-related logic from SSR renderer#7532
radium-v merged 2 commits into
mainfrom
users/radium-v/test-harness-remove-cem-workaround

Conversation

@radium-v
Copy link
Copy Markdown
Collaborator

Pull Request

📖 Description

Remove the parseDefaultValue function and all Custom Elements Manifest (CEM) loading logic from the SSR renderer in @microsoft/fast-test-harness. This code was a workaround to populate default component state from CEM metadata during SSR test rendering, but it is no longer needed. State is now supplied directly through the test fixtures.

This is a refactor that reduces complexity and removes an implicit dependency on custom-elements.json files at render time.

📑 Test Plan

All existing tests pass. The parseDefaultValue test suite was removed along with the function itself. Remaining SSR renderer tests (renderTemplate, buildEntryHtml, buildState, createSSRRenderer) are unaffected.

✅ Checklist

General

  • I have included a change request file using $ npm run change
  • I have tested my changes.
  • I have updated the project documentation to reflect my changes.
  • I have read the CONTRIBUTING documentation and followed the standards for this project.

@radium-v radium-v force-pushed the users/radium-v/test-harness-remove-cem-workaround branch from 4c69360 to 6343cc4 Compare May 22, 2026 21:44
@radium-v radium-v merged commit 454714a into main May 22, 2026
14 checks passed
@radium-v radium-v deleted the users/radium-v/test-harness-remove-cem-workaround branch May 22, 2026 22:33
@janechu janechu mentioned this pull request May 27, 2026
5 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants