Add regression tests to the visualization routines

[ranocha]

Many visualization routines have mostly smoke tests. We should also add some regression tests - maybe something based on computing the hash of rounded data? Or computing an error of an interpolated solution compared to evaluating the solution directly on the nodes?

[ranocha]

I added several regression tests for PlotData1D in #2334 and #2340. We should do something similar for the other plotting data types.

[midsorbet]

Hi, I'd like to work on this issue. Do you all happen to have any recommendation for what data type to start with? If not, I was planning to get started with ScalarPlotData2D? Thanks!

[ranocha]

Thanks a lot! I have been using PlotData2D extensively, so I think it would be nice to start with it. However, the scalar version is also fine - every contribution is meaningful!

[ShahuK283]

Hi! I recently contributed the extension renaming fix (#2543). I am interested in working on this issue. Is it still available?

[jlchan]

@ShahuK283 thank you, and I believe it is still available. For your fix of #2543, could you open a PR?

[ShahuK283]

Hi @jlchan!

Yes, I have already opened the PR for the extension renaming here: #2543.

Regarding this issue (#2335): I will look into test/test_visualization.jl and the examples mentioned (#2334) to see how to implement the regression tests. I'll likely target PlotData2D or ScalarPlotData2D if they aren't fully covered yet.

[jlchan]

Thanks @ShahuK283! I think you may have added a commit to fix the issue ShahuK283@4683378. If you can open a PR, we can run our CI tests on your commit :)

[JoshuaLampert]

Thanks @ShahuK283! I think you may have added a commit to fix the issue ShahuK283@4683378. If you can open a PR, we can run our CI tests on your commit :)

There already is a PR: #2747

[jlchan]

Ah sorry, I missed that - thanks @JoshuaLampert