Enhance platform requirements mechanism

- the new feature utilize regex to specify platform fixes adoptium#166 Signed-off-by: renfeiw <[email protected]>
renfeiw · Apr 12, 2021 · 3b367c3 · 3b367c3
1 parent 22b282d
commit 3b367c3
Show file tree

Hide file tree

Showing 2 changed files with 21 additions and 3 deletions.
diff --git a/src/org/testKitGen/TestInfo.java b/src/org/testKitGen/TestInfo.java
@@ -22,6 +22,7 @@
 public class TestInfo {
 	private String testCaseName;
 	private String command;
+	private String platform;
 	private String platformRequirements;
 	private List<Variation> vars;
 	private Map<String, String> capabilities;
@@ -37,6 +38,7 @@ public class TestInfo {
 	public TestInfo(Arguments arg) {
 		this.testCaseName = null;
 		this.command = null;
+		this.platform = null;
 		this.platformRequirements = null;
 		this.vars = new ArrayList<Variation>();
 		this.aotOptions = "";
@@ -72,6 +74,14 @@ public void setCommand(String command) {
 		this.command = command;
 	}
 
+	public String getPlatform() {
+		return this.platform;
+	}
+
+	public void setPlatform(String platform) {
+		this.platform = platform;
+	}
+
 	public String getPlatformRequirements() {
 		return this.platformRequirements;
 	}

diff --git a/src/org/testKitGen/TestInfoParser.java b/src/org/testKitGen/TestInfoParser.java
@@ -79,6 +79,11 @@ public TestInfo parse() {
 		}
 		if (!isValidAot) return null;
 
+		NodeList platNodes = testEle.getElementsByTagName("platform");
+		if (platNodes.getLength() > 0) {
+			ti.setPlatform(platNodes.item(0).getTextContent().trim());
+		}
+
 		NodeList preqNodes = testEle.getElementsByTagName("platformRequirements");
 		if (preqNodes.getLength() > 0) {
 			ti.setPlatformRequirements(preqNodes.item(0).getTextContent().trim());
@@ -89,12 +94,12 @@ public TestInfo parse() {
 		List<Variation> listOfVars = new ArrayList<Variation>();
 		for (int i = 0; i < variations.size(); i++) {
 			String subTestName = ti.getTestCaseName() + "_" + i;
-			Variation var = parseVariation(subTestName, variations.get(i), ti.getPlatformRequirements());
+			Variation var = parseVariation(subTestName, variations.get(i), ti.getPlatform(), ti.getPlatformRequirements());
 			listOfVars.add(var);
 		}
 		if (variations.size() == 0) {
 			String subTestName = ti.getTestCaseName() + "_0";
-			Variation var = parseVariation(subTestName, "NoOptions", ti.getPlatformRequirements());
+			Variation var = parseVariation(subTestName, "NoOptions", ti.getPlatform(), ti.getPlatformRequirements());
 			listOfVars.add(var);
 		}
 		ti.setVars(listOfVars);
@@ -214,6 +219,7 @@ private void parseDisableInfo(TestInfo ti) {
 	}
 
 	private boolean checkPlat(String plat) {
+		if (plat == null) return true;
 		Pattern pattern = Pattern.compile(plat);
 		Matcher matcher = pattern.matcher(arg.getPlat());
 		return matcher.matches();
@@ -262,7 +268,7 @@ private String joinStrList(List<String> list) {
 		return sb.toString();
 	}
 
-	private Variation parseVariation(String subTestName, String variation, String platformRequirements) {
+	private Variation parseVariation(String subTestName, String variation, String platform, String platformRequirements) {
 		Variation var = new Variation(subTestName, variation);
 
 		String jvmOptions = " " + variation + " ";
@@ -284,6 +290,8 @@ private Variation parseVariation(String subTestName, String variation, String pl
 			jvmOptions = jvmOptions.replace("Mode" + mode, clArgs);
 		}
 		jvmOptions = jvmOptions.trim();
+		isValid &= checkPlat(platform);
+		//TODO: remove platformRequirements
 		isValid &= checkPlatformReq(platformRequirements);
 
 		var.setJvmOptions(jvmOptions);