Merge pull request #21570 from QuLogic/spine-getattr

timhoffm · web-flow · commit b9065f737432 · 2021-11-09T17:09:57.000+01:00
Raise correct exception out of Spines.__getattr__
diff --git a/lib/matplotlib/spines.py b/lib/matplotlib/spines.py
@@ -550,7 +550,7 @@ def __getattr__(self, name):
         try:
             return self._dict[name]
         except KeyError:
-            raise ValueError(
+            raise AttributeError(
                 f"'Spines' object does not contain a '{name}' spine")
 
     def __getitem__(self, key):
diff --git a/lib/matplotlib/tests/test_spines.py b/lib/matplotlib/tests/test_spines.py
@@ -35,6 +35,8 @@ def set_val(self, val):
     spines[:].set_val('y')
     assert all(spine.val == 'y' for spine in spines.values())
 
+    with pytest.raises(AttributeError, match='foo'):
+        spines.foo
     with pytest.raises(KeyError, match='foo'):
         spines['foo']
     with pytest.raises(KeyError, match='foo, bar'):
