forge-workflow 0.0.4 → 0.0.6

package/scripts/forge-team/tests/hooks.test.sh

@@ -0,0 +1,239 @@
+#!/usr/bin/env bash
+set -euo pipefail
+
+SCRIPT_DIR="$(cd "$(dirname "${BASH_SOURCE[0]}")/.." && pwd)"
+
+# ── Setup temp dir for test isolation ────────────────────────────────────
+TEST_TMP="$(mktemp -d)"
+trap 'rm -rf "$TEST_TMP"' EXIT
+
+# ── Create mock gh (authenticated) ──────────────────────────────────────
+mock_dir="$(mktemp -d)"
+
+cat > "$mock_dir/gh" << 'MOCK'
+#!/usr/bin/env bash
+case "$*" in
+  "auth status")
+    echo "Logged in to github.com as devone"
+    exit 0
+    ;;
+  "issue edit"*)
+    # Accept any issue edit (sync_issue_status)
+    exit 0
+    ;;
+esac
+MOCK
+chmod +x "$mock_dir/gh"
+
+# ── Create mock gh (not authenticated) ──────────────────────────────────
+cat > "$mock_dir/gh-noauth" << 'MOCK'
+#!/usr/bin/env bash
+case "$*" in
+  "auth status")
+    echo "You are not logged in" >&2
+    exit 1
+    ;;
+esac
+MOCK
+chmod +x "$mock_dir/gh-noauth"
+
+# ── Create mock bd (with in_progress issues having github_issue state) ──
+cat > "$mock_dir/bd" << 'MOCK'
+#!/usr/bin/env bash
+case "$1 $2" in
+  "list --status=in_progress")
+    echo "◐ forge-aaa · Feature A"
+    echo "◐ forge-bbb · Feature B"
+    ;;
+  "show forge-aaa")
+    echo "◐ forge-aaa · Feature A [● P2 · IN_PROGRESS]"
+    echo "Owner: devone"
+    echo "Updated: 2026-03-27T10:00:00Z"
+    echo ""
+    echo "STATE"
+    echo "  github_issue:42"
+    ;;
+  "show forge-bbb")
+    echo "◐ forge-bbb · Feature B [● P2 · IN_PROGRESS]"
+    echo "Owner: devtwo"
+    echo "Updated: 2026-03-27T09:00:00Z"
+    echo ""
+    echo "STATE"
+    echo "  github_issue:55"
+    ;;
+esac
+MOCK
+chmod +x "$mock_dir/bd"
+
+# ── Create mock bd (no in_progress issues) ──────────────────────────────
+cat > "$mock_dir/bd-empty" << 'MOCK'
+#!/usr/bin/env bash
+case "$1 $2" in
+  "list --status=in_progress")
+    echo ""
+    ;;
+esac
+MOCK
+chmod +x "$mock_dir/bd-empty"
+
+# ── Create mock bd (issue without github_issue state) ───────────────────
+cat > "$mock_dir/bd-no-gh" << 'MOCK'
+#!/usr/bin/env bash
+case "$1 $2" in
+  "list --status=in_progress")
+    echo "◐ forge-xxx · Feature X"
+    ;;
+  "show forge-xxx")
+    echo "◐ forge-xxx · Feature X [● P2 · IN_PROGRESS]"
+    echo "Owner: devone"
+    echo "Updated: 2026-03-27T10:00:00Z"
+    ;;
+esac
+MOCK
+chmod +x "$mock_dir/bd-no-gh"
+
+# ── Create .beads/config.yaml for auto-sync tests ──────────────────────
+mkdir -p "$TEST_TMP/.beads"
+cat > "$TEST_TMP/.beads/config.yaml" << 'YAML'
+team:
+  auto-sync: true
+YAML
+
+export GH_CMD="$mock_dir/gh"
+export BD_CMD="$mock_dir/bd"
+export BEADS_ROOT="$TEST_TMP"
+
+# Source the library under test
+source "$SCRIPT_DIR/lib/hooks.sh"
+
+PASS=0
+FAIL=0
+
+assert_eq() {
+  local label="$1" expected="$2" actual="$3"
+  if [[ "$actual" == "$expected" ]]; then
+    PASS=$((PASS + 1)); echo "  PASS: $label"
+  else
+    FAIL=$((FAIL + 1)); echo "  FAIL: $label (expected '$expected', got '$actual')"
+  fi
+}
+
+assert_contains() {
+  local label="$1" expected="$2" actual="$3"
+  if [[ "$actual" == *"$expected"* ]]; then
+    PASS=$((PASS + 1)); echo "  PASS: $label"
+  else
+    FAIL=$((FAIL + 1)); echo "  FAIL: $label (expected to contain '$expected', got '$actual')"
+  fi
+}
+
+assert_not_contains() {
+  local label="$1" unexpected="$2" actual="$3"
+  if [[ "$actual" != *"$unexpected"* ]]; then
+    PASS=$((PASS + 1)); echo "  PASS: $label"
+  else
+    FAIL=$((FAIL + 1)); echo "  FAIL: $label (expected NOT to contain '$unexpected', got '$actual')"
+  fi
+}
+
+assert_exit() {
+  local label="$1" expected="$2" actual="$3"
+  if [[ "$actual" -eq "$expected" ]]; then
+    PASS=$((PASS + 1)); echo "  PASS: $label"
+  else
+    FAIL=$((FAIL + 1)); echo "  FAIL: $label (expected exit $expected, got $actual)"
+  fi
+}
+
+# ── Test 1: forge_team_sync with mock gh + bd → syncs in_progress issues ──
+echo "── Test 1: forge_team_sync syncs in_progress issues ──"
+export GH_CMD="$mock_dir/gh"
+export BD_CMD="$mock_dir/bd"
+rc=0
+output="$(forge_team_sync 2>&1)" || rc=$?
+assert_exit "exits 0" 0 "$rc"
+assert_contains "reports synced issues" "Synced 2 issues" "$output"
+
+# ── Test 2: forge_team_sync with gh not authenticated → error, exit 1 ──
+echo ""
+echo "── Test 2: forge_team_sync with gh not authenticated → error, exit 1 ──"
+export GH_CMD="$mock_dir/gh-noauth"
+export BD_CMD="$mock_dir/bd"
+rc=0
+output="$(forge_team_sync 2>&1)" || rc=$?
+assert_exit "exits 1" 1 "$rc"
+assert_contains "error message" "FORGE_AGENT_7f3a:ERROR:" "$output"
+
+# ── Test 3: forge_team_sync with auto-sync: false → skip, exit 0 ────────
+echo ""
+echo "── Test 3: forge_team_sync with auto-sync: false → skip, exit 0 ──"
+export GH_CMD="$mock_dir/gh"
+export BD_CMD="$mock_dir/bd"
+cat > "$TEST_TMP/.beads/config.yaml" << 'YAML'
+team:
+  auto-sync: false
+YAML
+rc=0
+output="$(forge_team_sync 2>&1)" || rc=$?
+assert_exit "exits 0" 0 "$rc"
+assert_not_contains "does not sync" "Synced" "$output"
+# Restore config
+cat > "$TEST_TMP/.beads/config.yaml" << 'YAML'
+team:
+  auto-sync: true
+YAML
+
+# ── Test 4: forge_team_sync with no in_progress issues → nothing to sync ──
+echo ""
+echo "── Test 4: forge_team_sync with no in_progress issues → nothing to sync ──"
+export GH_CMD="$mock_dir/gh"
+export BD_CMD="$mock_dir/bd-empty"
+rc=0
+output="$(forge_team_sync 2>&1)" || rc=$?
+assert_exit "exits 0" 0 "$rc"
+assert_contains "nothing to sync message" "Nothing to sync" "$output"
+
+# ── Test 5: forge_team_sync --quiet → no output on success ──────────────
+echo ""
+echo "── Test 5: forge_team_sync --quiet → no output on success ──"
+export GH_CMD="$mock_dir/gh"
+export BD_CMD="$mock_dir/bd"
+rc=0
+output="$(forge_team_sync --quiet 2>&1)" || rc=$?
+assert_exit "exits 0" 0 "$rc"
+assert_eq "no output" "" "$output"
+
+# ── Test 6: forge_team_sync_on_stage triggers correct status sync ────────
+echo ""
+echo "── Test 6: forge_team_sync_on_stage triggers correct status sync per stage ──"
+export GH_CMD="$mock_dir/gh"
+export BD_CMD="$mock_dir/bd"
+# dev transition → in_progress
+rc=0
+output="$(forge_team_sync_on_stage forge-aaa plan dev 2>&1)" || rc=$?
+assert_exit "dev transition exits 0" 0 "$rc"
+# ship transition → in_progress (still active)
+rc=0
+output="$(forge_team_sync_on_stage forge-aaa dev ship 2>&1)" || rc=$?
+assert_exit "ship transition exits 0" 0 "$rc"
+# verify transition → skip (closed by /verify)
+rc=0
+output="$(forge_team_sync_on_stage forge-aaa ship verify 2>&1)" || rc=$?
+assert_exit "verify transition exits 0" 0 "$rc"
+assert_contains "verify skips sync" "skipped" "$output"
+
+# ── Test 7: Sync failure doesn't block (non-fatal) ──────────────────────
+echo ""
+echo "── Test 7: Sync failure doesn't block (non-fatal warning) ──"
+export GH_CMD="$mock_dir/gh"
+export BD_CMD="$mock_dir/bd-no-gh"
+rc=0
+output="$(forge_team_sync 2>&1)" || rc=$?
+# Should still exit 0 even with sync failures (non-blocking)
+assert_exit "exits 0 despite sync failure" 0 "$rc"
+assert_contains "warns about failure" "warn" "$output"
+
+# ── Results ──────────────────────────────────────────────────────────────
+echo ""
+echo "Results: $PASS passed, $FAIL failed"
+[[ $FAIL -eq 0 ]] && exit 0 || exit 1

