Copilot commented on code in PR #7370:
URL: https://github.com/apache/texera/pull/7370#discussion_r3732542868
##########
frontend/src/app/dashboard/component/user/user-workflow/user-workflow.component.spec.ts:
##########
@@ -1019,5 +1020,143 @@ describe("SavedWorkflowSectionComponent", () => {
});
});
});
+
+ describe("template rendering", () => {
+ const q = (selector: string) =>
fixture.debugElement.query(By.css(selector));
+
Review Comment:
The new "template rendering" tests spy on multiple component methods (via
`vi.spyOn`) and also toggle view type via button clicks (which persists to
`localStorage`). Unlike other blocks in this spec (e.g. around lines ~404 and
~734), this block has no `afterEach` cleanup, so spies and the persisted
view-mode key can leak between tests and make future additions/order changes
flaky. Add an `afterEach` that restores mocks and clears the view-mode storage
key.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
