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

support TODO directive to not fail such tests

    XMLWordPrintable

Details

    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

          mika Michael Prokop created issue -

          +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

          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
          kinow Bruno P. Kinoshita made changes -
          Field Original Value New Value
          Status Open [ 1 ] In Progress [ 3 ]

          Hi Bruno,

          this sounds fantastic

          Thanks for taking care!

          regards,
          Michael

          mika Michael Prokop added a comment - Hi Bruno, this sounds fantastic Thanks for taking care! regards, Michael
          kinow Bruno P. Kinoshita made changes -
          Status In Progress [ 3 ] Open [ 1 ]
          kinow Bruno P. Kinoshita made changes -
          Resolution Fixed [ 1 ]
          Status Open [ 1 ] Resolved [ 5 ]

          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

          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

          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

          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

          Hi Bruno, no reason to be sorry Thanks a lot, very much looking forward to the new TAP plugin release!

          regards,
          Michael

          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

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

          kinow Bruno P. Kinoshita added a comment - Fixed in 1.8. This version will be available from update manager within few hours.
          kinow Bruno P. Kinoshita made changes -
          Status Resolved [ 5 ] Closed [ 6 ]
          rtyler R. Tyler Croy made changes -
          Workflow JNJira [ 146135 ] JNJira + In-Review [ 206190 ]

          People

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

            Dates

              Created:
              Updated:
              Resolved: