Improve test_ragone_chart.

[Rombur]

Since load charge is not implemented yet. This test cannot reach 100% of coverage.

[dalg24]

Actually what you want to use here is probably BOOST_DATA_TEST_CASE. I use it in test_equivalent_circuit.cc or you can directly look at the doc

[codecov-io]

Current coverage is 92.30%

Merging #148 into master will increase coverage by +0.66%

1. File ...test_ragone_chart.cc was modified. more
   • Misses -27
   • Partials 0
   • Hits +27

@@             master       #148   diff @@
==========================================
  Files            67         67          
  Lines          4283       4309    +26   
  Methods           0          0          
  Messages          0          0          
  Branches          0          0          
==========================================
+ Hits           3924       3977    +53   
+ Misses          359        332    -27   
  Partials          0          0          

Powered by Codecov. Last updated by 1db33dc...19f5b18

[Rombur]

I use BOOST_DATA_TEST_CASE

[Rombur]

OK this test is not 100% because of the constant load and the throws. Do you want me to remove the throws?

[dalg24]

what's the reason for not having the constant load here?

[dalg24]

because we only have equivalent circuits and they do have the method implemented don't they?

[dalg24]

No you're right...

[dalg24]

We could use expected failure otherwise

[dalg24]

Also, would you remove SeriesRC:: and ParallelRC::reset(...)?

[dalg24]

You can use the decorator boost::unit_test::expected_failures(N)

[Rombur]

No, you are right. The code throws because in the test there are several places where we don't support constant_load. I didn't pay attention why the code was throwing.

[Rombur]

I have removed the useless throws and the load_charge.