MNT: make sure we do not mutate input in Text.update

tacaswell · tacaswell · commit 6df12102639d · 2020-10-29T16:01:09.000-04:00
We take in a dictionary (called kwargs rather than collecting kwargs) which we mutate in the method. We pop bbox and fontproperties keys out of the user supplied dictionary (to make sure the precedence and default behaviors are correct) which is propagated back out to user code. This makes an internal copy of the dictionary before we mutate it. closes #18838
diff --git a/lib/matplotlib/tests/test_text.py b/lib/matplotlib/tests/test_text.py
@@ -12,7 +12,8 @@
 import matplotlib.pyplot as plt
 import matplotlib.transforms as mtransforms
 from matplotlib.testing.decorators import check_figures_equal, image_comparison
-
+from matplotlib.font_manager import FontProperties
+from matplotlib.text import Text
 
 needs_usetex = pytest.mark.skipif(
     not mpl.checkdep_usetex(True),
@@ -697,3 +698,13 @@ def test_transform_rotates_text():
                    transform_rotates_text=True)
     result = text.get_rotation()
     assert_almost_equal(result, 30)
+
+
+def test_update_mutate_input():
+    inp = dict(fontproperties=FontProperties(weight="bold"),
+               bbox=None)
+    cache = dict(inp)
+    t = Text()
+    t.update(inp)
+    assert inp['fontproperties'] == cache['fontproperties']
+    assert inp['bbox'] == cache['bbox']
diff --git a/lib/matplotlib/text.py b/lib/matplotlib/text.py
@@ -171,6 +171,8 @@ def __init__(self,
         self.update(kwargs)
 
     def update(self, kwargs):
+        # make a copy so we do not mutate user input!
+        kwargs = dict(kwargs)
         # docstring inherited
         sentinel = object()  # bbox can be None, so use another sentinel.
         # Update fontproperties first, as it has lowest priority.
