Branch: refs/heads/master
  Home:   https://github.com/jenkinsci/stash-pullrequest-builder-plugin
  Commit: d28f5adb62def244be11e6af10e68323a391f3e8
      
https://github.com/jenkinsci/stash-pullrequest-builder-plugin/commit/d28f5adb62def244be11e6af10e68323a391f3e8
  Author: Pavel Roskin <plros...@gmail.com>
  Date:   2019-07-11 (Thu, 11 Jul 2019)

  Changed paths:
    M 
src/test/java/stashpullrequestbuilder/stashpullrequestbuilder/StashRepositoryTest.java

  Log Message:
  -----------
  StashRepositoryTest: No need to stub getCiSkipPhrases() everywhere

Mockito returns null for unstubbed methods on mocked classes. It's OK for
getCiSkipPhrases() to return null, as it's only used in isSkipBuild()
that can deal with null.

Stub getCiSkipPhrases() only when it's relevant to the test.


-- 
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/stash-pullrequest-builder-plugin/push/refs/heads/master/d56749-d28f5a%40github.com.
For more options, visit https://groups.google.com/d/optout.

Reply via email to