Skip to content

Commit eeb511a

Browse files
committed
Fixed iteration detection in JiraVersion.
1 parent e77f9b0 commit eeb511a

File tree

3 files changed

+11
-8
lines changed

3 files changed

+11
-8
lines changed

src/main/java/org/springframework/data/release/jira/JiraVersion.java

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -46,7 +46,8 @@ public JiraVersion(ModuleIteration moduleIteration) {
4646
@Override
4747
public String toString() {
4848

49-
Iteration iteration = module.hasCustomFirstIteration() ? module.getCustomFirstIteration() : this.iteration;
49+
Iteration iteration = this.iteration.isInitialIteration() && module.hasCustomFirstIteration() ? module
50+
.getCustomFirstIteration() : this.iteration;
5051

5152
if (iteration.isServiceIteration()) {
5253
return String.format("%s.%s (%s %s)", module.getVersion(), iteration.getBugfixValue(), train.getName(),

src/main/java/org/springframework/data/release/model/ModuleIteration.java

Lines changed: 0 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -56,13 +56,6 @@ public Iteration getIteration() {
5656
.getCustomFirstIteration() : this.iteration;
5757
}
5858

59-
public String getJiraVersionName() {
60-
61-
Iteration iteration = module.hasCustomFirstIteration() ? module.getCustomFirstIteration() : this.iteration;
62-
63-
return String.format("%s %s (%s)", module.getVersion(), iteration.getName(), train.getName());
64-
}
65-
6659
/**
6760
* Returns the {@link String} representation of the logical version of the {@link ModuleIteration}.
6861
*

src/test/java/org/springframework/data/release/jira/JiraVersionUnitTests.java

Lines changed: 9 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -52,6 +52,15 @@ public void usesCustomModuleIterationStartVersion() {
5252
assertThat(version.toString(), is("1.0 M2 (Dijkstra)"));
5353
}
5454

55+
@Test
56+
public void doesNotUseCustomIterationOnNonFirstiterations() {
57+
58+
ModuleIteration module = ReleaseTrains.DIJKSTRA.getModuleIteration(Iteration.RC1, "Elasticsearch");
59+
60+
JiraVersion version = new JiraVersion(module);
61+
assertThat(version.toString(), is("1.0 RC1 (Dijkstra)"));
62+
}
63+
5564
private void assertIterationVersion(Iteration iteration, String expected) {
5665

5766
ModuleIteration module = ReleaseTrains.DIJKSTRA.getModuleIteration(iteration, "Commons");

0 commit comments

Comments
 (0)