diff --git a/pom.xml b/pom.xml index d2af9a1c..2c1286c5 100644 --- a/pom.xml +++ b/pom.xml @@ -56,6 +56,18 @@ import pom + + + org.jenkins-ci.plugins.workflow + workflow-step-api + 655.v6e098b_87b_84f + + + org.jenkins-ci.plugins.workflow + workflow-step-api + tests + 655.v6e098b_87b_84f + @@ -70,7 +82,6 @@ org.jenkins-ci.plugins.workflow workflow-step-api - 655.v6e098b_87b_84f org.jenkins-ci.plugins diff --git a/src/main/java/org/jenkinsci/plugins/credentialsbinding/impl/BindingStep.java b/src/main/java/org/jenkinsci/plugins/credentialsbinding/impl/BindingStep.java index d57589a5..7bbbd55a 100644 --- a/src/main/java/org/jenkinsci/plugins/credentialsbinding/impl/BindingStep.java +++ b/src/main/java/org/jenkinsci/plugins/credentialsbinding/impl/BindingStep.java @@ -127,7 +127,7 @@ private void doStart() throws Exception { for (MultiBinding binding : step.bindings) { if (binding.getDescriptor().requiresWorkspace() && (workspace == null || launcher == null)) { - throw new MissingContextVariableException(FilePath.class); + throw new MissingContextVariableException(FilePath.class, step.getDescriptor()); } MultiBinding.MultiEnvironment environment = binding.bind(run, workspace, launcher, listener); unbinders.add(environment.getUnbinder()); diff --git a/src/test/java/org/jenkinsci/plugins/credentialsbinding/impl/BindingStepTest.java b/src/test/java/org/jenkinsci/plugins/credentialsbinding/impl/BindingStepTest.java index 13c71866..c246ce67 100644 --- a/src/test/java/org/jenkinsci/plugins/credentialsbinding/impl/BindingStepTest.java +++ b/src/test/java/org/jenkinsci/plugins/credentialsbinding/impl/BindingStepTest.java @@ -218,7 +218,7 @@ public void widerRequiredContext() throws Throwable { WorkflowRun b = r.assertBuildStatus(Result.FAILURE, p.scheduleBuild2(0)); r.assertLogNotContains("We should fail before getting here", b); r.assertLogContains("Required context class hudson.FilePath is missing", b); - r.assertLogContains("Perhaps you forgot to surround the step with a step that provides this, such as: node", b); + r.assertLogContains("Perhaps you forgot to surround the withCredentials step with a step that provides this, such as: node", b); }); }