Modulus Errors

[Tuxhedoh]

PSKoans/PSKoans/Koans/Foundations/AboutAssignmentAndArithmetic.Koans.ps1

Line 136 in 87d30da

} | Should -Throw -ExpectedMessage __

I understand what you're going for here, but I'm not convinced that this is a good excersize. I hate pointing out something without offering a fix.

[vexx32]

Is there a more appropriate way to illustrate the limitations of operators, do you think? 🙂

[Tuxhedoh]

Not really.... I know my experience with POSH. I'm not sure someone with less experience would be able to pass this test. I had to jump into the terminal and run the line and look at $error[0]. Not sure how else to see this. Maybe I am the target audience. If that's the case... than maybe it's perfectly fine.

[vexx32]

Does running the Measure-Karma command when at this test point not show the expected error message?

[Tuxhedoh]

I'm guessing it does, though didn't think that was a valid way of finding the answer.

…

On Nov 15, 2018, 4:32 PM, at 4:32 PM, vexx32 ***@***.***> wrote: Does running the `Measure-Karma` command when at this test point not show the expected error message? -- You are receiving this because you authored the thread. Reply to this email directly or view it on GitHub: #92 (comment)

[vexx32]

In this case, I think it is. 😄

It's more intended to simply illustrate that a) an error will be thrown, and b) what that error looks like, so folks can learn a little of what to look for when their code breaks in certain ways. 😁

[Tuxhedoh]

Of course if they just run the code, they don't even see that error message by default.

I answered this with System.Management.Automation.RuntimeException, because that's the actual exceptiontype (I think). Where it looks like you're expecting the Exception.Message. I don't know. Just offering feedback.

[vexx32]

I think I've already got that changed in the latest commit to use -ExpectedMessage, yeah. Actually, that's doubly a bug because there is a type called __ but not one ___ so that message would be much clearer if I had typed the dummy type name properly 😂

[vexx32]

I believe this has been addressed in the recent release(s). 😄