test: update coverage comment to reflect current reality
- Remove all historical references (#48, first-principles, newly extracted, etc.) - Remove mentions of deleted components (5s Timer, old copy state machine) - Make the comment strictly about what the tests actually cover and the real shortcoming (no automated QML coverage) - Per request: code comments exist to explain the code, not preserve history
This commit is contained in:
parent
6e4e5d51ec
commit
b02f8ba45b
1 changed files with 6 additions and 9 deletions
|
|
@ -4,16 +4,13 @@ import lib from "../tailscalectl/lib.js"
|
|||
const { parsePeers, makeExitNodeCommand, findActiveExitNode, errorMessage, formatError, getStatusCommand, isValidExitNodeHostname, getClipboardCommands, buildToggleCommand, parseStatusResult, getStrings, shouldShowClearExitNode, isActiveExitNode } = lib
|
||||
|
||||
/*
|
||||
* Test coverage note for #48:
|
||||
* All pure functions exported from lib.js (including the newly extracted
|
||||
* getStrings, shouldShowClearExitNode, isActiveExitNode, and getClipboardCommands
|
||||
* are exercised here via node:test.
|
||||
* Unit tests for the pure functions exported from lib.js.
|
||||
*
|
||||
* The four Process objects, 5-second Timer, onExited handlers, copy fallback
|
||||
* state machine, and overall widget behavior in TailscaleWidget.qml have no
|
||||
* automated test coverage. There is no QML test runner in this project.
|
||||
* Those paths are verified manually (see verification checklist in the
|
||||
* feature branch commits).
|
||||
* All functions in lib.js are exercised via Node's built-in test runner.
|
||||
*
|
||||
* TailscaleWidget.qml has no automated test coverage. The four Process
|
||||
* objects, their onExited handlers, and all widget UI behavior must be
|
||||
* verified manually.
|
||||
*/
|
||||
|
||||
test("parsePeers extracts exitNode from ExitNodeOption", () => {
|
||||
|
|
|
|||
Loading…
Add table
Reference in a new issue