fix for abstract classes (Azure#35891)

Co-authored-by: Catalina Peralta <caperal@microsoft.com>
azure-sdk · Jun 4, 2024 · 2ff9ddb · 2ff9ddb
1 parent f838ccc
commit 2ff9ddb
Showing 1 changed file with 5 additions and 1 deletion.
diff --git a/scripts/breaking_changes_checker/detect_breaking_changes.py b/scripts/breaking_changes_checker/detect_breaking_changes.py
@@ -164,7 +164,11 @@ def get_properties(cls: Type) -> Dict:
             analyzer = ClassTreeAnalyzer(base_class.__name__)
             analyzer.visit(module)
             cls_node = analyzer.cls_node
-            get_property_names(cls_node, attribute_names)
+            if cls_node:
+                get_property_names(cls_node, attribute_names)
+            else:
+                # Abstract base classes fail here, e.g. "collections.abc.MuttableMapping"
+                _LOGGER.info(f"Unable to get class node for {base_class.__name__}. Skipping...")
     else:
         get_property_names(cls_node, attribute_names)
     return attribute_names