make check

Author: mrkeshav-05

backend/tests/apps/owasp/management/commands/owasp_aggregate_contributions_test.py

@@ -59,7 +59,9 @@ def mock_chapter(self):
         chapter.owasp_repository = mock.Mock()
         chapter.owasp_repository.id = 1
         # Fix Django ORM compatibility
-        chapter.owasp_repository.resolve_expression = mock.Mock(return_value=chapter.owasp_repository)
+        chapter.owasp_repository.resolve_expression = mock.Mock(
+            return_value=chapter.owasp_repository
+        )
         chapter.owasp_repository.get_source_expressions = mock.Mock(return_value=[])
         return chapter
 
@@ -71,18 +73,20 @@ def mock_project(self):
         project.owasp_repository = mock.Mock()
         project.owasp_repository.id = 1
         # Fix Django ORM compatibility
-        project.owasp_repository.resolve_expression = mock.Mock(return_value=project.owasp_repository)
+        project.owasp_repository.resolve_expression = mock.Mock(
+            return_value=project.owasp_repository
+        )
         project.owasp_repository.get_source_expressions = mock.Mock(return_value=[])
-        
+
         # Mock additional repositories
         additional_repo1 = mock.Mock(id=2)
         additional_repo1.resolve_expression = mock.Mock(return_value=additional_repo1)
         additional_repo1.get_source_expressions = mock.Mock(return_value=[])
-        
+
         additional_repo2 = mock.Mock(id=3)
         additional_repo2.resolve_expression = mock.Mock(return_value=additional_repo2)
         additional_repo2.get_source_expressions = mock.Mock(return_value=[])
-        
+
         project.repositories.all.return_value = [additional_repo1, additional_repo2]
         return project
 
@@ -212,11 +216,20 @@ def test_aggregate_project_without_repositories(self, command, mock_project):
     @mock.patch("apps.owasp.management.commands.owasp_aggregate_contributions.Issue")
     @mock.patch("apps.owasp.management.commands.owasp_aggregate_contributions.PullRequest")
     @mock.patch("apps.owasp.management.commands.owasp_aggregate_contributions.Release")
-    def test_handle_chapters_only(self, mock_release, mock_pr, mock_issue, mock_commit, mock_chapter_model, command, mock_chapter):
+    def test_handle_chapters_only(
+        self,
+        mock_release,
+        mock_pr,
+        mock_issue,
+        mock_commit,
+        mock_chapter_model,
+        command,
+        mock_chapter,
+    ):
         """Test command execution for chapters only."""
         mock_chapter_model.objects.filter.return_value = MockQuerySet([mock_chapter])
         mock_chapter_model.bulk_save = mock.Mock()
-        
+
         # Mock ORM queries to return counts
         mock_commit.objects.filter.return_value.count.return_value = 5
         mock_issue.objects.filter.return_value.count.return_value = 3