package/scripts/forge-team/tests/identity.test.sh

@@ -0,0 +1,176 @@
+#!/usr/bin/env bash
+set -euo pipefail
+
+SCRIPT_DIR="$(cd "$(dirname "${BASH_SOURCE[0]}")/.." && pwd)"
+
+# ── Setup temp dir for test isolation ────────────────────────────────────
+TEST_TMP="$(mktemp -d)"
+trap 'rm -rf "$TEST_TMP"' EXIT
+
+# Override TEAM_MAP_ROOT so tests don't touch real .beads/
+export TEAM_MAP_ROOT="$TEST_TMP"
+
+# ── Create mock gh ───────────────────────────────────────────────────────
+mock_dir="$(mktemp -d)"
+cat > "$mock_dir/gh" << 'MOCK'
+#!/usr/bin/env bash
+if [[ "$1" == "api" && "$2" == "user" ]]; then
+  echo "testuser"
+  exit 0
+fi
+echo "unknown" >&2; exit 1
+MOCK
+chmod +x "$mock_dir/gh"
+
+# Create a failing mock gh
+cat > "$mock_dir/gh-fail" << 'MOCK'
+#!/usr/bin/env bash
+echo "not logged in" >&2
+exit 1
+MOCK
+chmod +x "$mock_dir/gh-fail"
+
+# Source the library under test
+source "$SCRIPT_DIR/lib/identity.sh"
+
+PASS=0
+FAIL=0
+
+assert_eq() {
+  local label="$1" expected="$2" actual="$3"
+  if [[ "$actual" == "$expected" ]]; then
+    PASS=$((PASS + 1)); echo "  PASS: $label"
+  else
+    FAIL=$((FAIL + 1)); echo "  FAIL: $label (expected '$expected', got '$actual')"
+  fi
+}
+
+assert_contains() {
+  local label="$1" expected="$2" actual="$3"
+  if [[ "$actual" == *"$expected"* ]]; then
+    PASS=$((PASS + 1)); echo "  PASS: $label"
+  else
+    FAIL=$((FAIL + 1)); echo "  FAIL: $label (expected to contain '$expected')"
+  fi
+}
+
+assert_exit() {
+  local label="$1" expected="$2" actual="$3"
+  if [[ "$actual" -eq "$expected" ]]; then
+    PASS=$((PASS + 1)); echo "  PASS: $label"
+  else
+    FAIL=$((FAIL + 1)); echo "  FAIL: $label (expected exit $expected, got $actual)"
+  fi
+}
+
+# ── Test 1: get_github_user with mock gh returning "testuser" ────────────
+echo "── Test 1: get_github_user with mock gh ──"
+unset _GITHUB_USER_CACHE
+export GH_CMD="$mock_dir/gh"
+result="$(get_github_user)"
+rc=$?
+assert_eq "returns testuser" "testuser" "$result"
+assert_exit "exit code 0" 0 "$rc"
+
+# ── Test 2: get_github_user with mock gh failing ────────────────────────
+echo ""
+echo "── Test 2: get_github_user with failing gh ──"
+unset _GITHUB_USER_CACHE
+export GH_CMD="$mock_dir/gh-fail"
+result="$(get_github_user 2>/dev/null)" || true
+rc=0
+get_github_user >/dev/null 2>/dev/null || rc=$?
+assert_exit "returns exit 1 on failure" 1 "$rc"
+
+# ── Test 3: team_map_add creates JSONL entry ────────────────────────────
+echo ""
+echo "── Test 3: team_map_add creates parseable JSONL ──"
+export GH_CMD="$mock_dir/gh"
+unset _GITHUB_USER_CACHE
+team_map_add "testuser" "Test User"
+mapfile="$TEAM_MAP_ROOT/.beads/team-map.jsonl"
+# Entry should be parseable JSON
+last_line="$(tail -1 "$mapfile")"
+parsed="$(echo "$last_line" | jq -r '.github')"
+assert_eq "github field is testuser" "testuser" "$parsed"
+display="$(echo "$last_line" | jq -r '.display_name')"
+assert_eq "display_name is Test User" "Test User" "$display"
+is_bot_val="$(echo "$last_line" | jq -r '.is_bot')"
+assert_eq "is_bot is false" "false" "$is_bot_val"
+updated="$(echo "$last_line" | jq -r '.updated_at')"
+assert_contains "updated_at has timestamp" "20" "$updated"
+
+# ── Test 4: team_map_read with LWW resolution ──────────────────────────
+echo ""
+echo "── Test 4: team_map_read with LWW resolution ──"
+# Add a second entry for same user with different display name
+sleep 1  # ensure different timestamp
+team_map_add "testuser" "Updated User"
+# Add another user
+team_map_add "otheruser" "Other"
+result="$(team_map_read)"
+# Should have 2 unique users, with testuser having "Updated User"
+count="$(echo "$result" | jq 'length')"
+assert_eq "2 unique users" "2" "$count"
+testuser_display="$(echo "$result" | jq -r '.[] | select(.github=="testuser") | .display_name')"
+assert_eq "LWW: last entry wins" "Updated User" "$testuser_display"
+
+# ── Test 5: team_map_get "testuser" ─────────────────────────────────────
+echo ""
+echo "── Test 5: team_map_get testuser ──"
+result="$(team_map_get "testuser")"
+gh_val="$(echo "$result" | jq -r '.github')"
+assert_eq "returns correct github" "testuser" "$gh_val"
+
+# ── Test 6: team_map_get "nonexistent" ──────────────────────────────────
+echo ""
+echo "── Test 6: team_map_get nonexistent ──"
+result="$(team_map_get "nonexistent")"
+assert_eq "returns null" "null" "$result"
+
+# ── Test 7: is_bot "dependabot[bot]" ────────────────────────────────────
+echo ""
+echo "── Test 7: is_bot dependabot[bot] ──"
+is_bot "dependabot[bot]"
+rc=$?
+assert_exit "bot detected" 0 "$rc"
+
+# ── Test 8: is_bot "harshanandak" ───────────────────────────────────────
+echo ""
+echo "── Test 8: is_bot harshanandak ──"
+rc=0
+is_bot "harshanandak" || rc=$?
+assert_exit "not a bot" 1 "$rc"
+
+# ── Test 9: auto_detect_identity with mock gh ───────────────────────────
+echo ""
+echo "── Test 9: auto_detect_identity ──"
+# Clean state
+rm -rf "$TEAM_MAP_ROOT/.beads"
+unset _GITHUB_USER_CACHE
+export GH_CMD="$mock_dir/gh"
+auto_detect_identity
+rc=$?
+assert_exit "silent success" 0 "$rc"
+# Verify entry was added
+result="$(team_map_get "testuser")"
+gh_val="$(echo "$result" | jq -r '.github')"
+assert_eq "entry created in JSONL" "testuser" "$gh_val"
+
+# ── Test 10: Invalid username with special chars ────────────────────────
+echo ""
+echo "── Test 10: Invalid username rejected ──"
+rc=0
+team_map_add "bad user!" "Bad" 2>/dev/null || rc=$?
+assert_exit "rejects invalid username" 1 "$rc"
+rc=0
+team_map_add "-startdash" "Bad" 2>/dev/null || rc=$?
+assert_exit "rejects leading hyphen" 1 "$rc"
+rc=0
+team_map_add "has spaces" "Bad" 2>/dev/null || rc=$?
+assert_exit "rejects spaces" 1 "$rc"
+
+# ── Results ──────────────────────────────────────────────────────────────
+echo ""
+echo "Results: $PASS passed, $FAIL failed"
+[[ $FAIL -eq 0 ]] && exit 0 || exit 1