Skip to content

test(cli): skip the node-dependent check when node is absent #9

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
pluginslab merged 1 commit into from
Sep 21, 2026
Merged

test(cli): skip the node-dependent check when node is absent #9

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
42 changes: 27 additions & 15 deletions tests/cli/test-substitutions.sh
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -82,21 +82,33 @@ assert_file_contains "cli → classPrefix joins on underscore" "$CLI" ".join('_'
# --- Case 4: deriveIdentity produces the expected forms ---
# Exercise the real derivation rather than trusting the source reads right.
# node evaluates the same expressions the CLI uses, for a representative slug.
derived=$(node -e '
const slug = "acme-order-tracker";
const cap = s => s[0].toUpperCase() + s.slice(1);
const parts = slug.split("-").filter(Boolean);
console.log([
parts.map(cap).join(""), // namespace
parts.map(cap).join("_"), // classPrefix
slug.toUpperCase().replace(/-/g, "_"),
slug.replace(/-/g, "_"),
].join(" "));
' 2>/dev/null)

assert_equals "deriveIdentity → all four forms for acme-order-tracker" \
"AcmeOrderTracker Acme_Order_Tracker ACME_ORDER_TRACKER acme_order_tracker" \
"$derived"
#
# Guarded on node being installed. The CLI is a Node program, so a host without
# node cannot run the scaffolder at all and has nothing to assert about it —
# but the rest of this file is plain bash and stays useful. Without the guard
# this failed with a baffling `got ''`, because the error was swallowed by a
# 2>/dev/null. That is the same "the test encodes the author's machine" trap
# that hid the Linux mtime bug in the hooks; it does not get a pass here just
# because it is our own test.
if command -v node >/dev/null 2>&1; then
derived=$(node -e '
const slug = "acme-order-tracker";
const cap = s => s[0].toUpperCase() + s.slice(1);
const parts = slug.split("-").filter(Boolean);
console.log([
parts.map(cap).join(""), // namespace
parts.map(cap).join("_"), // classPrefix
slug.toUpperCase().replace(/-/g, "_"),
slug.replace(/-/g, "_"),
].join(" "));
')

assert_equals "deriveIdentity → all four forms for acme-order-tracker" \
"AcmeOrderTracker Acme_Order_Tracker ACME_ORDER_TRACKER acme_order_tracker" \
"$derived"
else
echo " - skipped: deriveIdentity check needs node on PATH"
fi

# --- Case 5: the main plugin file gets renamed, not just rewritten ---
# WordPress expects {slug}.php; leaving pl-example.php in place breaks the
Expand Down
Loading