Put success closures last

This allows to use the success closure as a trailing closure (instead of the failure closure), which feels more natural.
`fetchValueForKey` still being called without using trailing losures in `testSetValueInDefaultFormat ` and `testSetValueInFormat`