test(frontend): add unit tests for execute-workflow guards

[Ma77Ball]

What changes were proposed in this PR?

• Add frontend/src/app/workspace/types/execute-workflow.interface.spec.ts, a new Vitest spec for the execute-workflow type guards, which previously had no dedicated unit tests.
• Cover isWebPaginationUpdate (pagination vs snapshot/delta) and isWebDataUpdate (snapshot/delta vs pagination).
• Cover isNotInExecution for all four terminal/uninitialized states (true) and the seven active states (false).

Any related issues, documentation, discussions?

Closes: #6648

How was this PR tested?

• Run: cd frontend && node --max-old-space-size=8192 ./node_modules/nx/dist/bin/nx.js test gui --watch=false --include=src/app/workspace/types/execute-workflow.interface.spec.ts, expect all 6 tests passing.
• Test-only change; no production code is modified.

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:

• Contributors with relevant context: @PG1204
  You can notify them by mentioning @PG1204 in a comment.

[aglinxinyuan]

LGTM!

[Copilot]

Pull request overview

Adds a new Vitest spec file to cover the execute-workflow type guards and execution-state helper used by the frontend workspace workflow execution flow.

Changes:

• Added unit tests for isWebPaginationUpdate and isWebDataUpdate (pagination vs snapshot/delta).
• Added unit tests for isNotInExecution across terminal/uninitialized vs active execution states.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[aglinxinyuan]

Please address the comments.

[aglinxinyuan]

Merged main to trigger coverage report.

[codecov-commenter]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 76.27%. Comparing base (110a54c) to head (c80e627).
⚠️ Report is 1 commits behind head on main.
✅ All tests successful. No failed tests found.

Additional details and impacted files

@@            Coverage Diff            @@
##               main    #6649   +/-   ##
=========================================
  Coverage     76.27%   76.27%           
  Complexity     3452     3452           
=========================================
  Files          1161     1161           
  Lines         45919    45918    -1     
  Branches       5100     5099    -1     
=========================================
  Hits          35025    35025           
  Misses         9329     9329           
+ Partials       1565     1564    -1     

Flag	Coverage Δ		*Carryforward flag
access-control-service	70.00% <ø> (ø)		Carriedforward from c24755c
agent-service	76.76% <ø> (ø)		Carriedforward from c24755c
amber	67.33% <ø> (ø)		Carriedforward from c24755c
computing-unit-managing-service	20.49% <ø> (ø)		Carriedforward from c24755c
config-service	66.66% <ø> (ø)		Carriedforward from c24755c
file-service	67.21% <ø> (ø)		Carriedforward from c24755c
frontend	81.71% <ø> (+<0.01%)	⬆️	Carriedforward from c24755c
notebook-migration-service	78.94% <ø> (ø)		Carriedforward from c24755c
pyamber	92.15% <ø> (ø)		Carriedforward from c24755c
workflow-compiling-service	55.14% <ø> (ø)		Carriedforward from c24755c

*This pull request uses carry forward flags. Click here to find out more.

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.