aglinxinyuan opened a new pull request, #6817: URL: https://github.com/apache/texera/pull/6817
### What changes were proposed in this PR? Extends `download.service.spec.ts` with 4 tests (19 -> 23) and un-skips the previously-skipped zip tests, reaching 100% coverage of `download.service.ts`. New/strengthened tests cover the multi-file operators-result zip branch (loads the produced blob back and asserts entries), the `flat()` merge routing, `createWorkflowsZip` assembly (one JSON per workflow), `nameWorkflow` collision de-duplication, and the `exportWorkflowResultToLocal` empty-token fallback. **Source change:** `import * as JSZip` -> `import JSZip from "jszip"` in `download.service.ts`. The namespace import makes `new JSZip()` throw "not a constructor" under esbuild/Vitest (and esbuild warns it will crash at runtime), which had forced every zip path to be skipped. The default import matches the existing usage in `user-workflow.component.ts` (already shipped in the prod build) and is safe for the webpack prod build — jszip is CommonJS (`module.exports = JSZip`) and `allowSyntheticDefaultImports` is set. ### Any related issues, documentation, discussions? Closes #6813. ### How was this PR tested? `ng test --include='**/download.service.spec.ts'` -> 23/23 passing (0 skipped). `yarn format:ci` passes. The default-import pattern is already used and shipped in `user-workflow.component.ts`. ### Was this PR authored or co-authored using generative AI tooling? Generated-by: Claude Code (Opus 4.8 [1M context]) -- 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]
