-
Notifications
You must be signed in to change notification settings - Fork 22
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Slice 4: Use ModInstance
type in mods updater
#9192
base: feature/devex-mod-instance-mods-screen
Are you sure you want to change the base?
Slice 4: Use ModInstance
type in mods updater
#9192
Conversation
@@ -143,42 +143,6 @@ describe("getActivatedMarketplaceModVersions function", () => { | |||
}, | |||
]); | |||
}); | |||
|
|||
it("reports error if multiple mod component versions activated for same mod", async () => { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can't happen anymore - ModInstance
only has a single version
); | ||
|
||
expect(nextModComponentState.activatedModComponents).toHaveLength(1); | ||
}); | ||
|
||
it("should do nothing if mod id does not have any activated mod components", async () => { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Instead of having it do nothing, the caller is now responsible for skipping. (The caller must produce a ModInstance
value)
Playwright test resultsDetails Open report ↗︎ Flaky testschrome › tests/pageEditor/addStarterBrick.spec.ts › Add starter brick to mod Skipped testschrome › tests/regressions/doNotCloseSidebarOnPageEditorSave.spec.ts › #8104: Do not automatically close the sidebar when saving in the Page Editor |
Remove test debugging statement
eefe60b
to
56f447b
Compare
ModInstance
type in mods updater (4/3)
ModInstance
type in mods updater (4/3)ModInstance
type in mods updater
What does this PR do?
ModInstance
type on the mods screen (3/3) #9191ModInstance
typeFuture Work
Continue gradual work toward eliminating mod components on the slice boundary:
selectActivatedModComponents
,selectGetModComponentsForMod
,selectModHasAnyActivatedModComponents
, etc.For more information on our expectations for the PR process, see the
code review principles doc