Comfy-Org / Comfy-Org/ComfyUI_frontend

Fix or remove 4 skipped/TODO test files from litegraph migration

Open
#11,078 0 comments 0 reactions 1 assignee Claimed by @kaili-yang View on GitHub
area:testing audit:half-implemented audit:testing code-audit developer experience effort:medium priority:nice-to-have
Dominant language
TypeScript
Stars
2k
Forks
699
Avg merge
1d 7h
Merged PRs (30d)
490

Description

## Recommendation

### 🔧 FINISH: Post-Migration Test Cleanup

4 test files are marked with `TODO: Fix these tests after migration` or have skipped tests:

| File | Issue |
|------|-------|
| `src/lib/litegraph/src/measure.test.ts` | `// TODO: Fix these tests after migration` |
| `src/lib/litegraph/src/canvas/LinkConnector.integration.test.ts` | `// TODO: Fix these tests after migration` |
| `src/lib/litegraph/src/canvas/LinkConnectorSubgraphInputValidation.test.ts` | `// TODO: Fix these tests after migration` |
| `src/stores/queueStore.test.ts:192` | `it.skip('should parse text outputs')` |

## How to Fix

For each:
1. Attempt to fix the test to work with the current code
2. If the tested functionality was removed/changed, update or remove the test
3. Never leave `it.skip` indefinitely — either fix or remove with a comment why

---
**Category:** Half-Implemented / Testing | **Priority:** Nice-to-have | **Effort:** Medium
Part of #11022
_Filed by repo-audit skill_

┆Issue is synchronized with this [Notion page](https://www.notion.so/Issue-11078-Fix-or-remove-4-skipped-TODO-test-files-from-litegraph-migration-33e6d73d36508189952ace1481bf7cce) by [Unito](https://www.unito.io)

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.