Ignore topics since they will not be present in the CSV for Khan channels

Author: bjester

contentcuration/contentcuration/management/commands/fix_missing_import_sources.py

@@ -14,6 +14,7 @@
 from django.db.models import Q
 from django.db.models.expressions import F
 from django_cte import With
+from le_utils.constants import content_kinds
 
 from contentcuration.models import Channel
 from contentcuration.models import ContentNode
@@ -215,6 +216,7 @@ def handle_channel(self, lookup: LicensingFixesLookup, channel: dict) -> int:
                 public_channel_name=public_cte.col.name,
                 public_channel_deleted=public_cte.col.deleted,
             )
+            .exclude(kind=content_kinds.TOPIC)
             .filter(
                 Q(public_channel_deleted=True)
                 | ~Exists(

contentcuration/contentcuration/tests/management/commands/test_fix_missing_import_sources.py

@@ -2,6 +2,7 @@
 from unittest.mock import patch
 
 from django.core.management import call_command
+from le_utils.constants import content_kinds
 
 from contentcuration.management.commands.fix_missing_import_sources import (
     LicensingFixesLookup,
@@ -117,6 +118,31 @@ def test_handle__skips_node_when_lookup_returns_no_license(self):
         self.assertEqual(self.copied_node.license_description, "Nothing")
         self.assertEqual(self.copied_node.copyright_holder, "Nothing")
 
+    def test_handle__skips_topic_nodes_with_missing_source(self):
+        self.original_node.delete()
+        self.copied_node.refresh_from_db()
+        self.copied_node.kind_id = content_kinds.TOPIC
+        self.copied_node.license = None
+        self.copied_node.license_description = ""
+        self.copied_node.copyright_holder = ""
+        self.copied_node.save()
+
+        with patch(
+            "contentcuration.management.commands.fix_missing_import_sources.LicensingFixesLookup"
+        ) as lookup_cls:
+            lookup = lookup_cls.return_value
+            lookup.get_info.return_value = (5, "", "Khan Academy")
+
+            call_command("fix_missing_import_sources")
+
+        lookup.get_info.assert_not_called()
+
+        self.copied_node.refresh_from_db()
+        self.assertIsNone(self.copied_node.license_id)
+        self.assertEqual(self.copied_node.kind_id, content_kinds.TOPIC)
+        self.assertEqual(self.copied_node.license_description, "")
+        self.assertEqual(self.copied_node.copyright_holder, "")
+
 
 class LicensingFixesLookupTestCase(StudioTestCase):
     @classmethod