Comfy-Org / Comfy-Org/ComfyUI_frontend

[test-recorder] Add regression tests for UX-critical behaviors (standby start, inspector suppression, reassurance copy, tag instructions)

Open
#15,853 0 comments 0 reactions 1 assignee Claimed by @mattmillerai View on GitHub
area:testing developer experience enhancement
Dominant language
TypeScript
Stars
2k
Forks
699
Avg merge
1d 7h
Merged PRs (30d)
490

Description

### Problem
Several behaviors that fixed real user-testing failures are protected only by prose, not tests:

- Recorder starts in standby (not auto-recording) — `src/recorder/template.ts:74-85`; copy in `src/recorder/runner.ts:105-117` and `src/commands/guide.ts:62-66` must stay consistent with it.
- Inspector suppression via `PW_CODEGEN_NO_INSPECTOR=1` — `src/recorder/runner.ts:153-156`.
- Contributor-reassurance copy ("recordings can't be wrong", maintainers will tidy) — `src/commands/record.ts:248-260,788-793`; `src/commands/guide.ts:47-55`.
- Tag-multiselect keyboard instructions (SPACE to select, ENTER when done) — `src/commands/record.ts:475-488`.

Any of these can be silently removed in a refactor and would regress exactly the failures observed in user testing.

### Suggested fix
Add snapshot/behavioral tests: assert the codegen template starts in standby, assert the env var is passed to Playwright, and snapshot the reassurance/keyboard-instruction strings.

_Line references are against branch `cb/comfy-test-usability` (PR #15837); adjust after merge._

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.