Yicong-Huang commented on code in PR #4878:
URL: https://github.com/apache/texera/pull/4878#discussion_r3178906076
##########
frontend/src/tsconfig.spec.json:
##########
@@ -11,33 +11,65 @@
},
"include": ["**/*.spec.ts", "**/*.d.ts", "vitest-globals.d.ts",
"jsdom-svg-polyfill.ts"],
"exclude": [
- // Component specs deferred to a follow-up PR. They pull entire
- // template trees through the spec compile; the new unit-test builder
- // does stricter component / module resolution than the legacy karma
- // path and surfaces hundreds of "is not a known property/element"
- // errors that boil down to standalone vs NgModule scope mismatch in
- // the existing TestBed setups. Re-enabling them is tracked in #4861.
- "**/*.component.spec.ts",
- // Service specs that transitively pull a NgModule-declared component
- // through their import chain (most reach `auth.service` →
- // `RegistrationRequestModalComponent`, whose template fails the same
- // type check). Deferred for the same follow-up.
- "**/coeditor-presence.service.spec.ts",
- "**/execute-workflow.service.spec.ts",
- "**/user.service.spec.ts",
- "**/workflow-websocket.service.spec.ts",
- "**/workflow-result-export.service.spec.ts",
- "**/udf-debug.service.spec.ts",
- "**/user-config.service.spec.ts",
- "**/operator-menu.service.spec.ts",
- "**/workflow-console.service.spec.ts",
- "**/operator-reuse-cache-status.service.spec.ts",
- "**/workflow-result.service.spec.ts",
- "**/download.service.spec.ts",
- "**/preset.service.spec.ts",
// jsdom polyfill is enough to instantiate jointjs but its zero-
// dimension fake matrices break the actual graph-geometry math
- // these tests assert on. Real fix is Vitest browser mode (#4861).
- "**/drag-drop.service.spec.ts"
+ // these tests assert on. Real fix is Vitest browser mode (#4866).
+ "**/drag-drop.service.spec.ts",
+
+ // Component / service specs whose TestBed setups still need to be
+ // adapted to the standalone-component world. The migration sweep
+ // (PR #4862 + PR #4873) made the runtime resolution succeed, but
+ // each TestBed configures `declarations: [...]` plus selective
+ // module imports — that combo no longer matches what the new
+ // builder expects from a standalone-component graph. Tracked as
+ // follow-ups under #4861.
+ "**/app/common/formly/preset-wrapper/preset-wrapper.component.spec.ts",
+ "**/app/common/service/user/config/user-config.service.spec.ts",
+ "**/app/common/service/user/user.service.spec.ts",
+
"**/app/dashboard/component/admin/execution/admin-execution.component.spec.ts",
Review Comment:
will be removed later
--
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]
