Skip to content

Commit

Permalink
[JENKINS-25917] [JENKINS-32377] Let cookie test case run with 2.x only.
Browse files Browse the repository at this point in the history
Actually use a job name in the cookie test case that contains spaces.
  • Loading branch information
uhafner committed May 9, 2016
1 parent fa45d4c commit 75c8f77
Showing 1 changed file with 4 additions and 6 deletions.
10 changes: 4 additions & 6 deletions src/test/java/plugins/AbstractAnalysisTest.java
Expand Up @@ -18,6 +18,7 @@
import org.custommonkey.xmlunit.XMLUnit;
import org.jenkinsci.test.acceptance.junit.AbstractJUnitTest;
import org.jenkinsci.test.acceptance.junit.Resource;
import org.jenkinsci.test.acceptance.junit.Since;
import org.jenkinsci.test.acceptance.plugins.analysis_core.AnalysisAction;
import org.jenkinsci.test.acceptance.plugins.analysis_core.AnalysisAction.Tab;
import org.jenkinsci.test.acceptance.plugins.analysis_core.AnalysisConfigurator;
Expand Down Expand Up @@ -201,15 +202,12 @@ protected void assertThatProjectPageTrendIsCorrect(final FreeStyleJob job, final
* to re-enable the trend. Finally, this link is clicked in order open the trend configuration again.
*/
@Test
@Issue({"JENKINS-25917", "JENKINS-32377"})
@Issue({"JENKINS-25917", "JENKINS-32377"}) @Since("2.0")
public void should_store_trend_selection_in_cookie() {
FreeStyleJob job = buildJobTwoTimesInARow();

// FIXME: renaming of a job does change the name but not the URL
// job.configure();
// job.setName(job.name.replace("_", " "));
// job.save();
// clickButton("Yes");
assertThat(job.name, containsString("_"));
job = job.renameTo(job.name.replace("_", " "));

AnalysisAction action = createProjectAction(job);
verifyTrendGraphOverview(job, action);
Expand Down

0 comments on commit 75c8f77

Please sign in to comment.