Support a string argument to "ignore" describing the reason about test-case HOT 5 CLOSED

I think I'm leaning towards [] as proposed right now. I don't think that any of these is better, but
=> ignore = "reason" "result" may be a bit confusing.

from test-case.

Hmm, yeah, it's definitelly something missing. I remember ignore comment syntax was present since 1.3x or earlier, so it shouldn't be a breaking change, we have MSRV 1.49 anyway. It just wasn't thought of.

I like your second proposal, if you have resources and time PR is welcome.

from test-case.

I assumed we want the reason to be optional, for backward compat so that existing cases do not all need updating and can still use a bare inconclusive. Given that then using () to surround the reason argument was ambiguous -- since => inconclusive () should mean a test with output () but is treated as one with an empty reason and no output specified. Similar problems exist with => inconclusive ("this is the result") where this is the result ended up as the reason not the expected result...

I went with [] instead and will open a PR in a moment, {} would be an easy change if preferred. Other things (like <>) would need more custom handling.

from test-case.

What about => inconclusive = "this is reason" ("this is result")? @luke-biel would that be ambiguous or/and better?

from test-case.

I think that would be unambiguous? At least I don't think there are any output matchers which start with an = (nor an == FWIW` although I don't think that would be ambiguous even if there was).

I guess it's a question of taste/style and perhaps future direction, happy to defer to you both.

from test-case.