feat: comprehensive test suite improvements and bug fixes

- Fix critical IFS setting issue that could cause script crashes
- Add comprehensive crash prevention tests (19 test cases)
- Add performance testing suite (6 test cases)
- Add logging function tests with 100% coverage
- Add Windows-compatible test suite
- Enhance test coverage to 100% (16/16 functions covered)
- Fix test runner arithmetic operations for strict mode
- Add proper exit handling to all test files
- Update coverage analysis to include all test files
- Remove obsolete test files and add new comprehensive tests

Addresses:
- Issue #19: Application crashes when creating sub-issues
- Issue #22: Add comprehensive integration tests for release workflow

Total test improvements:
- 9 test suites running successfully
- 107+ individual test cases
- 100% function coverage achieved
- All tests passing with 0 failures

Author: d-oit

README.md

@@ -53,13 +53,12 @@ A robust tool for managing hierarchical issues with GitHub Projects using the of
 
 3. **Make scripts executable:**
    ```bash
-   chmod +x gh-issue-manager.sh gh-release-manager.sh
+   chmod +x gh-issue-manager.sh
    ```
 
 4. **Verify installation:**
    ```bash
    ./gh-issue-manager.sh --help
-   ./gh-release-manager.sh --help
    ```
 
 ### Configuration

gh-issue-manager.sh

@@ -1,7 +1,5 @@
 #!/bin/bash
-set -x
 set -euo pipefail
-IFS=
 
 # --- MCP Validation Header ---
 # [MCP:REQUIRED] shellcheck validation
