Yicong-Huang commented on code in PR #4881:
URL: https://github.com/apache/texera/pull/4881#discussion_r3179157432
##########
frontend/src/app/common/service/user/config/user-config.service.spec.ts:
##########
@@ -132,3 +132,9 @@ import type { Mock } from "vitest";
// });
// });
// });
+
+describe("UserConfigService", () => {
+ // Pre-existing spec body is commented out. Placeholder keeps Vitest's
+ // discovery happy; rewriting the real tests is tracked in #4880.
+ it.todo("add unit tests for UserConfigService");
+});
Review Comment:
Fair point — done in 06d07fe26d. The `describe("UserConfigService")`
placeholder block is removed, and the file is back on the `tsconfig.spec.json`
exclude list (with a comment noting the real tests still need to be rewritten).
The other three specs in this PR have actual `it()` cases, so only this one was
reverted.
--
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]
