assert-rs / snapbox

Snapshot testing for a herd of CLI tests

docs.rs/trycmd

Apache License 2.0

138 stars 18 forks source link

suggested improvement to 'ignored' message #245

Closed thor314 closed 9 months ago

thor314 commented 9 months ago

per #244, suggesting a slight change to "ignored". It took me some time to figure out why my tests weren't running, a slightly more informative message than "ignored" may save some time.

epage commented 9 months ago

imo this can get too repetitive and deviates too much from standard test output. I'd recommend finding an existing issue or opening a new issue for us to brainstorm on how to improve this.

AucaCoyan commented 9 months ago

It happened exactly the same to me. Is there any discussion issue or should I create one?

epage commented 9 months ago

105 is the only ignored-related issue. If that is relevant, feel free to comment on there, otherwise feel free to create a new one

doomdagadiggiedahdah commented 9 months ago

I had this same issue. Didn't get it to work until seeing this thread