removed indexing from repositories and projects that do not belong to any organization

backend/apps/github/index/registry/repository.py

@@ -72,6 +72,7 @@ def get_entities(self) -> QuerySet:
         """
         return Repository.objects.filter(
             is_template=False,
+            organization__isnull=False,
         ).prefetch_related(
             "repositorycontributor_set",
         )

backend/apps/owasp/index/registry/project.py

@@ -105,7 +105,11 @@ def update_synonyms():
 
     def get_entities(self):
         """Get entities for indexing."""
-        return Project.objects.prefetch_related(
-            "organizations",
-            "repositories",
+        return (
+            Project.objects.prefetch_related(
+                "organizations",
+                "repositories",
+            )
+            .filter(organizations__isnull=False)
+            .distinct()
         )

backend/tests/apps/github/index/registry/repository_test.py

@@ -40,7 +40,9 @@ def test_get_entities(self, repository_index):
         with patch.object(Repository, "objects", mock_manager):
             queryset = repository_index.get_entities()
 
-            mock_manager.filter.assert_called_once_with(is_template=False)
+            mock_manager.filter.assert_called_once_with(
+                is_template=False, organization__isnull=False
+            )
             mock_manager.filter.return_value.prefetch_related.assert_called_once_with(
                 "repositorycontributor_set"
             )

backend/tests/apps/owasp/index/registry/project_test.py

@@ -0,0 +1,58 @@
+from unittest.mock import MagicMock, patch
+
+import pytest
+
+from apps.owasp.index.registry.project import ProjectIndex
+from apps.owasp.models.project import Project
+
+
+@pytest.fixture
+def project_index(mocker):
+    """Return an instance of the ProjectIndex."""
+    mocker.patch("apps.common.index.IndexBase.__init__", return_value=None)
+    return ProjectIndex()
+
+
+class TestProjectIndex:
+    def test_class_attributes(self):
+        """Test that the basic class attributes are set correctly."""
+        assert ProjectIndex.index_name == "projects"
+        assert ProjectIndex.should_index == "is_indexable"
+        assert isinstance(ProjectIndex.fields, tuple)
+        assert len(ProjectIndex.fields) > 0
+        assert isinstance(ProjectIndex.settings, dict)
+        assert "attributesForFaceting" in ProjectIndex.settings
+
+    @patch("apps.common.index.IndexBase.configure_replicas")
+    def test_configure_replicas(self, mock_configure_replicas):
+        """Test that configure_replicas calls the parent method with correct args."""
+        ProjectIndex.configure_replicas()
+        assert mock_configure_replicas.call_count == 1
+        assert mock_configure_replicas.call_args[0][0] == "projects"
+        assert isinstance(mock_configure_replicas.call_args[0][1], dict)
+
+    @patch("apps.common.index.IndexBase.reindex_synonyms")
+    def test_update_synonyms(self, mock_reindex_synonyms):
+        """Test that update_synonyms calls the parent method with correct args."""
+        ProjectIndex.update_synonyms()
+        mock_reindex_synonyms.assert_called_once_with("owasp", "projects")
+
+    def test_get_entities(self, project_index):
+        """Test that get_entities constructs the correct queryset by chaining."""
+        mock_manager = MagicMock()
+        mock_manager.prefetch_related.return_value.filter.return_value.distinct.return_value = (
+            "final_queryset"
+        )
+
+        with patch.object(Project, "objects", mock_manager):
+            queryset = project_index.get_entities()
+
+            mock_manager.prefetch_related.assert_called_once_with(
+                "organizations",
+                "repositories",
+            )
+            mock_manager.prefetch_related.return_value.filter.assert_called_once_with(
+                organizations__isnull=False
+            )
+            mock_manager.prefetch_related.return_value.filter.return_value.distinct.assert_called_once()
+            assert queryset == "final_queryset"