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.