This code is fine, but you need to add unit tests.

Okay I will add some for this case. Thank you.

Your test logic is good, but you cannot pass strings other than ISO8601 Extended with Offset to the date constructor and get a reliable result. The format you are using is not a defined format, thus each browser may parse it differently. While it works on a machine with a US locale, it may be parsed as invalid in any country using DD/MM/YYYY format.

Instead, use explicit syntax when constructing your test dates:

new Date(2016,10,28)

👍 to @maggiepint's comment. But also, make sure to recognize that months in that syntax are zero-based.

Oh, and don't edit moment.js directly. It will be generated as part of the build.

Thank you for the guidance, I am still new at this and I need all the help I can get. :) I will fix them today.

Merged in 03d7831