test: add units tests for hardware details helpers (#1544)

* test: add units tests for hardware details helpers

part of #1496

* test: improve hardware details helpers tests

- Add missing edge cases for hardware details validation
- Improve assertion specificity in test cases
- Remove unused variables from hardware details fixtures
- Refactor test organization by moving tests between classes

---------

Co-authored-by: Gustavo Flores <[email protected]>

Author: LucasSantos27

backend/kernelCI_app/tests/unitTests/helpers/fixtures/hardware_details_data.py

@@ -0,0 +1,235 @@
+from collections import defaultdict
+from kernelCI_app.typeModels.hardwareDetails import Tree
+from kernelCI_app.typeModels.commonDetails import (
+    TestArchSummaryItem,
+    TestSummary,
+)
+from kernelCI_app.typeModels.common import StatusCount
+
+
+def create_tree(**overrides):
+    """Create Tree."""
+    base_tree = Tree(
+        index="1",
+        origin="test",
+        tree_name="mainline",
+        git_repository_branch="master",
+        git_repository_url="https://git.kernel.org",
+        head_git_commit_name="commit1",
+        head_git_commit_hash="abc123",
+        head_git_commit_tag=["v5.4"],
+        selected_commit_status=None,
+        is_selected=True,
+    )
+
+    for key, value in overrides.items():
+        setattr(base_tree, key, value)
+
+    return base_tree
+
+
+def create_tree_status_summary(**overrides):
+    """Create tree status summary."""
+    summary = defaultdict(lambda: defaultdict(lambda: defaultdict(int)))
+
+    summary["1"]["builds"]["PASS"] = 5
+    summary["2"]["builds"]["FAIL"] = 2
+
+    for tree_id, tree_data in overrides.items():
+        for category, status_data in tree_data.items():
+            for status, count in status_data.items():
+                summary[tree_id][category][status] = count
+
+    return summary
+
+
+def create_build_record(**overrides):
+    """Create build record."""
+    base_record = {
+        "build_id": "build123",
+        "status": "PASS",
+        "duration": 200,
+        "start_time": "2024-01-15T09:00:00Z",
+        "architecture": "x86_64",
+        "compiler": "gcc",
+        "config_name": "defconfig",
+        "config_url": "http://config.com",
+        "log_url": "http://log.com",
+        "origin": "test",
+        "command": "make",
+        "comment": "Test build",
+        "tree_name": "mainline",
+        "git_commit_hash": "abc123",
+        "git_commit_name": "commit1",
+        "git_commit_tag": ["v5.4"],
+        "git_repository_url": "https://git.kernel.org",
+        "git_repository_branch": "master",
+    }
+
+    base_record.update(overrides)
+    return base_record
+
+
+def create_test_summary(**overrides):
+    """Create TestSummary."""
+    base_summary = TestSummary(
+        status=StatusCount(DONE=0, PASS=50, FAIL=10, ERROR=2, SKIP=5, MISS=1, NULL=0),
+        origins={
+            "test": StatusCount(
+                DONE=0, PASS=30, FAIL=5, ERROR=1, SKIP=3, MISS=0, NULL=0
+            ),
+            "boot": StatusCount(
+                DONE=0, PASS=20, FAIL=5, ERROR=1, SKIP=2, MISS=1, NULL=0
+            ),
+        },
+        architectures=[
+            TestArchSummaryItem(
+                arch="x86_64",
+                compiler="gcc",
+                status=StatusCount(
+                    DONE=0, PASS=25, FAIL=5, ERROR=1, SKIP=2, MISS=0, NULL=0
+                ),
+            ),
+            TestArchSummaryItem(
+                arch="arm64",
+                compiler="gcc",
+                status=StatusCount(
+                    DONE=0, PASS=25, FAIL=5, ERROR=1, SKIP=3, MISS=1, NULL=0
+                ),
+            ),
+        ],
+        configs={
+            "defconfig": StatusCount(
+                DONE=0, PASS=40, FAIL=8, ERROR=1, SKIP=4, MISS=0, NULL=0
+            ),
+            "allmodconfig": StatusCount(
+                DONE=0, PASS=10, FAIL=2, ERROR=1, SKIP=1, MISS=1, NULL=0
+            ),
+        },
+        issues=[],
+        unknown_issues=0,
+        fail_reasons=defaultdict(
+            int, {"Test failure": 5, "Timeout": 3, "Memory error": 2}
+        ),
+        failed_platforms={"x86_64", "arm64"},
+        environment_compatible={"hardware1": StatusCount()},
+        environment_misc={"x86_64": StatusCount()},
+    )
+
+    for key, value in overrides.items():
+        setattr(base_summary, key, value)
+
+    return base_summary
+
+
+base_tree = create_tree()
+
+tree_with_different_commit = create_tree(
+    index="2",
+    tree_name="stable",
+    git_repository_branch="linux-5.4.y",
+    head_git_commit_name="commit2",
+    head_git_commit_hash="def456",
+    head_git_commit_tag=["v5.4.1"],
+)
+
+base_tree_status_summary = create_tree_status_summary()
+
+
+def create_process_filters_instance(**overrides):
+    """Create instance for process_filters tests."""
+    from unittest.mock import MagicMock
+
+    instance = MagicMock()
+    instance.global_configs = set()
+    instance.global_architectures = set()
+    instance.global_compilers = set()
+    instance.unfiltered_build_issues = set()
+    instance.unfiltered_boot_issues = set()
+    instance.unfiltered_test_issues = set()
+    instance.unfiltered_boot_platforms = set()
+    instance.unfiltered_test_platforms = set()
+    instance.unfiltered_uncategorized_issue_flags = {}
+    instance.unfiltered_origins = {"build": set(), "boot": set(), "test": set()}
+
+    for key, value in overrides.items():
+        setattr(instance, key, value)
+
+    return instance
+
+
+def create_process_filters_record(**overrides):
+    """Create record for process_filters tests."""
+    base_record = {
+        "id": "test123",
+        "path": "test.specific",
+        "status": "PASS",
+        "environment_misc": "{}",
+        "test_origin": "test",
+        "build_id": "build123",
+        "incidents__test_id": "test123",
+        "incidents__issue__id": "test_issue123",
+        "incidents__issue__version": 2,
+        "build__incidents__issue__id": "issue123",
+        "build__incidents__issue__version": 1,
+        "build__status": "PASS",
+        "build__config_name": "defconfig",
+        "build__architecture": "x86_64",
+        "build__compiler": "gcc",
+        "build__origin": "test",
+    }
+
+    base_record.update(overrides)
+    return base_record
+
+
+def create_handle_test_summary_record(**overrides):
+    """Create record for handle_test_summary tests."""
+    base_record = {
+        "status": "PASS",
+        "build__config_name": "defconfig",
+        "build__architecture": "x86_64",
+        "build__compiler": "gcc",
+        "environment_misc": "{}",
+        "misc": "{}",
+        "test_origin": "test",
+    }
+
+    base_record.update(overrides)
+    return base_record
+
+
+process_filters_instance = create_process_filters_instance()
+
+process_filters_instance_without_build = create_process_filters_instance()
+
+process_filters_record_with_build = create_process_filters_record()
+
+process_filters_record_without_build = create_process_filters_record(
+    id=None,
+    build_id=None,
+)
+
+process_filters_record_boot = create_process_filters_record(
+    id="boot123",
+    path="boot.test",
+    test_origin="boot",
+    build_id=None,
+    incidents__issue__id="boot_issue123",
+    incidents__issue__version=1,
+)
+
+
+handle_test_summary_record_new_config = create_handle_test_summary_record(
+    build__config_name="newconfig",
+)
+
+handle_test_summary_record_new_platform = create_handle_test_summary_record(
+    status="FAIL",
+    environment_misc='{"platform": "newplatform"}',
+    misc='{"error_msg": "Test failed"}',
+)
+
+handle_test_summary_record_new_origin = create_handle_test_summary_record(
+    test_origin="neworigin",
+)