Branch: refs/heads/master
  Home:   https://github.com/jenkinsci/embeddable-build-status-plugin
  Commit: 221c5b4c8fd9242667c952edf6a898888afc1ef9
      
https://github.com/jenkinsci/embeddable-build-status-plugin/commit/221c5b4c8fd9242667c952edf6a898888afc1ef9
  Author: Vandit Singh <107131545+vandit1...@users.noreply.github.com>
  Date:   2023-02-11 (Sat, 11 Feb 2023)

  Changed paths:
    A 
src/test/java/org/jenkinsci/plugins/badge/actions/PublicBuildStatusActionTest.java

  Log Message:
  -----------
  add test for PublicBuildStatusAction.java (#163)

* add test for PublicBuildStatusAction.java

* delete testDoText

* Use web client instead of mocks to call method

Web client is provided by JenkinsRule and avoids mocking in the tests.

Use a single JenkinsRule for the entire test class so that we don't
pay the JenkinsRule startup cost for every test.  JenkinsRule runs
slowly on Windows, so it is better to use a class level JenkinsRule
when tests are independent of each other.

Keep the tests independent of each other by defining the test job name
based on the name of the test method.

Use hamcrest assertions for easier diagnosis and simpler reading
of the assertions.

Format the source file with `mvn spotless:apply` so that the file
is consistent with the source formatting in the rest of the plugin.

* Use a real implementation of isWindows()

---------

Co-authored-by: Mark Waite <mark.earl.wa...@gmail.com>


-- 
You received this message because you are subscribed to the Google Groups 
"Jenkins Commits" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to jenkinsci-commits+unsubscr...@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/jenkinsci-commits/jenkinsci/embeddable-build-status-plugin/push/refs/heads/master/127c05-221c5b%40github.com.

Reply via email to