icecaster/sapphire — comment on commit 1a2ba5e7a41e78fb08c1f3606a1338ed1ac1d618
File: core/model/fieldtypes/Decimal.php line 16
By sminnee:

Yeah - as long as it works i'm happy. Can you amend one of the relevant unit tests to create a decimal field with a default value of say. 12345.67 and ensure that its constructed properly? Start by adding the test and confirming that it fails with your code, and then update the 4th argument and confirm that the test passes once you've made the change.

I know it may seem pedantic to add a test for a 1-line change but this is how we grow a robust test suite.