Initialize testing infrastructure with conftest and pytest configuration, remove obsolete hello.py file, and add test files and fixtures for commit message generation.

Author: MikeBirdTech

conftest.py

@@ -0,0 +1,9 @@
+"""Root conftest file to set up the Python path for testing."""
+
+import os
+import sys
+from pathlib import Path
+
+# Add the src directory to the Python path
+root_dir = Path(__file__).parent.absolute()
+sys.path.insert(0, str(root_dir / "src"))

pytest.ini

@@ -0,0 +1,7 @@
+[pytest]
+# Fail tests immediately if they try to get user input
+mock_use_standalone_module = true
+# Show more details about test failures
+addopts = -v
+# Timeout after 5 seconds (to prevent hanging on user input)
+timeout = 5 

src/blueprint/hello.py

@@ -0,0 +1,57 @@
+# Blueprint Tests
+
+This directory contains tests for the Blueprint project.
+
+## Structure
+
+- `conftest.py`: Contains pytest fixtures and shared test configuration.
+- `unit/`: Contains unit tests for specific modules.
+
+## Running Tests
+
+Run all tests using pytest:
+
+```bash
+# From project root
+pytest
+
+# Run with verbose output
+pytest -v
+
+# Run specific test file
+pytest tests/unit/test_commit_generator.py
+```
+
+## Test Fixtures
+
+The `conftest.py` file provides several fixtures that are useful for testing:
+
+- `sample_git_diff`: A sample git diff for testing diff parsing functionality.
+- `sample_ai_response`: A sample AI service response for testing commit message generation.
+
+## Writing Tests
+
+When adding new tests:
+
+1. Use the pytest fixture system where appropriate.
+2. Mock all external dependencies (subprocess, network calls, etc.).
+3. Follow the Arrange-Act-Assert (AAA) pattern in test methods.
+4. Add docstrings to test functions that explain what's being tested.
+
+## Test Coverage
+
+To generate a test coverage report:
+
+```bash
+# From project root
+pytest --cov=blueprint tests/
+
+# Generate HTML report
+pytest --cov=blueprint --cov-report=html tests/
+```
+
+## Mocking Strategy
+
+- Use `unittest.mock.patch` to mock dependencies like subprocess calls and AIService.
+- When testing functions that call git commands, mock subprocess functions.
+- For AI service interactions, mock the AIService class and its methods.

tests/README.md

@@ -0,0 +1,40 @@
+"""Pytest configuration for Blueprint tests."""
+
+import pytest
+
+
+@pytest.fixture
+def sample_git_diff():
+    """Return a sample git diff for testing."""
+    return """diff --git a/src/blueprint/commit_generator.py b/src/blueprint/commit_generator.py
+index 123456..789abc 100644
+--- a/src/blueprint/commit_generator.py
++++ b/src/blueprint/commit_generator.py
+@@ -10,7 +10,7 @@ def get_git_diff(max_chars: int = 5000) -> str:
+     \"\"\"Get the git diff of staged changes, or unstaged if no staged changes.\"\"\"
+     try:
+         diff = subprocess.check_output(["git", "diff", "--cached"], text=True)
+-        if not diff.strip():
++        if not diff:
+             diff = subprocess.check_output(["git", "diff"], text=True)
+         return diff[:max_chars]  # Limit to max_chars characters
+     except subprocess.CalledProcessError:
+@@ -50,7 +50,7 @@ def parse_commit_messages(response: str) -> List[str]:
+     messages = []
+     for line in response.split("\\n"):
+         if line.strip().startswith(("1.", "2.", "3.")):
+-            messages.append(line.split(".", 1)[1])
++            messages.append(line.split(".", 1)[1].strip())
+     return messages"""
+
+
+@pytest.fixture
+def sample_ai_response():
+    """Return a sample AI response for testing."""
+    return """Here are three commit message options for your changes:
+
+1. Remove strip() from diff check and add strip() to parsed commit messages
+2. Fix string handling in git diff and commit message parsing
+3. Improve robustness of git diff checking and commit message parsing
+
+These commit messages reflect the changes where you removed a strip() call in the git diff check and added one in the commit message parser."""

tests/conftest.py

@@ -0,0 +1,53 @@
+#!/bin/bash
+# Helper script to run tests with coverage
+
+# Ensure we're in the project root directory
+cd "$(dirname "$0")/.."
+
+# Colors for terminal output
+GREEN='\033[0;32m'
+YELLOW='\033[1;33m'
+RED='\033[0;31m'
+NC='\033[0m' # No Color
+
+echo -e "${YELLOW}Running tests with coverage...${NC}"
+
+# Check if pytest-cov and pytest-timeout are installed
+if ! python -c "import pytest_cov" 2>/dev/null; then
+    echo "Installing pytest-cov..."
+    uv pip install pytest-cov
+fi
+
+if ! python -c "import pytest_timeout" 2>/dev/null; then
+    echo "Installing pytest-timeout..."
+    uv pip install pytest-timeout
+fi
+
+# Create or update pytest.ini to ensure non-interactive tests
+cat > pytest.ini << EOF
+[pytest]
+# Fail tests immediately if they try to get user input
+mock_use_standalone_module = true
+# Show more details about test failures
+addopts = -v
+# Timeout after 5 seconds (to prevent hanging on user input)
+timeout = 5
+EOF
+
+# Run tests with coverage and generate HTML report
+# Set PYTHONPATH to ensure blueprint module can be found
+PYTHONPATH=./src python -m pytest tests/ --cov=blueprint --cov-report=term --cov-report=html
+
+# Check exit status
+if [ $? -eq 0 ]; then
+    echo -e "${GREEN}Tests passed!${NC}"
+    echo -e "HTML coverage report generated in htmlcov/"
+    
+    # Open coverage report on macOS
+    if [[ "$OSTYPE" == "darwin"* ]]; then
+        open htmlcov/index.html
+    fi
+else
+    echo -e "${RED}Tests failed.${NC}"
+    echo -e "${YELLOW}If tests are hanging or asking for input, check for non-mocked interactive components.${NC}"
+fi