[Ma77Ball]

What changes were proposed in this PR?

• Adds a Vitest spec for FileSaverService, the thin wrapper whose saveAs(data, filename?, options?) forwards straight to the file-saver library's saveAs.
• Mocks the file-saver library so the suite can assert the call is forwarded unchanged: a Blob payload passes through with its filename and options, and a string payload (such as a URL) passes through.
• Asserts that an omitted filename and options arrive as undefined, and that an error thrown by the underlying saveAs propagates to the caller instead of being swallowed.

Any related issues, documentation, discussions?

Closes: #6448
Supersedes #6460, which the stale bot auto-closed for inactivity. The branch is now merged up to current main.

How was this PR tested?

• Run cd frontend && npx ng test --include="**/file-saver.service.spec.ts" --watch=false, expect the FileSaverService suite to pass.
• Confirm the forwarding cases hold: a Blob and a string payload each reach the mocked saveAs unchanged, an omitted filename and options arrive as undefined, and a thrown error surfaces to the caller.

Was this PR authored or co-authored using generative AI tooling?

Co-authored with Claude Opus 4.8 in compliance with ASF

[github-actions]

Automated Reviewer Suggestions

Based on the git blame history of the changed files, we recommend the following reviewers:

• No candidates found from git blame history.

[codecov-commenter]

❌ 5 Tests Failed:

Tests completed	Failed	Passed	Skipped
6832	5	6827	1

View the top 3 failed test(s) by shortest run time

src/app/dashboard/service/user/file/file-saver.service.spec.ts > FileSaverService > passes undefined through when filename and options are omitted

Stack Traces | 0.00296s run time

AssertionError: expected "vi.fn()" to be called with arguments: [ Blob{ …(1) }, undefined, undefined ]

Number of calls: 0

 ❯ .../user/file/file-saver.service.spec.ts:71:20
 ❯ src/test-zone-setup.ts:90:33
 ❯ _ZoneDelegate.invoke node_modules/zone.js/fesm2015/zone.js:398:28
 ❯ ProxyZoneSpec.onInvoke node_modules/zone.js/fesm2015/zone-testing.js:2132:39
 ❯ _ZoneDelegate.invoke node_modules/zone.js/fesm2015/zone.js:397:34
 ❯ ZoneImpl.run node_modules/zone.js/fesm2015/zone.js:113:43
 ❯ src/test-zone-setup.ts:88:16

src/app/dashboard/service/user/file/file-saver.service.spec.ts > FileSaverService > propagates errors thrown by saveAs

Stack Traces | 0.00304s run time

AssertionError: expected function to throw an error, but it didn't
 ❯ .../user/file/file-saver.service.spec.ts:63:57
 ❯ src/test-zone-setup.ts:90:33
 ❯ _ZoneDelegate.invoke node_modules/zone.js/fesm2015/zone.js:398:28
 ❯ ProxyZoneSpec.onInvoke node_modules/zone.js/fesm2015/zone-testing.js:2132:39
 ❯ _ZoneDelegate.invoke node_modules/zone.js/fesm2015/zone.js:397:34
 ❯ ZoneImpl.run node_modules/zone.js/fesm2015/zone.js:113:43
 ❯ src/test-zone-setup.ts:88:16
 ❯ new ZoneAwarePromise node_modules/zone.js/fesm2015/zone.js:2701:25
 ❯ wrapper src/test-zone-setup.ts:86:16
 ❯ src/test-zone-setup.ts:90:33

src/app/dashboard/service/user/file/file-saver.service.spec.ts > FileSaverService > propagates errors thrown by saveAs instead of swallowing them

Stack Traces | 0.0036s run time

AssertionError: expected function to throw an error, but it didn't
 ❯ .../user/file/file-saver.service.spec.ts:80:66
 ❯ src/test-zone-setup.ts:90:33
 ❯ _ZoneDelegate.invoke node_modules/zone.js/fesm2015/zone.js:398:28
 ❯ ProxyZoneSpec.onInvoke node_modules/zone.js/fesm2015/zone-testing.js:2132:39
 ❯ _ZoneDelegate.invoke node_modules/zone.js/fesm2015/zone.js:397:34
 ❯ ZoneImpl.run node_modules/zone.js/fesm2015/zone.js:113:43
 ❯ src/test-zone-setup.ts:88:16
 ❯ new ZoneAwarePromise node_modules/zone.js/fesm2015/zone.js:2701:25
 ❯ wrapper src/test-zone-setup.ts:86:16
 ❯ src/test-zone-setup.ts:90:33

src/app/dashboard/service/user/file/file-saver.service.spec.ts > FileSaverService > forwards a string payload (e.g. a URL) unchanged

Stack Traces | 0.00433s run time

AssertionError: expected "vi.fn()" to be called with arguments: [ Array(3) ]

Number of calls: 0

 ❯ .../user/file/file-saver.service.spec.ts:54:20
 ❯ src/test-zone-setup.ts:90:33
 ❯ _ZoneDelegate.invoke node_modules/zone.js/fesm2015/zone.js:398:28
 ❯ ProxyZoneSpec.onInvoke node_modules/zone.js/fesm2015/zone-testing.js:2132:39
 ❯ _ZoneDelegate.invoke node_modules/zone.js/fesm2015/zone.js:397:34
 ❯ ZoneImpl.run node_modules/zone.js/fesm2015/zone.js:113:43
 ❯ src/test-zone-setup.ts:88:16

src/app/dashboard/service/user/file/file-saver.service.spec.ts > FileSaverService > delegates a Blob payload to saveAs with filename and options

Stack Traces | 0.00484s run time

AssertionError: expected "vi.fn()" to be called 1 times, but got 0 times
 ❯ .../user/file/file-saver.service.spec.ts:47:20
 ❯ src/test-zone-setup.ts:90:33
 ❯ _ZoneDelegate.invoke node_modules/zone.js/fesm2015/zone.js:398:28
 ❯ ProxyZoneSpec.onInvoke node_modules/zone.js/fesm2015/zone-testing.js:2132:39
 ❯ _ZoneDelegate.invoke node_modules/zone.js/fesm2015/zone.js:397:34
 ❯ ZoneImpl.run node_modules/zone.js/fesm2015/zone.js:113:43
 ❯ src/test-zone-setup.ts:88:16
 ❯ new ZoneAwarePromise node_modules/zone.js/fesm2015/zone.js:2701:25
 ❯ wrapper src/test-zone-setup.ts:86:16
 ❯ src/test-zone-setup.ts:90:33

To view more test analytics, go to the Test Analytics Dashboard
_{📋 Got 3 mins? Take this short survey to help us improve Test Analytics.}

[aglinxinyuan]

Please fix the ci.