It looks like this pull request may not have tests. Please make sure to add tests before merging. If you need an exemption to this rule, contact Hixie on the #hackers channel in Chat.

If you are not sure if you need tests, consider this rule of thumb: the purpose of a test is to make sure someone doesn't accidentally revert the fix. Ask yourself, is there anything in your PR that you feel it is important we not accidentally revert back to how it was before your fix?

Reviewers: Read the Tree Hygiene page and make sure this patch meets those guidelines before LGTMing.

Hi, @ansolantz
You should write tests code or request to be approved.
Please follow link below: https://github.com/flutter/flutter/wiki/Tree-hygiene#tests

Hi, @ansolantz You should write tests code or request to be approved. Please follow link below: https://github.com/flutter/flutter/wiki/Tree-hygiene#tests

Hi @SuhwanCha,
I honestly do not know how to write a test case for this, and since the solution is pretty straightforward I prefer to wait for the ticket to be answered.

@ansolantz that fix is probably correct (i honestly don't know the code well enough to say for sure) but @SuhwanCha is right, before we can review it it really needs a test. You can find the other datepicker tests here: https://github.com/flutter/flutter/blob/master/packages/flutter/test/cupertino/date_picker_test.dart

Thanks for the contribution! I'm going to close this PR since it needs tests, as per the comments above. If anyone wants to take over please don't hesitate to take this code and submit a new PR!