feat(glossary) removed debugging statements, fixed pipeline

Author: kevinkarchacryl

metadata-ingestion/src/datahub/ingestion/source/metadata/business_glossary.py

@@ -161,19 +161,21 @@ def get_owners_multiple_types(owners: OwnersMultipleTypes) -> models.OwnershipCl
     if isinstance(owners, Owners):
         return models.OwnershipClass(owners=list(get_owners(owners)))
 
-    owners_meta = []
+    owners_meta: List[models.OwnerClass]
     for owner in owners:
         owners_meta.extend(get_owners(owner))
 
     return models.OwnershipClass(owners=owners_meta)
 
 
 def get_owners(owners: Owners) -> Iterable[models.OwnerClass]:
-    if owners.type.startswith("urn:li:ownershipType:"):
+    if owners.type and owners.type.startswith("urn:li:ownershipType:"):
         ownership_type = "CUSTOM"
         ownership_type_urn = owners.type
     else:
-        ownership_type, ownership_type_urn = validate_ownership_type(owners.type)
+        ownership_type, ownership_type_urn = validate_ownership_type(
+            owners.type or models.OwnershipTypeClass.DEVELOPER
+        )
 
     if owners.typeUrn is not None:
         ownership_type_urn = owners.typeUrn

metadata-ingestion/tests/integration/business-glossary/test_business_glossary.py

@@ -80,8 +80,6 @@ def test_single_owner_types(
     mock_time,
 ):
     """Test basic single owner cases with different ownership types"""
-    output_mces_path: str = f"{tmp_path}/single_owner_types.json"
-    print(f"\nOutput file location: {output_mces_path}")
     test_resources_dir = pytestconfig.rootpath / "tests/integration/business-glossary"
     output_mces_path: str = f"{tmp_path}/single_owner_types.json"
     golden_mces_path: str = f"{test_resources_dir}/single_owner_types_golden.json"