more tests for visualization routine

[MerktSimon]

activate tests for minimal data file (cf issue #415)
tests without visu file now also with simu data

One might still argue that the minimal visualization file is not minimal since it still has the datasetId column. Imo it is only optional if one provide a dataset, condition or observable list to the function plot_data_and_simulation()
Any opinions on this?

[codecov]

Codecov Report

Merging #418 into develop will increase coverage by 0.09%.
The diff coverage is 0%.

@@             Coverage Diff             @@
##           develop     #418      +/-   ##
===========================================
+ Coverage    77.84%   77.93%   +0.09%     
===========================================
  Files           22       22              
  Lines         2126     2126              
  Branches       498      498              
===========================================
+ Hits          1655     1657       +2     
+ Misses         344      343       -1     
+ Partials       127      126       -1

Impacted Files	Coverage Δ
petab/visualize/helper_functions.py	88.15% <0%> (ø)	⬆️
petab/visualize/plot_data_and_simulation.py	77.27% <0%> (+3.03%)	⬆️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update d9260f0...58f611c. Read the comment docs.

[FFroehlich]

Adding tests where the DATASET_ID column are dropped from simdata/expdata shouldn't be too much effort right? Would also add coverage to the respective fill-in function.

[JanHasenauer]

Very good idea! The test would be very useful.

[MerktSimon]

Right now the routine seems still to struggle withouth the datasetId column.
But @plakrisenko is currently working on #272 and #190 including support of this. (https://github.com/PEtab-dev/PEtab/tree/pl_visualization)
I will include the according tests there.