jest-junit icon indicating copy to clipboard operation
jest-junit copied to clipboard

Feature Request: Mark 'todo' tests as 'skipped'

Open liamjones opened this issue 2 years ago • 5 comments

Currently, if you have a todo test:

it.todo('test not written yet')

It shows up as as success in the XML report:

<testcase classname="" name="test not written yet" time="0">
</testcase>

It feels like these should show the same as it.skip tests, skipped in the XML (since I don't believe there's any way of marking something as todo/pending in the XML schema)

liamjones avatar Feb 03 '23 11:02 liamjones

Is the fact that it shows up as a passing test causing a problem on your end?

palmerj3 avatar Feb 15 '23 22:02 palmerj3

Not as far as things failing to work, no. Only insofar as it feels like its misleading to say "test not written yet" passed.

liamjones avatar Feb 16 '23 09:02 liamjones

Yeah I tend to agree. I think I'd be open to either not having them in the report at all, or shown as skipped.

Feel free to submit a PR and I'll review! Otherwise I'll get to this soon.

palmerj3 avatar Feb 16 '23 13:02 palmerj3