fix: minimize except handler name

pysource_minimize/_minimize.py

@@ -734,6 +734,10 @@ def minimize_except_handler(handler):
                 if not handler.name and not try_star:
                     self.minimize_optional(handler.type)
 
+                if handler.name:
+                    self.try_attr(handler,"name",None)
+
+
             self.minimize_list(
                 node.handlers, minimize_except_handler, 1  # 0 if node.finalbody else 1
             )

tests/conftest.py

@@ -26,7 +26,7 @@ def pytest_sessionfinish(session, exitstatus):
         from .test_remove_one import generate_remove_one
         from .test_needle import generate_needle
 
-        for i in range(10):
+        for i in range(2):
             generate_remove_one()
             generate_needle()
 

tests/remove_one_samples/b51eb40ca6336b75936d070c9ed61d418acee4dd2e34a8c2ac9b5e3035a3c9b5.py

@@ -0,0 +1,6 @@
+try:
+    pass
+except name_2 if name_4 else name_2 as name_0:
+    pass
+except {name_3 for name_3 in name_0 if name_4} as name_1:
+    pass

tests/test_remove_one.py

@@ -20,6 +20,7 @@ def node_weights(source):
 
     def weight(node):
 
+        result = 1
         if isinstance(
             node,
             (
@@ -45,65 +46,69 @@ def weight(node):
                 ast.ImportFrom,
             ),
         ):
-            return 0
+            result = 0
 
         if isinstance(node, (ast.GeneratorExp, ast.ListComp, ast.SetComp)):
-            return 0
+            result = 0
         if isinstance(node, (ast.DictComp)):
-            return -2
+            result = -2
         if isinstance(node, ast.comprehension):
             # removing comrehension removes variable and iterable
-            return -1
+            result = -1
 
         if isinstance(node, (ast.Dict)):
-            return -len(node.keys) + 1
+            result = -len(node.keys) + 1
         if sys.version_info >= (3, 8) and isinstance(node, ast.NamedExpr):
-            return 0
+            result = 0
 
         if isinstance(node, ast.FormattedValue):
-            return 0
+            result = 0
         if isinstance(node, ast.JoinedStr):
             # work around for https://github.com/python/cpython/issues/110309
-            return -(
+            result = -(
                 sum(isinstance(n, ast.Constant) and n.value == "" for n in node.values)
             )
 
         if isinstance(node, ast.IfExp):
-            return -1
+            result = -1
         if isinstance(node, ast.Subscript):
-            return 0
+            result = 0
         if isinstance(node, ast.Index):
-            return 0
+            result = 0
 
         # match
         if sys.version_info >= (3, 10):
             if isinstance(node, ast.MatchValue):
-                return -1
+                result = -1
             if isinstance(node, (ast.MatchOr, ast.match_case, ast.MatchClass)):
-                return 0
+                result = 0
             if isinstance(node, ast.Match):
-                return -1  # for the subject
+                result = -1  # for the subject
             if isinstance(node, ast.MatchMapping):
                 # key-value pairs can only be removed together
-                return -len(node.patterns) + 1
+                result = -len(node.patterns) + 1
 
         # try
         if sys.version_info >= (3, 11) and isinstance(node, ast.TryStar):
             # execpt*: is invalid syntax
-            return -len(node.handlers) + 1
+            result = -len(node.handlers) + 1
 
         if isinstance(node, ast.excepthandler):
-            return 0
+            result = 0
+            if node.name:
+                result+=1
 
         if isinstance(node, ast.arguments):
             # kw_defaults and kwonlyargs can only be removed together
-            return -len(node.kw_defaults)
+            result = -len(node.kw_defaults)
 
         if sys.version_info >= (3, 12):
             if isinstance(node, ast.TypeAlias):
-                return 0
+                result = 0
 
-        return 1
+
+
+        return result
 
     return [(n, weight(n)) for n in ast.walk(tree)]