#8980 Remove standalone mod api calls

[BLoe]

What does this PR do?

• Closes Drop support for standalone mod components - Slice 2 - Remove standalone mod component api calls #8980

For more information on our expectations for the PR process, see the
code review principles doc

[github-actions]

Playwright test results

 121 passed
 3 flaky
 6 skipped

Details

 Open report ↗︎
 130 tests across 44 suites
 40 minutes, 14 seconds
 8c99124
 For more information on how to debug and view this report, see our readme

Flaky tests

chrome › tests/pageEditor/addStarterBrick.spec.ts › Add starter brick to mod
msedge › tests/pageEditor/addStarterBrick.spec.ts › Add starter brick to mod
chrome › tests/pageEditor/copyMod.spec.ts › run a copied mod with a built-in integration

Skipped tests

chrome › tests/regressions/doNotCloseSidebarOnPageEditorSave.spec.ts › #8104: Do not automatically close the sidebar when saving in the Page Editor
msedge › tests/regressions/doNotCloseSidebarOnPageEditorSave.spec.ts › #8104: Do not automatically close the sidebar when saving in the Page Editor
chrome › tests/runtime/googleSheetsIntegration.spec.ts › can activate a google spreadsheet mod with config options
msedge › tests/runtime/googleSheetsIntegration.spec.ts › can activate a google spreadsheet mod with config options
chrome › tests/runtime/insertAtCursor.spec.ts › Insert at Cursor › 8154: can insert at cursor after opening sidebar from selection menu
msedge › tests/runtime/insertAtCursor.spec.ts › Insert at Cursor › 8154: can insert at cursor after opening sidebar from selection menu

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 74.22%. Comparing base (8318d74) to head (8c99124).
Report is 177 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #9006      +/-   ##
==========================================
- Coverage   74.24%   74.22%   -0.03%     
==========================================
  Files        1332     1347      +15     
  Lines       40817    41684     +867     
  Branches     7634     7792     +158     
==========================================
+ Hits        30306    30938     +632     
- Misses      10511    10746     +235     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[mnholtz]

👍

[mnholtz]

Oh, @BLoe this looks like the code you were talking about that was introduced to make it easy to ease off standalone mod components? Was thinking to myself that I didn't recognize the forceModComponentId

[BLoe]

Yes, that was introduced recently

[mnholtz]

I'm fine with deferring to others on the conversation regarding removing the useMods tests, with a follow-up ticket to test that hook

[github-actions]

No loom links were found in the first post. Please add one there if you'd like to it to appear on Slack.

Do not edit this comment manually.

[BLoe]

follow-up ticket to test that hook

@mnholtz There is so much change coming soon to the mods page and this hook, that I didn't think it would be worth writing new tests for what is essentially a "transient" state in the code base. Once the Mods Page changes settle down, I plan on going back through and adding test coverage everywhere that it makes sense at that point.