Merge remote-tracking branch 'origin/test-grdimage-dvc' into test-grdimage-dvc

Author: willschlitzer

CONTRIBUTING.md

@@ -181,11 +181,12 @@ When editing documentation, use the following standards to demonstrate the examp
 
 1. Python arguments, such as import statements, Boolean expressions, and function
    arguments should be wrapped as ``code`` by using \`\` on both sides of the code.
-   Example: \`\`import pygmt\`\` results in ``import pygmt``.
+   Examples: \`\`import pygmt\`\` results in ``import pygmt``, \`\`True\`\` results in `True`,
+    \`\`style="v"\`\` results in `style="v"`.
 2. Literal GMT arguments should be **bold** by wrapping the arguments with \*\*
    (two asterisks) on both sides. The argument description should be in *italicized*
    with \* (single asterisk) on both sides.
-   Example: `**+l**\ *label*` results in **+l***label*.
+   Examples: `**+l**\ *label*` results in **+l***label*, `**05m**` results in **05m**.
 3. Optional arguments are placed wrapped with [ ] (square brackets).
 4. Arguments that are mutually exclusive are separated with a | (bar) to denote "or".
 

pygmt/tests/baseline/test_config.png

@@ -0,0 +1,4 @@
+outs:
+- md5: e473079025646ea07e44493f6ab396da
+  size: 33464
+  path: test_config.png

pygmt/tests/baseline/test_config.png.dvc

@@ -0,0 +1,4 @@
+outs:
+- md5: e1a3887151ea389de97f177c19488c5e
+  size: 80854
+  path: test_config_font_annot.png

pygmt/tests/baseline/test_config_font_annot.png.dvc

@@ -0,0 +1,4 @@
+outs:
+- md5: b011328f9e88799e3af073da4f2b3835
+  size: 97085
+  path: test_config_font_one.png

pygmt/tests/baseline/test_config_font_one.png.dvc

@@ -0,0 +1,4 @@
+outs:
+- md5: 3d1d843c5150f2f775fdb9382284fa31
+  size: 10906
+  path: test_config_format_time_map.png

pygmt/tests/baseline/test_config_format_time_map.png

@@ -0,0 +1,4 @@
+outs:
+- md5: 40a89f9014ac054f5f196ace06a00361
+  size: 9389
+  path: test_config_map_annot_offset.png

pygmt/tests/baseline/test_config_format_time_map.png.dvc

@@ -0,0 +1,4 @@
+outs:
+- md5: cedcf979e9d25129babae0fbeeeecfbc
+  size: 17539
+  path: test_config_map_grid_cross_size.png

pygmt/tests/baseline/test_config_map_annot_offset.png

@@ -0,0 +1,4 @@
+outs:
+- md5: 04de12d91ce2f48ad0b09061077f9780
+  size: 17179
+  path: test_config_map_grid_pen.png

pygmt/tests/baseline/test_config_map_annot_offset.png.dvc

@@ -0,0 +1,4 @@
+outs:
+- md5: d97081684520b6a91221d25f71942cc9
+  size: 16855
+  path: test_config_map_tick_length.png

pygmt/tests/baseline/test_config_map_grid_cross_size.png

@@ -0,0 +1,4 @@
+outs:
+- md5: 95a1755035175e331efc7508d9e8e3eb
+  size: 17321
+  path: test_config_map_tick_pen.png

pygmt/tests/baseline/test_config_map_grid_cross_size.png.dvc

@@ -1,9 +1,8 @@
 """
-Tests for gmt config.
+Tests for pygmt.config.
 """
 import pytest
 from pygmt import Figure, config
-from pygmt.helpers.testing import check_figures_equal
 
 
 @pytest.mark.mpl_image_compare
@@ -15,83 +14,61 @@ def test_config():
     # Change global settings of current figure
     config(FONT_ANNOT_PRIMARY="blue")
     fig.basemap(
-        region="0/10/0/10", projection="X10c/10c", frame=["af", '+t"Blue Annotation"']
+        region=[0, 10, 0, 10], projection="X5c/5c", frame=["af", '+t"Blue Annotation"']
     )
 
     with config(FONT_LABEL="red", FONT_ANNOT_PRIMARY="red"):
         fig.basemap(
-            region="0/10/0/10",
-            projection="X10c/10c",
+            region=[0, 10, 0, 10],
+            projection="X5c/5c",
             frame=['xaf+l"red label"', "yaf", '+t"red annotation"'],
-            X="15c",
+            xshift="7c",
         )
 
     fig.basemap(
-        region="0/10/0/10",
-        projection="X10c/10c",
+        region=[0, 10, 0, 10],
+        projection="X5c/5c",
         frame=["af", '+t"Blue Annotation"'],
-        X="15c",
+        xshift="7c",
     )
     # Revert to default settings in current figure
     config(FONT_ANNOT_PRIMARY="black")
     return fig
 
 
-@check_figures_equal()
+@pytest.mark.mpl_image_compare
 def test_config_font_one():
     """
-    Test that setting `FONT` config changes all `FONT_*` settings except
-    `FONT_LOGO`.
+    Test that setting FONT config changes all FONT_* settings except FONT_LOGO.
 
-    Specifically, this test only checks that `FONT_ANNOT_PRIMARY`,
-    `FONT_ANNOT_SECONDARY`, `FONT_LABEL`, and `FONT_TITLE` are modified.
+    Specifically, this test only checks that FONT_ANNOT_PRIMARY,
+    FONT_ANNOT_SECONDARY, FONT_LABEL, and FONT_TITLE are modified.
     """
-    fig_ref = Figure()
-    with config(
-        FONT_ANNOT_PRIMARY="8p,red",
-        FONT_ANNOT_SECONDARY="8p,red",
-        FONT_LABEL="8p,red",
-        FONT_TITLE="8p,red",
-    ):
-        fig_ref.basemap(R="0/9/0/9", J="C3/3/9c", Tm="jTL+w4c+d4.5+l")
-    fig_ref.basemap(Tm="jBR+w5c+d-4.5+l")
-
-    fig_test = Figure()
+    fig = Figure()
     with config(FONT="8p,red"):
-        fig_test.basemap(
-            region=[0, 9, 0, 9], projection="C3/3/9c", compass="jTL+w4c+d4.5+l"
-        )
-    fig_test.basemap(compass="jBR+w5c+d-4.5+l")
-
-    return fig_ref, fig_test
+        fig.basemap(region=[0, 9, 0, 9], projection="C3/3/9c", compass="jTL+w4c+d4.5+l")
+    fig.basemap(compass="jBR+w5c+d-4.5+l")
+    return fig
 
 
-@check_figures_equal()
+@pytest.mark.mpl_image_compare
 def test_config_font_annot():
     """
-    Test that setting `FONT_ANNOT` config changes both `FONT_ANNOT_PRIMARY` and
-    `FONT_ANNOT_SECONDARY`.
+    Test that setting FONT_ANNOT config changes both FONT_ANNOT_PRIMARY and
+    FONT_ANNOT_SECONDARY.
     """
-    fig_ref = Figure()
-    with config(FONT_ANNOT_PRIMARY="6p,red", FONT_ANNOT_SECONDARY="6p,red"):
-        fig_ref.basemap(R="0/9/0/9", J="C3/3/9c", Tm="jTL+w4c+d4.5")
-    fig_ref.basemap(compass="jBR+w5c+d-4.5")
-
-    fig_test = Figure()
+    fig = Figure()
     with config(FONT_ANNOT="6p,red"):
-        fig_test.basemap(
-            region=[0, 9, 0, 9], projection="C3/3/9c", compass="jTL+w4c+d4.5"
-        )
-    fig_test.basemap(compass="jBR+w5c+d-4.5")
-
-    return fig_ref, fig_test
+        fig.basemap(region=[0, 9, 0, 9], projection="C3/3/9c", compass="jTL+w4c+d4.5")
+    fig.basemap(compass="jBR+w5c+d-4.5")
+    return fig
 
 
 @pytest.mark.mpl_image_compare
 def test_config_format_time_map():
     """
-    Test that setting `FORMAT_TIME_MAP` config changes both
-    `FORMAT_TIME_PRIMARY_MAP` and `FORMAT_TIME_SECONDARY_MAP`.
+    Test that setting FORMAT_TIME_MAP config changes both
+    FORMAT_TIME_PRIMARY_MAP and FORMAT_TIME_SECONDARY_MAP.
     """
     fig = Figure()
     with config(FORMAT_TIME_MAP="abbreviation"):
@@ -107,8 +84,8 @@ def test_config_format_time_map():
 @pytest.mark.mpl_image_compare
 def test_config_map_annot_offset():
     """
-    Test that setting `MAP_ANNOT_OFFSET` config changes both
-    `MAP_ANNOT_OFFSET_PRIMARY` and `MAP_ANNOT_OFFSET_SECONDARY`.
+    Test that setting MAP_ANNOT_OFFSET config changes both
+    MAP_ANNOT_OFFSET_PRIMARY and MAP_ANNOT_OFFSET_SECONDARY.
     """
     fig = Figure()
     with config(MAP_ANNOT_OFFSET="15p"):
@@ -124,8 +101,8 @@ def test_config_map_annot_offset():
 @pytest.mark.mpl_image_compare
 def test_config_map_grid_cross_size():
     """
-    Test that setting `MAP_GRID_CROSS_SIZE` config changes both
-    `MAP_GRID_CROSS_SIZE_PRIMARY` and `MAP_GRID_CROSS_SIZE_SECONDARY`.
+    Test that setting MAP_GRID_CROSS_SIZE config changes both
+    MAP_GRID_CROSS_SIZE_PRIMARY and MAP_GRID_CROSS_SIZE_SECONDARY.
     """
     fig = Figure()
     with config(MAP_GRID_CROSS_SIZE="3p"):
@@ -142,8 +119,8 @@ def test_config_map_grid_cross_size():
 @pytest.mark.mpl_image_compare
 def test_config_map_grid_pen():
     """
-    Test that setting `MAP_GRID_PEN` config changes both `MAP_GRID_PEN_PRIMARY`
-    and `MAP_GRID_PEN_SECONDARY`.
+    Test that setting MAP_GRID_PEN config changes both MAP_GRID_PEN_PRIMARY and
+    MAP_GRID_PEN_SECONDARY.
     """
     fig = Figure()
     with config(MAP_GRID_PEN="thick,red"):
@@ -160,8 +137,8 @@ def test_config_map_grid_pen():
 @pytest.mark.mpl_image_compare
 def test_config_map_tick_length():
     """
-    Test that setting `MAP_TICK_LENGTH` config changes both
-    `MAP_TICK_LENGTH_PRIMARY` and `MAP_TICK_LENGTH_SECONDARY`.
+    Test that setting MAP_TICK_LENGTH config changes both
+    MAP_TICK_LENGTH_PRIMARY and MAP_TICK_LENGTH_SECONDARY.
     """
     fig = Figure()
     with config(MAP_TICK_LENGTH="5p"):
@@ -178,8 +155,8 @@ def test_config_map_tick_length():
 @pytest.mark.mpl_image_compare
 def test_config_map_tick_pen():
     """
-    Test that setting `MAP_TICK_PEN` config changes both `MAP_TICK_PEN_PRIMARY`
-    and `MAP_TICK_PEN_SECONDARY`.
+    Test that setting MAP_TICK_PEN config changes both MAP_TICK_PEN_PRIMARY and
+    MAP_TICK_PEN_SECONDARY.
     """
     fig = Figure()
     with config(MAP_TICK_PEN="thick,red"):

pygmt/tests/baseline/test_config_map_grid_pen.png

@@ -146,18 +146,16 @@ def test_grdimage_over_dateline(xrgrid):
 
 
 @check_figures_equal()
-@pytest.mark.parametrize("lon0", [0, 123, 180])
-@pytest.mark.parametrize("proj_type", ["H", "W"])
-def test_grdimage_central_meridians(grid, proj_type, lon0):
+def test_grdimage_central_meridians(grid):
     """
     Test that plotting a grid with different central meridians (lon0) using
     Hammer (H) and Mollweide (W) projection systems work.
     """
     fig_ref, fig_test = Figure(), Figure()
     fig_ref.grdimage(
-        "@earth_relief_01d_g", projection=f"{proj_type}{lon0}/15c", cmap="geo"
+        "@earth_relief_01d_g", projection="W123/15c", cmap="geo"
     )
-    fig_test.grdimage(grid, projection=f"{proj_type}{lon0}/15c", cmap="geo")
+    fig_test.grdimage(grid, projection="W123/15c", cmap="geo")
     return fig_ref, fig_test