Details
-
Bug
-
Status: Closed (View Workflow)
-
Major
-
Resolution: Fixed
Description
Quoting
http://testanything.org/wiki/index.php/TAP_version_13_specification#TODO_tests the spec says:
TODO tests
If the directive starts with # TODO, the test is counted as a todo test, and the text after TODO is the explanation.
not ok 13 # TODO bend space and time
Note that if the TODO has an explanation it must be separated from TODO by a space.
These tests represent a feature to be implemented or a bug to be fixed and act as something of an executable "things to do" list. They are not expected to succeed. Should a todo test point begin succeeding, the harness should report it as a bonus. This indicates that whatever you were supposed to do has been done and you should promote this to a normal test point.
AFAICT the TAP plugin doesn't seem to take care of the TODO directive accordingly though, therefore marking test runs as failures even though they should be fine.
Example:
http://michael-prokop.at/screeni/gkrellShoot_12-10-04_165641.png
Attachments
Activity
Michael Prokop
added a comment - Hi Bruno,
this sounds fantastic 
Thanks for taking care!
regards,
Michael
Michael Prokop
added a comment - Hi Bruno,
this sounds fantastic
Thanks for taking care!
regards,
Michael Michael Prokop
added a comment - Hi Bruno,
thanks for implementing this feature. Is there any chance that you might release a new version of the TAP plugin soon?
thanks && regards,
Michael
Michael Prokop
added a comment - Hi Bruno,
thanks for implementing this feature. Is there any chance that you might release a new version of the TAP plugin soon?
thanks && regards,
Michael Hi Michael! Sorry!!!! I couldn't release the plug-in live at Jenkins Meetup Sao Paulo (technical problems with my graphic card).
I'll try to release a new version tonight.
Sorry again.
Bruno
Michael Prokop
added a comment - Hi Bruno, no reason to be sorry Thanks a lot, very much looking forward to the new TAP plugin release!
regards,
Michael
Michael Prokop
added a comment - Hi Bruno, no reason to be sorry Thanks a lot, very much looking forward to the new TAP plugin release!
regards,
Michael Fixed in 1.8. This version will be available from update manager within few hours.
+1 thanks for reporting this issue Michael.
I'll add an option so that users can decide whether a TODO directive should be interpreted as failed or not. The default behavior will be follow the TAP specification, as you mentioned in the issue description.
Does that sound like a good idea for you?
Cheers, B