Skip to content

Commit 57ab39b

Browse files
committed
[BUG]Don't set a java version string in the uploader
This isn't required for LARS, it is old code and it has caused bugs as it can only handle a limited set of java versions.
1 parent f74e500 commit 57ab39b

11 files changed

Lines changed: 0 additions & 887 deletions

File tree

upload-lib/build.gradle

Lines changed: 0 additions & 27 deletions
Original file line numberDiff line numberDiff line change
@@ -23,32 +23,5 @@ dependencies {
2323
testCompile project(':test-utils')
2424
}
2525

26-
Map<String, List<String>> javaVersionEsas = [
27-
'requires_java6.esa': ['requires_java6_bundle.jar'],
28-
'requires_java7.esa': ['requires_java7_bundle.jar'],
29-
'requires_java7_exact.esa': ['requires_java7_exact_bundle.jar'],
30-
'requires_incompatible_versions.esa': ['requires_java7_exact_bundle.jar',
31-
'requires_java8_bundle.jar'],
32-
'requires_multiple_versions.esa': ['requires_java6_bundle.jar',
33-
'requires_java7_bundle.jar'],
34-
'requires_bad_java_version.esa':['requires_bad_java_version_bundle.jar']
35-
]
36-
37-
javaVersionEsas.each({target, bundles ->
38-
Task task = task "create--${target}" (type:Zip) {
39-
archiveName = target
40-
destinationDir = file('build/test-working/resources')
41-
bundles.each({
42-
from file("build/test-working/resources/${it}")
43-
dependsOn "create--${it}"
44-
})
45-
from (file('src/fat/testResources/javaVersionTests/SUBSYSTEM.MF')) {
46-
into('OSGI-INF')
47-
}
48-
}
49-
prepareTestResources.dependsOn task
50-
prepareTestResources.outputs.files(task)
51-
})
52-
5326

5427
// vim: set si:

upload-lib/src/fat/java/com/ibm/ws/massive/esa/MassiveEsaTest.java

Lines changed: 0 additions & 92 deletions
Original file line numberDiff line numberDiff line change
@@ -51,7 +51,6 @@
5151
import java.util.zip.ZipInputStream;
5252

5353
import org.junit.Before;
54-
import org.junit.Ignore;
5554
import org.junit.Rule;
5655
import org.junit.Test;
5756

@@ -86,7 +85,6 @@
8685
import com.ibm.ws.repository.transport.client.DataModelSerializer;
8786
import com.ibm.ws.repository.transport.model.AppliesToFilterInfo;
8887
import com.ibm.ws.repository.transport.model.Asset;
89-
import com.ibm.ws.repository.transport.model.JavaSEVersionRequirements;
9088
import com.ibm.ws.repository.transport.model.WlpInformation;
9189

