feature, test: added new statistics for commit types

Author: n0rfas

git_analytics/analyzers/commit_type.py

@@ -23,30 +23,40 @@ class CommitType(Enum):
 
 @dataclass
 class Result(AnalyticsResult):
-    items: Dict[date, Dict[CommitType, int]]
+    timeseries: Dict[date, Dict[CommitType, int]]
+    total_counter: Counter
+    author_total_counter: Dict[str, Counter]
 
 
 TYPE_COMMIT_LIST: tuple = tuple(ct.value for ct in CommitType)
 
 
 def _get_type_list(commit_message: str):
-    result = [tag for tag in TYPE_COMMIT_LIST if tag in commit_message]
+    result = [tag for tag in TYPE_COMMIT_LIST if tag in commit_message.lower()]
     if result:
         return result
-    return [CommitType.UNKNOWN]
+    return [CommitType.UNKNOWN.value]
 
 
 class CommitTypeAnalyzer(CommitAnalyzer):
     name = "commit_type"
 
     def __init__(self) -> None:
         self._by_date: Dict[date, Counter] = defaultdict(Counter)
+        self._total_counter: Counter = Counter()
+        self._author_total_counter: Dict[str, Counter] = defaultdict(Counter)
 
     def process(self, commit: AnalyticsCommit) -> None:
         commit_date = commit.committed_datetime.date()
         commit_types = _get_type_list(commit.message)
         for commit_type in commit_types:
             self._by_date[commit_date][commit_type] += 1
+            self._total_counter[commit_type] += 1
+            self._author_total_counter[commit.commit_author][commit_type] += 1
 
     def result(self) -> Result:
-        return Result(items={dt: dict(counter) for dt, counter in self._by_date.items()})
+        return Result(
+            timeseries={dt: dict(counter) for dt, counter in self._by_date.items()},
+            total_counter=self._total_counter,
+            author_total_counter=self._author_total_counter,
+        )

tests/test_analyzers/test_commit_type.py

