Suggested solution for two-fer doesn't pass tests.

[exercism]

Suggested solution for two-fer doesn't pass tests.

[tikaro]

TL:DR; SleeplessByte is correct; I had a stale spec file and didn't know it.

Aha! I see that you changed the test suite ten days ago in PR#649:
exercism/javascript#649

Previously, the spec used an intermediate variable:

const name = '';
expect(twoFer(name)).toEqual("One for you, one for me.")

...which passes '' to twoFer. Since there is a value present (a blank one), the default value is not triggered.

Now, the spec is:

expect(twoFer()).toEqual("One for you, one for me."

...which passes nothing/undefined to twoFer, and so the default value is triggered.

I had gotten it into my head that the "default value" solution was working for the regular function, but not for the arrow function. But I can't reproduce that behavior now; if I re-test with the older version of the spec, both the "regular" flavor and "arrow" flavor fail, as expected.

So this is definitely an issue caused by me running a stale spec.

Normally, I would depend on Git to tell me whether my local files were stale or not, but in this case since we're using a CLI instead of git, I don't have that guardrail. I'll need to remember to make sure that my specs are up to date before getting further into the weeds on language issues :)

Thanks very much for the rapid input, @SleeplessByte !

[SleeplessByte]

A quick sanity check is to first check your solution page at the bottom for a message that it's outdated. If it's not, then at least in the system you're working against the latest first. Now check the test stuite tab. If it doesn't match your local file, you're probably using something outdated locally and need to download again!

[SleeplessByte]

Default values always work.

Your test suite is likely outdated (and it's passing in '' instead of "nothing/undefined).