Skip to content

Commit e638da5

Browse files
chore: remove redundant grep tests from test-nohup-cleanup.sh (merge worktree-20260323-193735)
2 parents db8475b + 996882f commit e638da5

File tree

2 files changed

+4
-27
lines changed

2 files changed

+4
-27
lines changed

plugins/dso/.claude-plugin/plugin.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
{
22
"name": "dso",
3-
"version": "0.25.18",
3+
"version": "0.25.19",
44
"description": "Workflow infrastructure plugin for Claude Code projects",
55
"commands": "./commands/",
66
"skills": "./skills/",

tests/hooks/test-nohup-cleanup.sh

Lines changed: 3 additions & 26 deletions
Original file line numberDiff line numberDiff line change
@@ -200,32 +200,9 @@ fi
200200
assert_eq "test_session_start_calls_cleanup" "yes" "$_DISPATCHER_CALLS_CLEANUP"
201201

202202
# ============================================================
203-
# Test: function scans workflow-nohup-pids path
204-
# ============================================================
205-
_SCANS_REGISTRY="no"
206-
if grep -q 'workflow-nohup-pids' "$DSO_PLUGIN_DIR/hooks/lib/session-misc-functions.sh" 2>/dev/null; then
207-
_SCANS_REGISTRY="yes"
208-
fi
209-
assert_eq "test_scans_nohup_pid_registry" "yes" "$_SCANS_REGISTRY"
210-
211-
# ============================================================
212-
# Test: function checks command match (PID recycling protection)
213-
# ============================================================
214-
_CHECKS_CMD="no"
215-
if grep -qE 'command|cmd' "$DSO_PLUGIN_DIR/hooks/lib/session-misc-functions.sh" 2>/dev/null; then
216-
_CHECKS_CMD="yes"
217-
fi
218-
assert_eq "test_checks_command_match" "yes" "$_CHECKS_CMD"
219-
220-
# ============================================================
221-
# Test: function removes entry files
222-
# ============================================================
223-
_REMOVES_ENTRY="no"
224-
if grep -qE 'rm.*entry|remove.*entry' "$DSO_PLUGIN_DIR/hooks/lib/session-misc-functions.sh" 2>/dev/null; then
225-
_REMOVES_ENTRY="yes"
226-
fi
227-
assert_eq "test_removes_entry_files" "yes" "$_REMOVES_ENTRY"
228-
203+
# (Tests for registry scanning, command matching, and entry removal are
204+
# covered behaviorally by tests above: dead_process_entry_removed,
205+
# pid_recycling_protection_process_not_killed, multiple_dead_entries_removed.)
229206
# ============================================================
230207
# Test: 'test-plugin' orphan pattern is NOT in session-misc-functions.sh
231208
# ============================================================

0 commit comments

Comments
 (0)