ES6 Kata 69 "Reflect - defineProperty" kata needs updating about katas HOT 4 CLOSED

its the same thing as this, issue, right?
https://twitter.com/mkuehnel/status/769661864464748544
any suggestions to fix it?

from katas.

These are not quite the same issue, as one requires an array-like (Reflect.apply), and the other requires an object (Reflect.defineProperty), but the issues are rooted in the same general place - The third param cannot be omitted when demonstrating how the functions work.

A potential solution is to amend your approach to teaching the user how these functions work. Instead of introducing each param separately, each param must be included in each function call.

For example, there can't be any tests where the third param is undefined (Unless you want to indicate that the function will throw if the third param is not an object).

The test 'returns false -> when no property name is given (since no property has been added)' is not valid since the function will throw if you don't give an object as the third param. However, if you do provide an object as the third param, but the second param is "" (empty string), it will still not return false, because the property will be defined on the key "" (empty string) of the given object.

from katas.

@wolframkriesing our twitter conversation is related to issue no. 31 (as stated on top of this site by @csakai )

Possible solution if you want to keep the one param per test approach could be a wrapper for the functions where the params are optional, but I'm not sure if that might cause more confusion instead of being a real help.
As Chris said, the problems root is the same, so this approach is applicable on both issues.

from katas.

i believe I fixed it in #44 correct?
If not please reopen this issue. thx

from katas.