Review FAILED tests

[ES-Solar]

When I run tests on the master branch in macos, it is reporting 91 failed tests. For the first couple I looked at, the resulting ps files look good or maybe better (better formatting) than the original baseline. It would be helpful to better understand the failures and determine which actually represent problems.
So I'm creating this issue to help document my review of the chronically failing tests.

[github-actions]

👋 Thanks for opening your first issue here! Please make sure you filled out the template with as much detail as possible. We appreciate that you took the time to contribute!

Please make sure you read our Contributing Guide and abide by our Code of Conduct.

[seisman]

Have you set GMT_DATA_SERVER to static when you run the tests locally? The CI uses the static server, and there are only a few failing tests.

[joa-quim]

Extending a bit more what Dongdong said, we have the DVC exactly because of this. Before we used to have the PS files directly included in the GMT repo, but that was making the repo's size increase fast and not stop. That happened, for example, whenever data from one of the hosted data sources (earth_relief, for example) was updated. Because of this we have a collection of remote files used in tests that are not updated. To use them, the GMT_DATA_SERVER must be set to static.

[ES-Solar]

I was not aware of needing to set the GMT_DATA_SERVER variable to static, thanks for the guidance.

[ES-Solar]

I reran the test with the GMT_DATA_SERVER set to static and got the same results, 91 failures. Is that not close to what youwould expect?
I am including the examples in the docs as well as the 'tests', ie:
set (DO_EXAMPLES TRUE)
set (DO_TESTS TRUE)

[joa-quim]

To be honest, I also have troubles with that GMT_DATA_SERVER setting and since I've no patience to fight scripts, I set
set (GMT_DATA_SERVER static)
in ConfigUser.cmake before running the tests.

[MarceloBanik]

changing line 212 in ConfigUserAdvanced.cmake from

# set (GMT_DATA_SERVER "static")

to

set (GMT_DATA_SERVER "static")

I went from 80 tests failed out of 1194 to 31 tests failed out of 1194.

[ES-Solar]

Thanks for the comment, I had changed that line but I must have missed something in getting the change to be picked up in the ctest run. I'll review my re-building procedures