Fix snapshot documentation links.
Documentation links now contain the proper snapshot suffix. Closes #91
This commit is contained in:
@@ -318,12 +318,17 @@ public class ArtifactVersion implements Comparable<ArtifactVersion> {
|
||||
return modifierFormat ? SNAPSHOT_MODIFIER : SNAPSHOT_SUFFIX;
|
||||
}
|
||||
|
||||
public String getMajorMinor() {
|
||||
public String getMajorMinor(boolean includeSuffix) {
|
||||
|
||||
if (includeSuffix && isSnapshotVersion()) {
|
||||
return String.format("%s.%s-SNAPSHOT", version.getMajor(), version.getMinor());
|
||||
}
|
||||
|
||||
return String.format("%s.%s", version.getMajor(), version.getMinor());
|
||||
}
|
||||
|
||||
public String getGeneration() {
|
||||
return String.format("%s.x", getMajorMinor());
|
||||
return String.format("%s.x", getMajorMinor(false));
|
||||
}
|
||||
|
||||
@Override
|
||||
|
||||
@@ -114,7 +114,7 @@ public class DocumentationMetadata {
|
||||
}
|
||||
|
||||
String format = this.project.isCommercial() ? COMMERCIAL_ANTORA_BASE : ANTORA_BASE;
|
||||
return String.format(format, getProjectName(project), version.getMajorMinor());
|
||||
return String.format(format, getProjectName(project), this.version.getMajorMinor(true));
|
||||
}
|
||||
|
||||
public String getVersionOrTrainName(Train train) {
|
||||
|
||||
@@ -48,6 +48,17 @@ class DocumentationMetadataUnitTests {
|
||||
assertThat(metadata.getReferenceDocUrl()).isEqualTo("https://docs.spring.io/spring-data/mongodb/reference/3.1/");
|
||||
}
|
||||
|
||||
@Test // gh-91
|
||||
void shouldReportCorrectAntoraSnapshotDocumentationUrls() {
|
||||
|
||||
DocumentationMetadata metadata = DocumentationMetadata.of(DocumentationFormat.ANTORA,
|
||||
SupportedProject.of(Projects.MONGO_DB, SupportStatus.OSS), ArtifactVersion.of("3.1.0-SNAPSHOT"), false);
|
||||
|
||||
assertThat(metadata.getApiDocUrl()).isEmpty();
|
||||
assertThat(metadata.getReferenceDocUrl())
|
||||
.isEqualTo("https://docs.spring.io/spring-data/mongodb/reference/3.1-SNAPSHOT/");
|
||||
}
|
||||
|
||||
@Test // gh-197
|
||||
void shouldReportCorrectCurrentDocumentationUrls() {
|
||||
|
||||
|
||||
Reference in New Issue
Block a user