feat(runner): allow YAML testcase files

Author: Valentin Vetter

runner/pom.xml

@@ -1,4 +1,5 @@
-<project xmlns="http://maven.apache.org/POM/4.0.0" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
+<project xmlns="http://maven.apache.org/POM/4.0.0"
+	xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
 	xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 http://maven.apache.org/xsd/maven-4.0.0.xsd">
 	<modelVersion>4.0.0</modelVersion>
 
@@ -45,6 +46,11 @@
 			<artifactId>commons-lang3</artifactId>
 			<version>3.5</version>
 		</dependency>
+		<dependency>
+			<groupId>org.yaml</groupId>
+			<artifactId>snakeyaml</artifactId>
+			<version>1.24</version>
+		</dependency>
 	</dependencies>
 
 </project>

runner/src/main/java/com/codingame/gameengine/runner/ConfigHelper.java

@@ -67,7 +67,7 @@ public void setLeaguesDetected(boolean leaguesDetected) {
         }
     }
 
-    class TestCase {
+    public static class TestCase {
         private Map<Integer, String> title;
         private String testIn;
         private Boolean isTest;

runner/src/main/java/com/codingame/gameengine/runner/SoloGameRunner.java

@@ -3,14 +3,16 @@
 import java.io.File;
 import java.io.IOException;
 import java.nio.charset.StandardCharsets;
-import java.util.ArrayList;
 import java.util.Arrays;
 import java.util.List;
 
 import org.apache.commons.io.FileUtils;
+import org.apache.commons.io.FilenameUtils;
+import org.yaml.snakeyaml.Yaml;
 
-import com.google.gson.JsonObject;
-import com.google.gson.JsonParser;
+import com.codingame.gameengine.runner.ConfigHelper.TestCase;
+import com.google.gson.Gson;
+import com.google.gson.JsonSyntaxException;
 
 /**
  * The class to use to run local games and display the replay in a webpage on a temporary local server.
@@ -27,18 +29,33 @@ public SoloGameRunner() {
     }
 
     private List<String> getLinesFromTestCaseFile(File file) {
-        List<String> lines = new ArrayList<>();
+        TestCase testCase;
         try {
-            JsonObject testCaseJson = new JsonParser().parse(FileUtils.readFileToString(file, StandardCharsets.UTF_8)).getAsJsonObject();
-            lines.addAll(Arrays.asList(testCaseJson.get("testIn").getAsString().split("\\n")));
+            testCase = parseTestCaseFile(file);
         } catch (IOException e) {
-            throw new RuntimeException("Cannot read file", e);
-        } catch (NullPointerException e) {
-            throw new RuntimeException("Cannot find \"testIn\" property");
+            throw new RuntimeException("Cannot read file " + file.getName(), e);
         } catch (Exception e) {
-            throw new RuntimeException("Cannot parse file", e);
+            throw new RuntimeException("Cannot parse file " + file.getName(), e);
+        }
+
+        if (testCase.getTestIn() == null) {
+            throw new RuntimeException("Cannot find \"testIn\" property");
+        }
+
+        return Arrays.asList(testCase.getTestIn().split("\\n"));
+    }
+
+    private TestCase parseTestCaseFile(File file) throws JsonSyntaxException, IOException {
+        String extension = FilenameUtils.getExtension(file.getName());
+
+        switch (extension.toLowerCase()) {
+        case "yaml":
+        case "yml":
+            return new Yaml().loadAs(FileUtils.readFileToString(file, StandardCharsets.UTF_8), TestCase.class);
+        case "json":
+        default:
+            return new Gson().fromJson(FileUtils.readFileToString(file, StandardCharsets.UTF_8), TestCase.class);
         }
-        return lines;
     }
 
     /**
@@ -66,10 +83,10 @@ public void setTestCase(File testCaseFile) {
         if (!testCaseFile.isFile()) {
             throw new RuntimeException("Given test case is not a file " + testCaseFile.getAbsolutePath());
         }
-        
+
         setTestCaseInput(getLinesFromTestCaseFile(testCaseFile));
     }
-    
+
     /**
      * Sets a list of <code>String</code> as a test case input that will be sent to the Game Manager.
      * 
@@ -79,7 +96,7 @@ public void setTestCase(File testCaseFile) {
     public void setTestCaseInput(List<String> testCaseInput) {
         this.testCaseInput = testCaseInput;
     }
-    
+
     /**
      * Sets a <code>String</code> as a test case input that will be sent to the Game Manager.
      * <p>