@@ -238,11 +251,20 @@ def test_handle_chapters_only(self, mock_release, mock_pr, mock_issue, mock_comm
     @mock.patch("apps.owasp.management.commands.owasp_aggregate_contributions.Issue")
     @mock.patch("apps.owasp.management.commands.owasp_aggregate_contributions.PullRequest")
     @mock.patch("apps.owasp.management.commands.owasp_aggregate_contributions.Release")
-    def test_handle_projects_only(self, mock_release, mock_pr, mock_issue, mock_commit, mock_project_model, command, mock_project):
+    def test_handle_projects_only(
+        self,
+        mock_release,
+        mock_pr,
+        mock_issue,
+        mock_commit,
+        mock_project_model,
+        command,
+        mock_project,
+    ):
         """Test command execution for projects only."""
         mock_project_model.objects.filter.return_value = MockQuerySet([mock_project])
         mock_project_model.bulk_save = mock.Mock()
-        
+
         # Mock ORM queries to return counts
         mock_commit.objects.filter.return_value.count.return_value = 8
         mock_issue.objects.filter.return_value.count.return_value = 4
@@ -282,7 +304,7 @@ def test_handle_both_entities(
         mock_project_model.objects.filter.return_value = MockQuerySet([mock_project])
         mock_chapter_model.bulk_save = mock.Mock()
         mock_project_model.bulk_save = mock.Mock()
-        
+
         # Mock ORM queries to return counts
         mock_commit.objects.filter.return_value.count.return_value = 5
         mock_issue.objects.filter.return_value.count.return_value = 3
@@ -311,11 +333,20 @@ def test_handle_both_entities(
     @mock.patch("apps.owasp.management.commands.owasp_aggregate_contributions.Issue")
     @mock.patch("apps.owasp.management.commands.owasp_aggregate_contributions.PullRequest")
     @mock.patch("apps.owasp.management.commands.owasp_aggregate_contributions.Release")
-    def test_handle_with_specific_key(self, mock_release, mock_pr, mock_issue, mock_commit, mock_chapter_model, command, mock_chapter):
+    def test_handle_with_specific_key(
+        self,
+        mock_release,
+        mock_pr,
+        mock_issue,
+        mock_commit,
+        mock_chapter_model,
+        command,
+        mock_chapter,
+    ):
         """Test command execution with a specific entity key."""
         mock_chapter_model.objects.filter.return_value = MockQuerySet([mock_chapter])
         mock_chapter_model.bulk_save = mock.Mock()
-        
+
         # Mock ORM queries to return counts
         mock_commit.objects.filter.return_value.count.return_value = 3
         mock_issue.objects.filter.return_value.count.return_value = 2
@@ -337,12 +368,21 @@ def test_handle_with_specific_key(self, mock_release, mock_pr, mock_issue, mock_
     @mock.patch("apps.owasp.management.commands.owasp_aggregate_contributions.Issue")
     @mock.patch("apps.owasp.management.commands.owasp_aggregate_contributions.PullRequest")
     @mock.patch("apps.owasp.management.commands.owasp_aggregate_contributions.Release")
-    def test_handle_with_offset(self, mock_release, mock_pr, mock_issue, mock_commit, mock_chapter_model, command, mock_chapter):
+    def test_handle_with_offset(
+        self,
+        mock_release,
+        mock_pr,
+        mock_issue,
+        mock_commit,
+        mock_chapter_model,
+        command,
+        mock_chapter,
+    ):
         """Test command execution with offset parameter."""
         chapters = [mock_chapter, mock_chapter, mock_chapter]
         mock_chapter_model.objects.filter.return_value = MockQuerySet(chapters)
         mock_chapter_model.bulk_save = mock.Mock()
-        
+
         # Mock ORM queries to return counts
         mock_commit.objects.filter.return_value.count.return_value = 1
         mock_issue.objects.filter.return_value.count.return_value = 1
@@ -365,11 +405,20 @@ def test_handle_with_offset(self, mock_release, mock_pr, mock_issue, mock_commit
     @mock.patch("apps.owasp.management.commands.owasp_aggregate_contributions.Issue")
     @mock.patch("apps.owasp.management.commands.owasp_aggregate_contributions.PullRequest")
     @mock.patch("apps.owasp.management.commands.owasp_aggregate_contributions.Release")
-    def test_handle_custom_days(self, mock_release, mock_pr, mock_issue, mock_commit, mock_chapter_model, command, mock_chapter):
+    def test_handle_custom_days(
+        self,
+        mock_release,
+        mock_pr,
+        mock_issue,
+        mock_commit,
+        mock_chapter_model,
+        command,
+        mock_chapter,
+    ):
         """Test command execution with custom days parameter."""
         mock_chapter_model.objects.filter.return_value = MockQuerySet([mock_chapter])
         mock_chapter_model.bulk_save = mock.Mock()
-        
+
         # Mock ORM queries to return counts
         mock_commit.objects.filter.return_value.count.return_value = 0
         mock_issue.objects.filter.return_value.count.return_value = 0

frontend/src/components/ContributionHeatmap.tsx

@@ -22,11 +22,11 @@ const generateHeatmapSeries = (
       contributionData
     )
   }
-  
+
   const start = new Date(startDate)
   const end = new Date(endDate)
 
-  // Handle invalid dates by using default range  
+  // Handle invalid dates by using default range
   if (Number.isNaN(start.getTime()) || Number.isNaN(end.getTime())) {
     const defaultEnd = new Date()
     const defaultStart = new Date()
@@ -37,7 +37,7 @@ const generateHeatmapSeries = (
       contributionData
     )
   }
-  
+
   // Handle invalid range by swapping dates
   if (start > end) {
     return generateHeatmapSeries(endDate, startDate, contributionData)