Confusing error message in UnitIsUsedWhenThereIsNoReturnValueForAFunction

ChrisMarinos / FSharpKoans

A simple, fun, and interactive way to learn the F# language through testing.

Other

1.28k stars 671 forks source link

Confusing error message in UnitIsUsedWhenThereIsNoReturnValueForAFunction #24

Closed relentless closed 7 years ago

relentless commented 10 years ago

When this test fails, the message given is 'Expected: null'.

Looking at the code, I think what people are expected to put in is '()', but based on that message people often enter 'null', which also works but people find confusing.

Maybe if it's not possible to change the message, we could add a more explanatory comment or update the test in some way.

dsyme commented 7 years ago

Added comment to clarify this here: https://github.com/ChrisMarinos/FSharpKoans/pull/51