DoseMath.recommendTempBasalFromPredictedGlucose has a deceptive default value for allowPredictiveTempBelowRange #293

The default is for this value to be false, which can (and has) confused the reader to believe that this value is not set by default. But in actuality, this function is only called from:

which forces the value to be true. This is a significant deviation from expectations unless you read the code carefully.

I propose that we make this value non-optional. It should be relatively easy - all we have to do is update the unit tests to force the value to be false anywhere that it's missing.