Migrate Figure.inset tests to use dvc

[willschlitzer]

As discussed in #1131, this moves the reference figures in fig.inset to use dvc.

Fixes #

Reminders

• Run make format and make check to make sure the code follows the style guide.
• Add tests for new features or tests that would have caught the bug that you're fixing.
• Add new public functions/methods/classes to doc/api/index.rst.
• Write detailed docstrings for all functions/methods.
• If adding new functionality, add an example to docstrings or tutorials.

Slash Commands

You can write slash commands (/command) in the first line of a comment to perform
specific operations. Supported slash commands are:

• /format: automatically format and lint the code
• /test-gmt-dev: run full tests on the latest GMT development version

[github-actions]

Summary of changed images

This is an auto-generated report of images that have changed on the DVC remote

Status	Path
added	pygmt/tests/baseline/test_inset_aliases.png
added	pygmt/tests/baseline/test_inset_context_manager.png

Image diff(s)

• pygmt/tests/baseline/test_inset_aliases.png

• pygmt/tests/baseline/test_inset_context_manager.png

Report last updated at commit 85a365d

[seisman]

This pull request keeps @check_figures_equal for test_inset_context_manager() as it is testing the use of the context manager.

I think we still should refactor this test.

[willschlitzer]

This pull request keeps @check_figures_equal for test_inset_context_manager() as it is testing the use of the context manager.

I think we still should refactor this test.

What should I use for the reference image to test the context manager? Which part of the test should remain in the refactored test and which should be discarded?

[seisman]

Test that the inset context manager works and, once closed, plotting elements are added to the larger figure.

The main purpose of this test is to check if basemap(rose="jTR+w3c") plots the rose in the large figure, not in the inset map. So you can simply remove the reference image.

[willschlitzer]

Test that the inset context manager works and, once closed, plotting elements are added to the larger figure.

The main purpose of this test is to check if basemap(rose="jTR+w3c") plots the rose in the large figure, not in the inset map. So you can simply remove the reference image.

Updated it to use a reference image.

[weiji14]

Just need to remove the check_figures_equal import and it should be good to merge I think.