@@ -0,0 +1,117 @@
+import pytest
+
+from datetime import date
+
+from git_analytics.analyzers.commit_type import CommitTypeAnalyzer, _get_type_list
+from tests.fakes import FAKE_COMMITS, FakeCommitSource
+
+
+@pytest.mark.parametrize(
+    "commit_message, expected_types",
+    [
+        ("[JIRA-101] feature: implement checkout page", ["feature"]),
+        ("fix: handle null user session | ABC-77", ["fix"]),
+        ("#452 docs: update README install section", ["docs"]),
+        ("style: reformat css grid | #98", ["style"]),
+        ("refactor: split cart service into modules | PROJ-204", ["refactor"]),
+        ("test: add unit tests for price calculator #311", ["test"]),
+        ("chore: bump dependencies to new version | LIB-55", ["chore"]),
+        ("wip: async payment flow proof-of-concept | #1200", ["wip"]),
+        ("merge: branch 'dev' into 'main' | MERGE-15", ["merge"]),
+        ("unknown: tweak timeout value | OPS-33", ["unknown"]),
+        ("FEATURE: add order history page | #909", ["feature"]),
+        ("[TKT-44] FIX: timezone bug in reports", ["fix"]),
+        ("DOCS: add API usage examples #700", ["docs"]),
+        ("STYLE: lint fixes and import sort | LINT-3", ["fix", "style"]),
+        ("REFACTOR: extract email sender interface | CORE-20", ["refactor"]),
+        ("[QA-12] TEST: integration tests for webhook", ["test"]),
+        ("CHORE: rotate staging secrets | #845", ["chore"]),
+        ("WIP: redesign header navigation | UI-61", ["wip"]),
+        ("[REL-2] MERGE: prepare release branch", ["merge"]),
+        ("UNKNOWN: quick experiment with caching | #321", ["unknown"]),
+        ("PROJ-888: feature add dark mode toggle", ["feature"]),
+        ("fix race condition in job runner | TASK-61", ["fix"]),
+        ("docs add contribution guide | #12", ["docs"]),
+        ("style align buttons on mobile | APP-5", ["style"]),
+        ("refactor | simplify auth middleware #670", ["refactor"]),
+        ("test | snapshot tests for components | FE-42", ["test"]),
+        ("chore cleanup old flags #400", ["chore"]),
+        ("wip migrate to postgres 16 | DB-160", ["wip"]),
+        ("merge revert accidental changes | #2004", ["merge"]),
+        ("unknown minor tweak in env loader | ENV-7", ["unknown"]),
+        ("#1500 | feature: SSO login via OAuth", ["feature"]),
+        ("ABC-321 | fix: N+1 query in orders", ["fix"]),
+        ("#77 | docs: ADR for architecture decisions", ["docs"]),
+        ("XYZ-19 | style: apply prettier defaults", ["style"]),
+        ("#901 | refactor: extract price rules engine", ["refactor"]),
+        ("PAY-24 | test: contract tests for provider", ["test"]),
+        ("#66 | chore: update Makefile targets", ["chore"]),
+        ("OPS-88 | wip: k8s manifests for staging", ["wip"]),
+        ("#555 | merge: sync fork with upstream", ["merge"]),
+        ("CI-11 | unknown: temp debug logs", ["unknown"]),
+        ("feature: add CSV export | REPORTS-14", ["feature"]),
+        ("[BUG-909] fix: off-by-one in pagination", ["fix"]),
+        ("docs: add FAQ section | #740", ["docs"]),
+        ("style: remove dead scss variables #213", ["style"]),
+        ("refactor: replace globals with DI | ARCH-3", ["refactor"]),
+        ("test: e2e checkout happy path | #1010", ["test"]),
+        ("chore: rename default branch to main | REPO-1", ["chore"]),
+        ("wip: multi-tenant groundwork | #602", ["wip"]),
+        ("merge hotfix into release/0.2.0 | REL-020", ["fix", "merge"]),
+        ("add small perf tweak | PERF-9 | unknown", ["unknown"]),
+        ("#120 test, docs: update API spec and add unit tests", ["docs", "test"]),
+        ("fix + refactor: cleanup auth flow and resolve session bug | AUTH-45", ["fix", "refactor"]),
+        ("docs & style: polish README and fix markdown formatting | #77", ["fix", "docs", "style"]),
+        ("[UI-22] test/refactor: split components and adjust snapshots", ["refactor", "test"]),
+        ("feature, docs: implement new search and update user guide | #908", ["feature", "docs"]),
+        ("refactor + chore: restructure configs and remove legacy flags | OPS-12", ["refactor", "chore"]),
+        ("test & fix: cover edge cases and resolve race condition | #311", ["fix", "test"]),
+        ("docs/refactor: move ADRs into separate folder | ARCH-33", ["docs", "refactor"]),
+        ("fix, style: adjust lint rules and resolve formatting bugs | LINT-55", ["fix", "style"]),
+        ("wip + test: draft for analytics module with early unit tests | #700", ["test", "wip"]),
+    ],
+)
+def test_commit_type_get_type_list(commit_message, expected_types):
+    assert _get_type_list(commit_message) == expected_types
+
+
+def test_commit_type_timeseries():
+    source = FakeCommitSource(FAKE_COMMITS)
+
+    analyzer = CommitTypeAnalyzer()
+    for commit in source.iter_commits():
+        analyzer.process(commit)
+    result = analyzer.result()
+
+    assert len(result.timeseries) == 25
+    assert result.timeseries[date(2025, 5, 2)] == {"unknown": 1, "style": 1, "test": 1}
+    assert result.timeseries[date(2025, 4, 2)] == {"unknown": 2, "fix": 1}
+    assert result.timeseries[date(2025, 2, 2)] == {"chore": 1, "refactor": 1, "fix": 1}
+    assert result.timeseries[date(2025, 1, 2)] == {"unknown": 2}
+    assert result.timeseries[date(2025, 1, 1)] == {"unknown": 1}
+    assert result.timeseries[date(2024, 9, 2)] == {"unknown": 4, "fix": 1, "refactor": 1}
+    assert result.timeseries[date(2024, 1, 1)] == {"unknown": 4}
+
+
+def test_commit_type_total_counter():
+    source = FakeCommitSource(FAKE_COMMITS)
+
+    analyzer = CommitTypeAnalyzer()
+    for commit in source.iter_commits():
+        analyzer.process(commit)
+    result = analyzer.result()
+
+    assert sum(result.total_counter.values()) == 40
+    assert result.total_counter == {"unknown": 23, "refactor": 5, "fix": 5, "chore": 3, "style": 2, "test": 2}
+
+
+def test_commit_type_author_total_counter():
+    source = FakeCommitSource(FAKE_COMMITS)
+
+    analyzer = CommitTypeAnalyzer()
+    for commit in source.iter_commits():
+        analyzer.process(commit)
+    result = analyzer.result()
+
+    assert result.author_total_counter.keys() == {"Alice", "Bob", "Carol", "Dave", "Oscar"}
+    assert sum(sum(v.values()) for v in result.author_total_counter.values()) == 40