jtmorris/dms_tailscalectl
1
0
Fork 0
Code Issues 2 Pull requests Projects Releases Packages Activity Actions

Code formatting cleanup.

Browse source
This commit is contained in:
Vybe (Coding Agent) 2026-05-22 21:47:41 +00:00
parent b02f8ba45b
commit 12ef07040c
2 changed files with 7 additions and 7 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
tailscalectl/lib.js
View file

@ -59,8 +59,8 @@ function getStrings() {
} }
} }
// Light UI predicates — keep the view thin. // Light UI predicates — keep the view thin.
function shouldShowClearExitNode(currentExitNode) { function shouldShowClearExitNode(currentExitNode) {
return currentExitNode !== "" return currentExitNode !== ""
} }

10
test/lib.test.js
View file

@ -85,7 +85,7 @@ test("errorMessage returns generic message for unknown command", () => {
assert.strictEqual(msg, "Tailscale command failed") assert.strictEqual(msg, "Tailscale command failed")
}) })
// --- getClipboardCommands (simple array, argv only, per #48 first-principles) --- // --- getClipboardCommands ---
test("getClipboardCommands returns ordered argv arrays with text appended", () => { test("getClipboardCommands returns ordered argv arrays with text appended", () => {
const cmds = getClipboardCommands("1.2.3.4") const cmds = getClipboardCommands("1.2.3.4")
@ -108,7 +108,7 @@ test("getClipboardCommands is deterministic and open for future tools", () => {
assert.ok(Array.isArray(cmds[1])) assert.ok(Array.isArray(cmds[1]))
}) })
// --- getStrings (for #34 i18n prep) --- // --- getStrings ---
test("getStrings returns canonical UI strings for the widget", () => { test("getStrings returns canonical UI strings for the widget", () => {
const s = getStrings() const s = getStrings()
@ -196,7 +196,7 @@ test("parseStatusResult sets isConnected false for non-Running BackendState", ()
assert.strictEqual(state.isConnected, false) assert.strictEqual(state.isConnected, false)
}) })
// --- formatError (central error + detail formatting per first-principles plan) --- // --- formatError (central error + detail formatting) ---
test("formatError returns base message without detail", () => { test("formatError returns base message without detail", () => {
assert.strictEqual(formatError("status"), "Failed to read Tailscale status") assert.strictEqual(formatError("status"), "Failed to read Tailscale status")
@ -216,7 +216,7 @@ test("formatError handles empty or falsy detail gracefully", () => {
assert.strictEqual(formatError("connect", null), "Failed to connect to Tailscale") assert.strictEqual(formatError("connect", null), "Failed to connect to Tailscale")
}) })
// --- isValidExitNodeHostname + makeExitNodeCommand safety (balanced paranoia, direct-argv defense-in-depth) --- // --- isValidExitNodeHostname + makeExitNodeCommand safety ---
test("isValidExitNodeHostname accepts empty string (clear)", () => { test("isValidExitNodeHostname accepts empty string (clear)", () => {
assert.strictEqual(isValidExitNodeHostname(""), true) assert.strictEqual(isValidExitNodeHostname(""), true)
@ -244,7 +244,7 @@ test("makeExitNodeCommand still produces correct argv for valid input", () => {
assert.deepStrictEqual(makeExitNodeCommand("gluetun-sjc"), ["tailscale", "set", "--exit-node=gluetun-sjc"]) assert.deepStrictEqual(makeExitNodeCommand("gluetun-sjc"), ["tailscale", "set", "--exit-node=gluetun-sjc"])
}) })
// --- getStatusCommand (new pure helper for low-duty-cycle architecture) --- // --- getStatusCommand ---
test("getStatusCommand returns the canonical tailscale status --json argv", () => { test("getStatusCommand returns the canonical tailscale status --json argv", () => {
const cmd = getStatusCommand() const cmd = getStatusCommand()