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

[liamjones]

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)

[palmerj3]

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

[liamjones]

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

[palmerj3]

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.

[work933k]

I think this issue should be marked as resolved.

[liamjones]

@work933k why? The issue still exists and the PR that potentially fixes it isn't merged yet.