isEmpty logic and tests not coinciding about radash

[sodiray]

Hey @st-clair-clarke thanks for bringing this up and putting in the work to give all the details!

The isEmpty function is one I implemented to reach enough parity with Lodash so I could convince a team to use radash as a Lodash replacement. Now that I think about it, we ended up dumping isEmpty completely for a better solutions all together so this function has been unused AFAIK personally.

I'll mark this as a bug and get it fixed asap 🚀

[sodiray]

@st-clair-clarke just put up a PR to fix this. I went through your test cases and replicated them. The only one I'm not sure about is the Date tests. Logically, I agree with your test case that a valid Date object should be not-empty. I was validating these against Lodash and noticed they return true for any Date object...

Not that I care too much about what Lodash does. To be honest, I'm not sure what the implications are for other people trying to use radash. Specifically, those trying to directly replace Lodash with Radash.