Fix/arrhenius kc

[SylvainPlessis]

This is both a slight bug fix and a cleaning/bettering of the tests.

For some reason that eludes me, I haven't made the overload with the KineticsConditions for all the rate constants' models computation.

So this PR corrects this, and update the tests because it upset me that they were not already all standardized.

[SylvainPlessis]

By the way, if ever the kinetics rate tests can be a little bit more standardized and upgraded, please shout it there, I don't know why I didn't do it correctly in the first place, so if I can have them proper in the second place, that'd be great.

[SylvainPlessis]

merge 0.3.0-release into master, rebased this PR.

[SylvainPlessis]

can we have a merge of 0.3.0 into master and then this branch on master (after a rebase, I like rebases when they're easy)? I'll need it for PR #142.

[pbauman]

an we have a merge of 0.3.0 into master

Done.

[pbauman]

(after a rebase, I like rebases when they're easy)?

Agreed. That's my MO on this - rebase is preferred if not terribly difficult. If it's really bad, fall back to merge to limit possibility of negatively rewriting history.

[SylvainPlessis]

rebased and happy on my computer

[pbauman]

I'm good with this, but I don't see the "slight bug fix", I only see the added overloads. What was the bug fix?

[SylvainPlessis]

That's the bug fix, thing is, when using a KineticsConditions object for far up, if ever you end with one of those kinetics model, you get a compiling error.

Edit: For the new added sensitivity stuff that is.

[SylvainPlessis]

Thanks for that remark, I'm missing the most important part of this bugfix: the KineticsType object still passed a const StateType & T to those not updated models.

Correcting this right away.

[pbauman]

Replaced by #182.