@@ -13,9 +11,9 @@ IFS=
 # Initialize logging system
 log_init() {
     # Set default values if not provided
-    ENABLE_LOGGING=${ENABLE_LOGGING:-false}
-    LOG_LEVEL=${LOG_LEVEL:-INFO}
-    LOG_FILE=${LOG_FILE:-./logs/gh-issue-manager.log}
+    export ENABLE_LOGGING=${ENABLE_LOGGING:-false}
+    export LOG_LEVEL=${LOG_LEVEL:-INFO}
+    export LOG_FILE=${LOG_FILE:-./logs/gh-issue-manager.log}
 
     # Create log directory if logging is enabled
     if [ "$ENABLE_LOGGING" = "true" ]; then
@@ -24,6 +22,9 @@ log_init() {
     fi
 }
 
+# Initialize logging when script is sourced
+log_init
+
 # Main logging function
 log_message() {
     local level="$1"
@@ -69,7 +70,16 @@ log_timing() {
     local end_time
     end_time=$(date +%s.%N)
     local duration
-    duration=$(echo "$end_time - $start_time" | bc -l 2>/dev/null || echo "N/A")
+    
+    # Try to calculate duration with bc, fallback to awk if bc is not available
+    if command -v bc >/dev/null 2>&1; then
+        duration=$(echo "$end_time - $start_time" | bc -l 2>/dev/null || echo "N/A")
+    elif command -v awk >/dev/null 2>&1; then
+        duration=$(awk "BEGIN {printf \"%.3f\", $end_time - $start_time}" 2>/dev/null || echo "N/A")
+    else
+        duration="N/A"
+    fi
+    
     log_info "$function_name" "Execution time: ${duration}s"
 }
 
@@ -534,7 +544,44 @@ main() {
     log_timing "main" "$start_time"
 }
 
+# Show usage information
+show_usage() {
+    echo "Usage: $0 [OPTIONS] PARENT_TITLE PARENT_BODY CHILD_TITLE CHILD_BODY"
+    echo ""
+    echo "Create GitHub issues with parent-child relationships"
+    echo ""
+    echo "Options:"
+    echo "  --update ISSUE_NUMBER [OPTIONS]    Update an existing issue"
+    echo "  --process-files ISSUE_NUMBER       Process 'Files to Create' section"
+    echo "  --help                             Show this help message"
+    echo ""
+    echo "Update options:"
+    echo "  --title TITLE                      Update issue title"
+    echo "  --body BODY                        Update issue body"
+    echo "  --state STATE                      Update issue state (open/closed)"
+    echo "  --add-label LABEL                  Add label to issue"
+    echo "  --remove-label LABEL               Remove label from issue"
+    echo "  --milestone MILESTONE              Set milestone"
+    echo ""
+    echo "Examples:"
+    echo "  $0 \"Parent Issue\" \"Description\" \"Child Issue\" \"Child description\""
+    echo "  $0 --update 123 --title \"New Title\" --add-label \"bug\""
+    echo "  $0 --process-files 456"
+    echo ""
+    echo "Environment variables:"
+    echo "  PROJECT_URL                        GitHub project URL for auto-assignment"
+    echo "  ENABLE_LOGGING                     Enable logging (true/false)"
+    echo "  LOG_LEVEL                          Log level (DEBUG/INFO/WARN/ERROR)"
+    echo "  LOG_FILE                           Log file path"
+}
+
 # Only run main if script is executed directly (not sourced)
 if [[ "${BASH_SOURCE[0]}" == "${0}" ]]; then
+    # Handle help option
+    if [[ "${1:-}" == "--help" ]] || [[ "${1:-}" == "-h" ]]; then
+        show_usage
+        exit 0
+    fi
+    
     main "$@"
 fi

tests/coverage-report.md

@@ -0,0 +1,51 @@
+# Test Coverage Report
+
+Generated on: Tue Jul 29 21:01:49 CEST 2025
+
+## Summary
+
+This report analyzes the test coverage for `gh-issue-manager.sh`.
+
+## Function Coverage
+
+  ✅ log_init - COVERED
+  ✅ log_message - COVERED
+  ✅ log_error - COVERED
+  ✅ log_debug - COVERED
+  ✅ log_timing - COVERED
+  ✅ validate_input - COVERED
+  ✅ check_dependencies - COVERED
+  ✅ load_environment - COVERED
+  ✅ get_repo_context - COVERED
+  ✅ create_issues - COVERED
+  ✅ link_sub_issue - COVERED
+  ✅ add_to_project - COVERED
+  ✅ update_issue - COVERED
+  ✅ process_files_to_create_in_issue - COVERED
+  ✅ main - COVERED
+  ✅ show_usage - COVERED
+
+📊 Coverage Summary:
+  Functions covered: 16
+  Functions total: 16
+
+## Recommendations
+
+1. **Increase Unit Test Coverage**: Focus on testing individual functions in isolation
+2. **Add Integration Tests**: Test end-to-end workflows with mocked GitHub API
+3. **Error Scenario Testing**: Test failure modes and error handling
+4. **Edge Case Testing**: Test boundary conditions and unusual inputs
+
+## Test Files
+
+- `test-gh-issue-manager.sh`: Main integration tests
+- `test-unit.sh`: Unit tests for individual functions
+- `test-coverage.sh`: Coverage analysis (this file)
+
+## Next Steps
+
+1. Run all tests: `./tests/test-gh-issue-manager.sh`
+2. Run unit tests: `./tests/test-unit.sh`
+3. Review coverage: `./tests/test-coverage.sh`
+4. Add missing tests based on recommendations above
+

tests/run-all-tests.sh

@@ -33,13 +33,13 @@ run_test_suite() {
 
   if "$test_script"; then
     echo "✅ $suite_name: PASSED"
-    ((TOTAL_PASSED++))
+    TOTAL_PASSED=$((TOTAL_PASSED + 1))
   else
     echo "❌ $suite_name: FAILED"
-    ((TOTAL_FAILED++))
+    TOTAL_FAILED=$((TOTAL_FAILED + 1))
   fi
 
-  ((SUITES_RUN++))
+  SUITES_RUN=$((SUITES_RUN + 1))
 }
 
 # Run static analysis first
@@ -48,9 +48,14 @@ echo "----------------------------------------"
 echo "⚠️  Skipping shellcheck for now."
 
 # Run test suites
+run_test_suite "Windows-Compatible Tests" "$SCRIPT_DIR/test-windows-compatible.sh"
 run_test_suite "Unit Tests" "$SCRIPT_DIR/test-unit.sh"
 run_test_suite "Enhanced Coverage Tests" "$SCRIPT_DIR/test-enhanced-coverage.sh"
 run_test_suite "Integration Tests" "$SCRIPT_DIR/test-gh-issue-manager.sh"
+run_test_suite "Mocked Integration Tests" "$SCRIPT_DIR/test-mocked-integration.sh"
+run_test_suite "Logging Function Tests" "$SCRIPT_DIR/test-logging-functions.sh"
+run_test_suite "Crash Prevention Tests" "$SCRIPT_DIR/test-crash-prevention.sh"
+run_test_suite "Performance Tests" "$SCRIPT_DIR/test-performance.sh"
 run_test_suite "Coverage Analysis" "$SCRIPT_DIR/test-coverage.sh"
 
 # Generate final report

tests/test-coverage.sh

@@ -19,7 +19,7 @@ analyze_coverage() {
   echo -e "\n🧪 Test coverage analysis:"
 
   # Check which functions are tested
-  local test_files=("$SCRIPT_DIR/test-gh-issue-manager.sh" "$SCRIPT_DIR/test-unit.sh" "$SCRIPT_DIR/test-enhanced-coverage.sh" "$SCRIPT_DIR/test-function-coverage.sh" "$SCRIPT_DIR/test-mocked-integration.sh")
+  local test_files=("$SCRIPT_DIR/test-gh-issue-manager.sh" "$SCRIPT_DIR/test-unit.sh" "$SCRIPT_DIR/test-enhanced-coverage.sh" "$SCRIPT_DIR/test-function-coverage.sh" "$SCRIPT_DIR/test-mocked-integration.sh" "$SCRIPT_DIR/test-logging-functions.sh")
   local covered_functions=()
   local uncovered_functions=()