test: add more test cases

Author: BryanttV

openedx_authz/tests/api/test_data.py

@@ -737,6 +737,7 @@ class TestOrgContentLibraryGlobData(TestCase):
         ("lib:Org+WithPlus:*", False),
         ("lib:(Org):*", False),
         ("lib:Org", False),
+        ("lib:Org*", False),
         ("other:DemoX:*", False),
         ("lib:DemoX:*:*", False),
     )
@@ -752,14 +753,18 @@ def test_validate_external_key(self, external_key, expected_valid):
         ("lib:Org:With:Colon:*", "Org:With:Colon"),
         ("lib:DemoX", None),
         ("lib:DemoX:+*", None),
+        ("lib:DemoX*", None),
+        ("lib:DemoX:**", None),
+        ("lib:DemoX:suffix", None),
     )
     @unpack
     def test_get_org(self, external_key, expected_org):
         """Test organization extraction from library glob pattern."""
         self.assertEqual(OrgContentLibraryGlobData.get_org(external_key), expected_org)
 
-    def test_exists_true_when_org_has_libraries_in_db(self):
-        """exists() returns True when at least one library with the org exists in the DB."""
+    @patch("openedx_authz.api.data.Organization", Organization)
+    def test_exists_true_when_org_exists(self):
+        """exists() returns True when the org exists."""
         org_name = "DemoX"
         organization = Organization.objects.create(short_name=org_name)
         ContentLibrary.objects.create(org=organization, slug="testlib", title="Test Library")
@@ -768,8 +773,8 @@ def test_exists_true_when_org_has_libraries_in_db(self):
 
         self.assertTrue(result)
 
-    def test_exists_false_when_org_does_not_exist_in_db(self):
-        """exists() returns False when the org does not exist in the DB."""
+    def test_exists_false_when_org_does_not_exist(self):
+        """exists() returns False when the org does not exist."""
         org_name = "DemoX"
 
         result = OrgContentLibraryGlobData(external_key=f"lib:{org_name}:*").exists()
@@ -806,6 +811,7 @@ class TestOrgCourseOverviewGlobData(TestCase):
         ("course-v1:(Org)+*", False),
         ("course-v1:Org:With:Plus+*", False),
         ("course-v1:OpenedX", False),
+        ("course-v1:OpenedX*", False),
         ("other:OpenedX+*", False),
         ("course-v1:OpenedX**", False),
     )
@@ -819,6 +825,10 @@ def test_validate_external_key(self, external_key, expected_valid):
         ("course-v1:My-Org_1+*", "My-Org_1"),
         ("course-v1:Org.with.dots+*", "Org.with.dots"),
         ("course-v1:Org:With:Plus+*", "Org:With:Plus"),
+        ("course-v1:OpenedX", None),
+        ("course-v1:OpenedX*", None),
+        ("course-v1:OpenedX+**", None),
+        ("course-v1:OpenedX+suffix", None),
     )
     @unpack
     def test_get_org(self, external_key, expected_org):