From 715436380334ff590faf8cf53f8f86fe5d15cd61 Mon Sep 17 00:00:00 2001 From: Christoph Kutzinski Date: Sat, 6 Jul 2013 11:26:14 +0200 Subject: [PATCH] This helps a lot when mocking AbstractProject in unit tests, because so I don't have to suppress the constructor with PowerMockito --- core/src/main/java/hudson/model/AbstractProject.java | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/core/src/main/java/hudson/model/AbstractProject.java b/core/src/main/java/hudson/model/AbstractProject.java index dd49267875..190d45ca67 100644 --- a/core/src/main/java/hudson/model/AbstractProject.java +++ b/core/src/main/java/hudson/model/AbstractProject.java @@ -266,7 +266,7 @@ public abstract class AbstractProject

,R extends A protected AbstractProject(ItemGroup parent, String name) { super(parent,name); - if(!Jenkins.getInstance().getNodes().isEmpty()) { + if(Jenkins.getInstance()!=null && !Jenkins.getInstance().getNodes().isEmpty()) { // if a new job is configured with Hudson that already has slave nodes // make it roamable by default canRoam = true; -- GitLab