提交 7b82a301 编写于 作者: A Alvaro Lobato

[FIX JENKINS-41457] Use BUILD_NOW_TEXT for parameterised jobs

上级 92720ab9
......@@ -278,7 +278,8 @@ public abstract class ParameterizedJobMixIn<JobT extends Job<JobT, RunT> & Param
* Uses {@link #BUILD_NOW_TEXT}.
*/
public final String getBuildNowText() {
return isParameterized() ? Messages.ParameterizedJobMixIn_build_with_parameters() : AlternativeUiTextProvider.get(BUILD_NOW_TEXT, asJob(), Messages.ParameterizedJobMixIn_build_now());
return isParameterized() ? AlternativeUiTextProvider.get(BUILD_NOW_TEXT, asJob(), Messages.ParameterizedJobMixIn_build_with_parameters())
: AlternativeUiTextProvider.get(BUILD_NOW_TEXT, asJob(), Messages.ParameterizedJobMixIn_build_now());
}
/**
......
......@@ -25,6 +25,8 @@ package hudson.util;
import hudson.model.AbstractProject;
import hudson.model.FreeStyleProject;
import hudson.model.ParametersDefinitionProperty;
import hudson.model.StringParameterDefinition;
import jenkins.model.ParameterizedJobMixIn;
import static org.hamcrest.Matchers.containsString;
import static org.junit.Assert.assertThat;
......@@ -58,9 +60,27 @@ public class AlternativeUiTextProviderTest {
*/
@Test
public void basics() throws Exception {
Impl.oldschool = false;
FreeStyleProject p = j.createFreeStyleProject("aaa");
assertThat(j.createWebClient().getPage(p).asText(), containsString("newschool:aaa"));
Impl.oldschool = true;
assertThat(j.createWebClient().getPage(p).asText(), containsString("oldschool:aaa"));
}
/**
* Makes sure that {@link AlternativeUiTextProvider} actually works with a parameterized Job.
*/
@Test
public void basicsWithParameter() throws Exception {
Impl.oldschool = false;
FreeStyleProject p = j.createFreeStyleProject("aaa");
p.addProperty(new ParametersDefinitionProperty(new StringParameterDefinition("FOO", null)));
String pageText = j.createWebClient().getPage(p).asText();
assertThat(pageText, containsString("newschool:aaa"));
Impl.oldschool = true;
pageText = j.createWebClient().getPage(p).asText();
assertThat(pageText, containsString("oldschool:aaa"));
}
}
Markdown is supported
0% .
You are about to add 0 people to the discussion. Proceed with caution.
先完成此消息的编辑!
想要评论请 注册