9290
/**
@@ -95,9 +93,6 @@
9593
*/
9694
public class MassiveEsaTest {
9795

98-
private static final String EE_7_FEATURE_COMPATIBILITY = "Java SE 7, Java SE 8";
99-
private static final String EE_6_FEATURE_COMPATIBILITY = "Java SE 6, Java SE 7, Java SE 8";
100-
10196
private static final String ENABLES = "Features that this feature enables";
10297
private static final String ENABLED_BY = "Features that enable this feature";
10398
private static final String SUPERSEDES = "Features that this feature supersedes";
@@ -183,93 +178,6 @@ public void testRealFeature() throws Throwable {
183178
assertEquals("Wrong attachment size", (long) esaSizes.get("wlp/lafiles/LA_zh"), license.getSize());
184179
}
185180

186-
@Test
187-
public void testJava6Feature() throws Throwable {
188-
File java6_esa = new File(esaDir, "requires_java6.esa");
189-
EsaResourceImpl java6Feature = (EsaResourceImpl) uploadAsset(java6_esa);
190-
191-
JavaSEVersionRequirements reqs = java6Feature.getJavaSEVersionRequirements();
192-
assertEquals("Incorrect minimum version", "1.6.0", reqs.getMinVersion());
193-
assertNull("Max version should be null, actually was: " + reqs.getMaxVersion(), reqs.getMaxVersion());
194-
assertEquals("Display version was incorrect", EE_6_FEATURE_COMPATIBILITY, reqs.getVersionDisplayString());
195-
}
196-
197-
@Test
198-
public void testJava7Feature() throws Throwable {
199-
File java7_esa = new File(esaDir, "requires_java7.esa");
200-
EsaResourceImpl java7Feature = (EsaResourceImpl) uploadAsset(java7_esa);
201-
202-
JavaSEVersionRequirements reqs = java7Feature.getJavaSEVersionRequirements();
203-
assertEquals("Incorrect minimum version", "1.7.0", reqs.getMinVersion());
204-
assertNull("Max version should be null, actually was: " + reqs.getMaxVersion(), reqs.getMaxVersion());
205-
assertEquals("Display version was incorrect", EE_7_FEATURE_COMPATIBILITY, reqs.getVersionDisplayString());
206-
}
207-
208-
/**
209-
* Test an esa where a bundle has an '=' requirement for the java version, rather than an
210-
* implied exact version from a combination of intersecting ranges of the various bundles.
211-
* Should still just end up with a minimum required, as maximums don't need to be specified
212-
*
213-
* @throws Throwable
214-
*/
215-
@Test
216-
public void testExactJavaRequirementFeature() throws Throwable {
217-
File broken_esa = new File(esaDir, "requires_java7_exact.esa");
218-
EsaResourceImpl java7Feature = (EsaResourceImpl) uploadAsset(broken_esa);
219-
JavaSEVersionRequirements reqs = java7Feature.getJavaSEVersionRequirements();
220-
assertEquals("Incorrect minimum version", "1.7.0", reqs.getMinVersion());
221-
assertEquals("Incorrect maximum version", "1.7.0", reqs.getMaxVersion());
222-
assertEquals("Display version was incorrect", EE_7_FEATURE_COMPATIBILITY, reqs.getVersionDisplayString());
223-
224-
}
225-
226-
@Test
227-
public void testIncompatibleVersionsFeature() throws Throwable {
228-
File java7_esa = new File(esaDir, "requires_incompatible_versions.esa");
229-
try {
230-
uploadAsset(java7_esa);
231-
} catch (RepositoryException e) {
232-
assertEquals("The message from the exception was unexpected.",
233-
"ESA requires_incompatible_versions.esa is invalid, two bundles require incompatible JavaSE versions"
234-
, e.getMessage());
235-
return;
236-
}
237-
fail("The ESA is invalid so an exception should have been thrown");
238-
}
239-
240-
@Test
241-
public void testMultipleVersionFeature() throws Throwable {
242-
File java7_esa = new File(esaDir, "requires_multiple_versions.esa");
243-
EsaResourceImpl java7Feature = (EsaResourceImpl) uploadAsset(java7_esa);
244-
245-
JavaSEVersionRequirements reqs = java7Feature.getJavaSEVersionRequirements();
246-
assertEquals("Incorrect minimum version", "1.7.0", reqs.getMinVersion());
247-
assertNull("Incorrect maxmimum version, should have been null", reqs.getMaxVersion());
248-
assertEquals("Incorrect minimum version", EE_7_FEATURE_COMPATIBILITY, reqs.getVersionDisplayString());
249-
}
250-
251-
/**
252-
* The Java version required by the ESA needs converting into something displayable. If the
253-
* required version is unexpected and hence can't be converted to a nice format for display, an
254-
* exception should be thrown
255-
*
256-
* @throws Throwable
257-
*/
258-
@Test
259-
public void testBadJavaVersionFeature() throws Throwable {
260-
File java7_esa = new File(esaDir, "requires_bad_java_version.esa");
261-
try {
262-
uploadAsset(java7_esa);
263-
} catch (RepositoryException e) {
264-
assertEquals("Incorrect message in the exception",
265-
"Lower bound to Java version range is expected to be either Java 6 or Java 7. Actually was 5.9.0", e.getMessage());
266-
return;
267-
}
268-
269-
fail("An exception should be thrown if an unexpected Java version is required");
270-
271-
}
272-
273181
@Test
274182
public void testMainAttachment() throws Throwable {
275183
File simpleESA = new File(esaDir, "simple.esa");

upload-lib/src/fat/testResources/javaVersionTests/SUBSYSTEM.MF

Lines changed: 0 additions & 6 deletions
This file was deleted.

upload-lib/src/fat/testResources/requires_bad_java_version_bundle.jar/META-INF/MANIFEST.MF

Lines changed: 0 additions & 3 deletions
This file was deleted.

upload-lib/src/fat/testResources/requires_java6_bundle.jar/META-INF/MANIFEST.MF

Lines changed: 0 additions & 3 deletions
This file was deleted.

upload-lib/src/fat/testResources/requires_java7_bundle.jar/META-INF/MANIFEST.MF

Lines changed: 0 additions & 3 deletions
This file was deleted.

upload-lib/src/fat/testResources/requires_java7_exact_bundle.jar/META-INF/MANIFEST.MF

Lines changed: 0 additions & 2 deletions
This file was deleted.

upload-lib/src/fat/testResources/requires_java7_or_less_bundle.jar/META-INF/MANIFEST.MF

Lines changed: 0 additions & 3 deletions
This file was deleted.

upload-lib/src/fat/testResources/requires_java8_bundle.jar/META-INF/MANIFEST.MF

Lines changed: 0 additions & 3 deletions
This file was deleted.

0 commit comments

Comments
 (0)