change tests to thread local changes

Author: cvanelteren

ultraplot/tests/test_config.py

@@ -8,31 +8,27 @@ def test_wrong_keyword_reset():
     """
     The context should reset after a failed attempt.
     """
-    # Init context
-    uplt.rc.context()
-    config = uplt.rc
-    # Set a wrong key
-    with pytest.raises(KeyError):
-        config._get_item_dicts("non_existing_key", "non_existing_value")
-    # Set a known good value
-    config._get_item_dicts("coastcolor", "black")
-    # Confirm we can still plot
-    fig, ax = uplt.subplots(proj="cyl")
-    ax.format(coastcolor="black")
-    fig.canvas.draw()
+    # Use context manager for temporary rc changes
+    # Use context manager with direct value setting
+    with uplt.rc.context(coastcolor="black"):
+        # Set a wrong key
+        with pytest.raises(KeyError):
+            uplt.rc._get_item_dicts("non_existing_key", "non_existing_value")
+        # Confirm we can still plot
+        fig, ax = uplt.subplots(proj="cyl")
+        ax.format(coastcolor="black")
+        fig.canvas.draw()
 
 
 def test_cycle_in_rc_file(tmp_path):
     """
     Test that loading an rc file correctly overwrites the cycle setting.
     """
+    rc = uplt.config.Configurator()
     rc_content = "cycle: colorblind"
     rc_file = tmp_path / "test.rc"
     rc_file.write_text(rc_content)
-
-    # Load the file directly. This should overwrite any existing settings.
-    uplt.rc.load(str(rc_file))
-
+    rc.load(str(rc_file))
     assert uplt.rc["cycle"] == "colorblind"
 
 
@@ -208,6 +204,8 @@ def test_cycle_rc_setting(cycle, raises_error):
     """
     if raises_error:
         with pytest.raises(ValueError):
-            uplt.rc["cycle"] = cycle
+            with uplt.rc.context(cycle=cycle):
+                pass
     else:
-        uplt.rc["cycle"] = cycle
+        with uplt.rc.context(cycle=cycle):
+            pass