@theihtisham/dev-pulse 1.0.0

package/tests/conftest.py

@@ -0,0 +1,208 @@
+"""Shared test fixtures for DevPulse tests."""
+
+import os
+import tempfile
+import pytest
+
+from devpulse.core.database import Database
+from devpulse.core.config import reset_settings
+
+
+@pytest.fixture(autouse=True)
+def isolated_environment(tmp_path, monkeypatch):
+    """Ensure each test uses an isolated database and config."""
+    db_path = str(tmp_path / "test_devpulse.db")
+    reports_dir = str(tmp_path / "reports")
+    export_dir = str(tmp_path / "exports")
+
+    monkeypatch.setenv("DEVPULSE_DATABASE_PATH", db_path)
+    monkeypatch.setenv("DEVPULSE_REPORTS_DIR", reports_dir)
+    monkeypatch.setenv("DEVPULSE_EXPORT_DIR", export_dir)
+    monkeypatch.setenv("DEVPULSE_GITHUB_TOKEN", "test_token_12345")
+    monkeypatch.setenv("DEVPULSE_GITHUB_USERNAME", "testuser")
+
+    reset_settings()
+    yield
+    reset_settings()
+
+
+@pytest.fixture
+def db(isolated_environment) -> Database:
+    """Provide a fresh database instance."""
+    from devpulse.core.config import get_settings
+    settings = get_settings()
+    return Database(db_path=settings.database_path)
+
+
+@pytest.fixture
+def sample_commits():
+    """Sample commit data for testing."""
+    return [
+        {
+            "sha": "abc1234567890",
+            "repo": "testorg/testrepo",
+            "author": "Alice",
+            "author_date": "2026-04-01T10:00:00Z",
+            "message": "Add new feature",
+            "additions": 50,
+            "deletions": 10,
+            "url": "https://github.com/testorg/testrepo/commit/abc1234567890",
+        },
+        {
+            "sha": "def1234567890",
+            "repo": "testorg/testrepo",
+            "author": "Alice",
+            "author_date": "2026-04-01T14:00:00Z",
+            "message": "Fix bug in feature",
+            "additions": 20,
+            "deletions": 5,
+            "url": "https://github.com/testorg/testrepo/commit/def1234567890",
+        },
+        {
+            "sha": "ghi1234567890",
+            "repo": "testorg/testrepo",
+            "author": "Bob",
+            "author_date": "2026-04-02T09:00:00Z",
+            "message": "Update docs",
+            "additions": 30,
+            "deletions": 0,
+            "url": "https://github.com/testorg/testrepo/commit/ghi1234567890",
+        },
+        {
+            "sha": "jkl1234567890",
+            "repo": "testorg/testrepo",
+            "author": "Alice",
+            "author_date": "2026-04-03T11:00:00Z",
+            "message": "Refactor module",
+            "additions": 100,
+            "deletions": 80,
+            "url": "https://github.com/testorg/testrepo/commit/jkl1234567890",
+        },
+    ]
+
+
+@pytest.fixture
+def sample_prs():
+    """Sample pull request data for testing."""
+    return [
+        {
+            "number": 1,
+            "repo": "testorg/testrepo",
+            "title": "Add new feature",
+            "author": "Alice",
+            "state": "merged",
+            "created_at": "2026-04-01T10:00:00Z",
+            "merged_at": "2026-04-02T10:00:00Z",
+            "closed_at": "2026-04-02T10:00:00Z",
+            "additions": 200,
+            "deletions": 50,
+            "changed_files": 5,
+            "review_comments": 3,
+            "url": "https://github.com/testorg/testrepo/pull/1",
+        },
+        {
+            "number": 2,
+            "repo": "testorg/testrepo",
+            "title": "Fix critical bug",
+            "author": "Bob",
+            "state": "merged",
+            "created_at": "2026-04-01T08:00:00Z",
+            "merged_at": "2026-04-01T20:00:00Z",
+            "closed_at": "2026-04-01T20:00:00Z",
+            "additions": 30,
+            "deletions": 10,
+            "changed_files": 2,
+            "review_comments": 5,
+            "url": "https://github.com/testorg/testrepo/pull/2",
+        },
+        {
+            "number": 3,
+            "repo": "testorg/testrepo",
+            "title": "WIP: big refactor",
+            "author": "Alice",
+            "state": "open",
+            "created_at": "2026-04-03T09:00:00Z",
+            "merged_at": None,
+            "closed_at": None,
+            "additions": 500,
+            "deletions": 300,
+            "changed_files": 15,
+            "review_comments": 0,
+            "url": "https://github.com/testorg/testrepo/pull/3",
+        },
+    ]
+
+
+@pytest.fixture
+def sample_issues():
+    """Sample issue data for testing."""
+    return [
+        {
+            "number": 10,
+            "repo": "testorg/testrepo",
+            "title": "Bug: crash on startup",
+            "author": "Bob",
+            "state": "closed",
+            "labels": ["bug"],
+            "created_at": "2026-03-25T10:00:00Z",
+            "closed_at": "2026-04-01T14:00:00Z",
+            "url": "https://github.com/testorg/testrepo/issues/10",
+        },
+        {
+            "number": 11,
+            "repo": "testorg/testrepo",
+            "title": "Feature: add export",
+            "author": "Alice",
+            "state": "open",
+            "labels": ["enhancement"],
+            "created_at": "2026-04-02T08:00:00Z",
+            "closed_at": None,
+            "url": "https://github.com/testorg/testrepo/issues/11",
+        },
+        {
+            "number": 12,
+            "repo": "testorg/testrepo",
+            "title": "Bug: memory leak",
+            "author": "Charlie",
+            "state": "open",
+            "labels": ["bug", "priority:high"],
+            "created_at": "2026-04-03T15:00:00Z",
+            "closed_at": None,
+            "url": "https://github.com/testorg/testrepo/issues/12",
+        },
+    ]
+
+
+@pytest.fixture
+def sample_reviews():
+    """Sample review data for testing."""
+    return [
+        {
+            "id": 100,
+            "repo": "testorg/testrepo",
+            "pr_number": 1,
+            "author": "Bob",
+            "state": "APPROVED",
+            "submitted_at": "2026-04-01T18:00:00Z",
+            "body": "Looks good!",
+        },
+        {
+            "id": 101,
+            "repo": "testorg/testrepo",
+            "pr_number": 2,
+            "author": "Alice",
+            "state": "CHANGES_REQUESTED",
+            "submitted_at": "2026-04-01T12:00:00Z",
+            "body": "Please add tests.",
+        },
+    ]
+
+
+@pytest.fixture
+def populated_db(db, sample_commits, sample_prs, sample_issues, sample_reviews):
+    """Database pre-populated with sample data."""
+    db.upsert_commits(sample_commits)
+    db.upsert_pull_requests(sample_prs)
+    db.upsert_issues(sample_issues)
+    db.upsert_reviews(sample_reviews)
+    return db

