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

Create unit test for JobAction

XMLWordPrintable

    • Icon: Task Task
    • Resolution: Fixed
    • Icon: Minor Minor
    • warnings-ng-plugin
    • None

      See https://github.com/jenkinsci/warnings-ng-plugin/blob/master/plugin/src/main/java/io/jenkins/plugins/analysis/core/model/JobAction.java

      Not all methods need to be covered, we need to discuss which make sense to have a test case.

          [JENKINS-59619] Create unit test for JobAction

          Ulli Hafner created issue -
          Ulli Hafner made changes -
          Labels Original: help-wanted newbie-friendly New: help-wanted newbie-friendly testing-course
          Ulli Hafner made changes -
          Rank New: Ranked higher
          Ulli Hafner made changes -
          Description New: See https://github.com/jenkinsci/warnings-ng-plugin/blob/master/plugin/src/main/java/io/jenkins/plugins/analysis/core/model/JobAction.java

          Not all methods need to be covered, we need to discuss which make sense to have a test case.
          Ulli Hafner made changes -
          Labels Original: help-wanted newbie-friendly testing-course New: Mockito help-wanted newbie-friendly testing-course
          Ulli Hafner made changes -
          Rank New: Ranked lower

          Naveen Sundar added a comment -

          Seems like the unit test is already contributed by kasperhey and merged

          https://github.com/jenkinsci/warnings-ng-plugin/blob/master/plugin/src/test/java/io/jenkins/plugins/analysis/core/model/JobActionTest.java

          Naveen Sundar added a comment - Seems like the unit test is already contributed by kasperhey  and merged https://github.com/jenkinsci/warnings-ng-plugin/blob/master/plugin/src/test/java/io/jenkins/plugins/analysis/core/model/JobActionTest.java

          Ulli Hafner added a comment -

          Thanks for checking. This has been done by one of my students in the last semester. Seems that we forget to close the issue.

          Ulli Hafner added a comment - Thanks for checking. This has been done by one of my students in the last semester. Seems that we forget to close the issue.
          Ulli Hafner made changes -
          Resolution New: Fixed [ 1 ]
          Status Original: Open [ 1 ] New: Resolved [ 5 ]
          Ulli Hafner made changes -
          Labels Original: Mockito help-wanted newbie-friendly testing-course

            Unassigned Unassigned
            drulli Ulli Hafner
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

              Created:
              Updated:
              Resolved: