Boost up test coverage

[eduardo-rodrigues]

As the title says. Assigning to you @KaranSinghDev as your time permits.

Thank you very much.

[KaranSinghDev]

Hi @eduardo-rodrigues, @henryiii, I had a look at this and based on my understanding the coverage is currently 30% (1064 of 1514 statements missed) and with whole suite of 5 tests taking about an avg of 2 mins.

The biggest gaps are save_state/load_state ( about 1%covered), all the plotting methods ( 2%) and the whole _scan_hist_multi path ( 0.8%).

I ran each of those, they all work so it seems about mising tests only. I am thnking of three new files under test/ (state round-trip, plots via tmppath, multi−channel) as tests only. We can use a small npe to keep the suite fast and assert shapes and invariants (like nworker=4 must match nworker=1) rather than exact p-values since those shift with npe, the two ATLAS validated tests are not affected by this.

Let me know your thoughts on this.

[eduardo-rodrigues]

Hello @KaranSinghDev, that all very much makes sense to me 👍. Sounds good. Thank you.