package/tests/test_analytics.py

@@ -0,0 +1,284 @@
+"""Tests for the AnalyticsEngine."""
+
+import pytest
+
+from devpulse.core.analytics import AnalyticsEngine, _parse_date, _days_between
+from devpulse.core.models import DeveloperMetrics, TeamHealth
+
+
+class TestParseDate:
+    """Test date parsing utility."""
+
+    def test_iso_with_tz(self):
+        dt = _parse_date("2026-04-01T10:00:00Z")
+        assert dt is not None
+        assert dt.year == 2026
+        assert dt.month == 4
+        assert dt.day == 1
+
+    def test_iso_without_tz(self):
+        dt = _parse_date("2026-04-01T10:00:00")
+        assert dt is not None
+
+    def test_date_only(self):
+        dt = _parse_date("2026-04-01")
+        assert dt is not None
+
+    def test_empty_string(self):
+        assert _parse_date("") is None
+
+    def test_none(self):
+        assert _parse_date(None) is None
+
+    def test_invalid_format(self):
+        assert _parse_date("not-a-date") is None
+
+
+class TestDaysBetween:
+    """Test time difference computation."""
+
+    def test_same_day(self):
+        hours = _days_between("2026-04-01T10:00:00Z", "2026-04-01T14:00:00Z")
+        assert hours == 4.0
+
+    def test_24_hours(self):
+        hours = _days_between("2026-04-01T00:00:00Z", "2026-04-02T00:00:00Z")
+        assert hours == 24.0
+
+    def test_empty_start(self):
+        hours = _days_between("", "2026-04-01T10:00:00Z")
+        assert hours == 0.0
+
+
+class TestDeveloperMetrics:
+    """Test developer metrics computation."""
+
+    def test_basic_metrics(self, populated_db):
+        engine = AnalyticsEngine(db=populated_db)
+        metrics = engine.developer_metrics(author="Alice", days=30)
+
+        assert isinstance(metrics, DeveloperMetrics)
+        assert metrics.author == "Alice"
+        assert metrics.commits_count == 3
+        assert metrics.prs_created == 2
+        assert metrics.prs_merged == 1  # Only PR #1 is merged and authored by Alice
+
+    def test_bob_metrics(self, populated_db):
+        engine = AnalyticsEngine(db=populated_db)
+        metrics = engine.developer_metrics(author="Bob", days=30)
+
+        assert metrics.author == "Bob"
+        assert metrics.commits_count == 1
+        assert metrics.prs_created == 1
+
+    def test_unknown_developer(self, populated_db):
+        engine = AnalyticsEngine(db=populated_db)
+        metrics = engine.developer_metrics(author="Nobody", days=30)
+
+        assert metrics.commits_count == 0
+        assert metrics.author == "Nobody"
+
+    def test_active_days(self, populated_db):
+        engine = AnalyticsEngine(db=populated_db)
+        metrics = engine.developer_metrics(author="Alice", days=30)
+
+        assert metrics.active_days == 2  # April 1 and April 3
+
+    def test_commits_per_day(self, populated_db):
+        engine = AnalyticsEngine(db=populated_db)
+        metrics = engine.developer_metrics(author="Alice", days=30)
+
+        assert metrics.commits_per_day > 0
+        assert metrics.period_days == 30
+
+
+class TestTeamMetrics:
+    """Test team-wide metrics."""
+
+    def test_team_metrics_list(self, populated_db):
+        engine = AnalyticsEngine(db=populated_db)
+        team = engine.team_metrics(days=30)
+
+        assert len(team) >= 2  # At least Alice and Bob
+        authors = {m.author for m in team}
+        assert "Alice" in authors
+        assert "Bob" in authors
+
+    def test_empty_team(self, db):
+        engine = AnalyticsEngine(db=db)
+        team = engine.team_metrics(days=30)
+        assert team == []
+
+
+class TestTeamHealth:
+    """Test team health analysis."""
+
+    def test_team_health_with_data(self, populated_db):
+        engine = AnalyticsEngine(db=populated_db)
+        health = engine.team_health(days=30)
+
+        assert isinstance(health, TeamHealth)
+        assert 0 <= health.overall_score <= 100
+        assert 0 <= health.workload_balance <= 1
+        assert 0 <= health.collaboration_score <= 1
+        assert health.velocity_trend in ("growing", "stable", "declining")
+        assert len(health.recommendations) > 0
+
+    def test_team_health_empty(self, db):
+        engine = AnalyticsEngine(db=db)
+        health = engine.team_health(days=30)
+
+        assert health.overall_score == 0
+        assert len(health.recommendations) > 0  # Should have "no data" recommendation
+
+
+class TestSprintAnalytics:
+    """Test sprint analytics."""
+
+    def test_sprint_burndown(self, db):
+        # Record multiple snapshots
+        for i, completed in enumerate([0, 5, 10, 15, 18]):
+            db.save_sprint_snapshot({
+                "sprint_name": "Sprint 1",
+                "total_points": 20,
+                "completed_points": completed,
+                "remaining_points": 20 - completed,
+                "added_points": 0,
+            })
+
+        engine = AnalyticsEngine(db=db)
+        data = engine.sprint_burndown("Sprint 1")
+        assert len(data) == 5
+        assert data[0]["remaining"] == 20
+        assert data[-1]["remaining"] == 2
+
+    def test_sprint_burndown_nonexistent(self, populated_db):
+        engine = AnalyticsEngine(db=populated_db)
+        data = engine.sprint_burndown("Nonexistent Sprint")
+        assert data == []
+
+    def test_predict_sprint_completion(self, db):
+        for i, completed in enumerate([0, 5, 10, 15]):
+            db.save_sprint_snapshot({
+                "sprint_name": "Sprint X",
+                "total_points": 20,
+                "completed_points": completed,
+                "remaining_points": 20 - completed,
+            })
+
+        engine = AnalyticsEngine(db=db)
+        result = engine.predict_sprint_completion("Sprint X", total_points=20, days_elapsed=4, total_days=10)
+        assert result["prediction"] in ("on_track", "at_risk", "unknown")
+        assert "message" in result
+
+    def test_predict_nonexistent_sprint(self, populated_db):
+        engine = AnalyticsEngine(db=populated_db)
+        result = engine.predict_sprint_completion("Ghost Sprint", 20, 5, 10)
+        assert result["prediction"] == "unknown"
+
+
+class TestCodeQuality:
+    """Test code quality analytics."""
+
+    def test_quality_trend(self, db):
+        db.save_quality_snapshot({
+            "repo": "testorg/testrepo",
+            "test_coverage": 0.75,
+            "open_bugs": 5,
+            "tech_debt_score": 3.5,
+        })
+        db.save_quality_snapshot({
+            "repo": "testorg/testrepo",
+            "test_coverage": 0.80,
+            "open_bugs": 3,
+            "tech_debt_score": 3.0,
+        })
+
+        engine = AnalyticsEngine(db=db)
+        trends = engine.code_quality_trend(repo="testorg/testrepo")
+        assert len(trends) == 2
+        assert trends[0].test_coverage == 0.75
+        assert trends[1].test_coverage == 0.80
+
+    def test_compute_quality_score(self, populated_db):
+        engine = AnalyticsEngine(db=populated_db)
+        score = engine.compute_quality_score(repo="testorg/testrepo", days=30)
+
+        assert score["repo"] == "testorg/testrepo"
+        assert score["total_commits"] == 4
+        assert score["total_prs"] == 3
+        assert isinstance(score["tech_debt_score"], float)
+
+
+class TestInsights:
+    """Test AI insight generation."""
+
+    def test_generate_insights_with_data(self, populated_db):
+        engine = AnalyticsEngine(db=populated_db)
+        insights = engine.generate_insights(days=30)
+
+        assert isinstance(insights, list)
+        assert len(insights) > 0
+        for insight in insights:
+            assert "type" in insight
+            assert "severity" in insight
+            assert "message" in insight
+            assert "recommendation" in insight
+
+    def test_generate_insights_empty(self, db):
+        engine = AnalyticsEngine(db=db)
+        insights = engine.generate_insights(days=30)
+
+        assert len(insights) > 0
+        # Should have a "no data" or default insight
+
+    def test_insights_include_merge_time(self, populated_db):
+        engine = AnalyticsEngine(db=populated_db)
+        insights = engine.generate_insights(days=30, repo="testorg/testrepo")
+
+        messages = " ".join(i["message"] for i in insights)
+        assert "merge time" in messages.lower() or "healthy" in messages.lower()
+
+
+class TestGoalCoaching:
+    """Test goal coaching."""
+
+    def test_coaching_active_goals(self, db):
+        db.upsert_goal({
+            "title": "Commits per day",
+            "target_value": 10,
+            "current_value": 7,
+            "metric": "commits_per_day",
+        })
+
+        engine = AnalyticsEngine(db=db)
+        coaching = engine.goal_coaching()
+
+        assert len(coaching) == 1
+        assert coaching[0]["progress_pct"] == 70.0
+        assert coaching[0]["status"] in ("on_track", "at_risk", "achieved")
+
+    def test_coaching_achieved_goal(self, db):
+        db.upsert_goal({
+            "title": "Write code",
+            "target_value": 1,
+            "current_value": 5,
+            "metric": "commits",
+        })
+
+        engine = AnalyticsEngine(db=db)
+        coaching = engine.goal_coaching()
+
+        assert coaching[0]["status"] == "achieved"
+
+    def test_coaching_specific_goal(self, db):
+        gid = db.upsert_goal({
+            "title": "Reviews",
+            "target_value": 5,
+            "current_value": 1,
+            "metric": "reviews",
+        })
+
+        engine = AnalyticsEngine(db=db)
+        coaching = engine.goal_coaching(goal_id=gid)
+        assert len(coaching) == 1