Added new test and comments

AryanSheka · AryanSheka · commit 43947b7b0e65 · 2023-07-28T20:03:13.000+05:30
diff --git a/lib/matplotlib/tests/test_path.py b/lib/matplotlib/tests/test_path.py
@@ -242,7 +242,13 @@ def test_make_compound_path_stops():
     assert np.sum(compound_path.codes == Path.STOP) == 0
 
 
+def test_path_sketch_seed():
+    # the default value of path.sketch_seed should be 0
+    assert rcParams['path.sketch_seed'] == 0
+
+
 def test_xkcd_seed_update():
+    # when passing a seed to xkcd, the global rcParam should be updated
     with plt.xkcd(seed=2000):
         assert rcParams['path.sketch_seed'] == 2000
 
@@ -284,7 +290,9 @@ def test_artist_seed():
     ln[0].set_sketch_params(3, 120, 40, 2000)
 
     # set_sketch_params seed should override seed set by rcParam
-    rcParams['path.sketch_seed'] = 0
+    # when seed is passed in set_sketch, it should be used
+    # else rcParam should be used as seed
+    rcParams['path.sketch_seed'] = 59856
     ln_2 = ax.plot(x, y_2, color='green')
     ln_2[0].set_sketch_params(3, 120, 40, 19680801)
 
@@ -306,8 +314,10 @@ def test_xkcd_seed():
         ln = ax.plot(x, y_1, color='black')
         ln[0].set_sketch_params(3, 120, 40)
 
-    # xkcd should override rcParam seed.
-    rcParams['path.sketch_seed'] = 0
+    # xkcd should override rcParam seed
+    # when seed is passed in xkcd, it should be used,else rcParam should be used as seed
+    # passing seed to xkcd should also update global rcParam
+    rcParams['path.sketch_seed'] = 8448
     with plt.xkcd(3, 120, 40, seed=420):
         ln_2 = ax.plot(x, y_2, color='orange')
 
@@ -318,6 +328,8 @@ def test_path_seed():
     y_1 = 20*np.sin(x)
     y_2 = 20*np.sin(x)+10
     fig, ax = plt.subplots()
+
+    # if no seed is passed in set_sketch, rcParam should be used as seed
     rcParams['path.sketch_seed'] = 645
     ln = ax.plot(x, y_1, color='blue')
     ln[0].set_sketch_params(3, 120, 40)
