Skip to content

refactor: rename WebApplication on-disk directory from webapplications to uiBundles @W-21674925@#1719

Open
deepu-mungamuri94 wants to merge 1 commit intoforcedotcom:mainfrom
deepu-mungamuri94:rename-webapplications-to-uiBundles
Open

refactor: rename WebApplication on-disk directory from webapplications to uiBundles @W-21674925@#1719
deepu-mungamuri94 wants to merge 1 commit intoforcedotcom:mainfrom
deepu-mungamuri94:rename-webapplications-to-uiBundles

Conversation

@deepu-mungamuri94
Copy link
Copy Markdown

Aligns the SDR metadata registry and all test fixtures with the recent rename in plugin-templates (webapp → ui-bundle), where the user-facing folder constant changed from webapplications to uiBundles.

What does this PR do?

  • metadataRegistry.json: update directoryName and directoryNames map key for the webapplication type from "webapplications" to "uiBundles"
  • Rename test fixture directories: originalMdapi, snapshot expected outputs, and NUT testProj source directories
  • Update hardcoded "webapplications" path strings in test TypeScript files: filePathGenerator.test.ts, webApplicationValidation.test.ts, webApplications.nut.ts, webApplicationsDeploy.nut.ts, metadataApiDeploy.test.ts

Intentionally left unchanged:

  • Registry id/name/suffix (webapplication / WebApplication) — Salesforce API names
  • Adapter strategy string, class names, TypeScript file names — internal SDR identifiers
  • File suffixes (.webapplication-meta.xml, webapplication.json) — Salesforce API surface
  • Metadata API server-response fileName strings in deploy test (server-side concern)

What issues does this PR fix or reference?

@W-21674925@

…s to uiBundles @W-21674925@

Aligns the SDR metadata registry and all test fixtures with the recent
rename in plugin-templates (webapp → ui-bundle), where the user-facing
folder constant changed from `webapplications` to `uiBundles`.

Changes:
- metadataRegistry.json: update `directoryName` and `directoryNames` map
  key for the webapplication type from "webapplications" to "uiBundles"
- Rename test fixture directories: originalMdapi, snapshot expected
  outputs, and NUT testProj source directories
- Update hardcoded "webapplications" path strings in test TypeScript
  files: filePathGenerator.test.ts, webApplicationValidation.test.ts,
  webApplications.nut.ts, webApplicationsDeploy.nut.ts,
  metadataApiDeploy.test.ts

Intentionally left unchanged:
- Registry id/name/suffix (webapplication / WebApplication) — Salesforce API names
- Adapter strategy string, class names, TypeScript file names — internal SDR identifiers
- File suffixes (.webapplication-meta.xml, webapplication.json) — Salesforce API surface
- Metadata API server-response fileName strings in deploy test (server-side concern)

Made-with: Cursor
"appTemplates": "appframeworktemplatebundle",
"lightningTypes": "lightningtypebundle",
"webapplications": "webapplication"
"uiBundles": "webapplication"
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this needs to be uibundle

expect(zip.file('webapplications/HappyApp/HappyApp.webapplication-meta.xml')).to.exist;
expect(zip.file('webapplications/HappyApp/webapplication.json')).to.exist;
expect(zip.file('webapplications/HappyApp/src/index.html')).to.exist;
expect(zip.file('uiBundles/HappyApp/HappyApp.webapplication-meta.xml')).to.exist;
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

all these need to be updated

const zip = await JSZip.loadAsync(zipBuffer as Buffer);
zip.remove('webapplications/HappyApp/webapplication.json');
zip.remove('webapplications/HappyApp/src/index.html');
zip.remove('uiBundles/HappyApp/webapplication.json');
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

here

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants