Uploaded image for project: 'Jenkins'
  1. Jenkins
  2. JENKINS-15401

support TODO directive to not fail such tests

    Details

    • Similar Issues:

      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

          Hide
          kinow Bruno P. Kinoshita added a comment -

          +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

          Show
          kinow Bruno P. Kinoshita added a comment - +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
          Hide
          mika Michael Prokop added a comment -

          Hi Bruno,

          this sounds fantastic

          Thanks for taking care!

          regards,
          Michael

          Show
          mika Michael Prokop added a comment - Hi Bruno, this sounds fantastic Thanks for taking care! regards, Michael
          Hide
          mika 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

          Show
          mika 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
          Hide
          kinow Bruno P. Kinoshita added a comment -

          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

          Show
          kinow Bruno P. Kinoshita added a comment - 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
          Hide
          mika 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

          Show
          mika 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
          Hide
          kinow Bruno P. Kinoshita added a comment -

          Fixed in 1.8. This version will be available from update manager within few hours.

          Show
          kinow Bruno P. Kinoshita added a comment - Fixed in 1.8. This version will be available from update manager within few hours.

            People

            • Assignee:
              kinow Bruno P. Kinoshita
              Reporter:
              mika Michael Prokop
            • Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: