Refactor test_image to plot @circuit.png instead of test_logo.png (#2922

)

So that we don't need to pull test_logo.png from dvc,
or run test_logo first to generate the image.

* Add @circuit.png to cache
* Remove dvc pull step from ci_tests_legacy.yaml
* Try running test_image on Windows to see if it crashes

.github/workflows/ci_tests_legacy.yaml

@@ -76,7 +76,6 @@ jobs:
  rioxarray
  sphinx-gallery
  build
- dvc
  make
  pip
  pytest
@@ -101,10 +100,6 @@ jobs:
  touch ~/.gmt/server/gmt_data_server.txt ~/.gmt/server/gmt_hash_server.txt
  ls -lhR ~/.gmt
 
- # Pull baseline image data from dvc remote (DAGsHub)
- - name: Pull baseline image data from dvc remote
- run: dvc pull pygmt/tests/baseline/test_logo.png && ls -lhR pygmt/tests/baseline/
-
  # Install the package that we want to test
  - name: Install the package
  run: make install

pygmt/helpers/testing.py

@@ -202,6 +202,7 @@ def download_test_data():
  "@S90E000.earth_wdmam_03m_g.nc", # Specific grid for 03m test
  # Other cache files
  "@capitals.gmt",
+ "@circuit.png",
  "@earth_relief_20m_holes.grd",
  "@EGM96_to_36.txt",
  "@MaunaLoa_CO2.txt",

pygmt/tests/baseline/test_image.png.dvc

@@ -1,4 +1,4 @@
 outs:
-- md5: c2e05c0d6fb17bcc7bf2f96193f91ad6
- size: 10005
+- md5: d7d0d71a44a232d5907dbd44f7a08f18
+ size: 30811
  path: test_image.png

pygmt/tests/test_image.py

@@ -1,21 +1,15 @@
 """
 Test Figure.image.
 """
-import os
-import sys
-
 import pytest
 from pygmt import Figure
 
-TEST_IMG = os.path.join(os.path.dirname(__file__), "baseline", "test_logo.png")
-
 
-@pytest.mark.skipif(sys.platform == "win32", reason="crashes on Windows")
 @pytest.mark.mpl_image_compare
 def test_image():
  """
  Place images on map.
  """
  fig = Figure()
- fig.image(TEST_IMG, position="x0/0+w2c", box="+pthin,blue")
+ fig.image(imagefile="@circuit.png", position="x0/0+w2c", box="+pthin,blue")
  return fig