Merge pull request #85 from DerMistkaefer/feature/run-tests

Enabling Automated Tests in Maven Build & Repairing Failed Tests

pom.xml

@@ -26,6 +26,16 @@
  </plugins>
  </pluginManagement>
  <plugins>
+ <plugin>
+ <artifactId>maven-surefire-plugin</artifactId>
+ <version>2.22.2</version>
+ </plugin>
+
+ <plugin>
+ <artifactId>maven-failsafe-plugin</artifactId>
+ <version>2.22.2</version>
+ </plugin>
+
  <plugin>
  <artifactId>maven-shade-plugin</artifactId>
  <configuration>

src/main/java/de/leonhard/storage/internal/editor/toml/TomlManager.java

@@ -4,6 +4,7 @@
 import de.leonhard.storage.util.FastStringWriter;
 import java.io.*;
 import java.nio.charset.StandardCharsets;
+import java.nio.file.Files;
 import java.time.format.DateTimeFormatter;
 import java.time.format.DateTimeFormatterBuilder;
 import java.util.Map;
@@ -159,7 +160,7 @@ public Map<String, Object> read(final File file) throws IOException, TomlExcepti
  */
  public Map<String, Object> read(final File file, final boolean strictAsciiBareKeys)
  throws IOException, TomlException {
- return read(new FileInputStream(file), strictAsciiBareKeys);
+ return read(Files.newInputStream(file.toPath()), strictAsciiBareKeys);
  }
 
  /**

src/test/java/de/leonhard/storage/ConfigTest.java

@@ -3,6 +3,7 @@
 import de.leonhard.storage.annotation.ConfigPath;
 import de.leonhard.storage.internal.exceptions.SimplixValidationException;
 import de.leonhard.storage.internal.settings.DataType;
+import lombok.Getter;
 import org.junit.jupiter.api.AfterAll;
 import org.junit.jupiter.api.Assertions;
 import org.junit.jupiter.api.BeforeEach;
@@ -89,25 +90,18 @@ void testAnnotations() {
  }
 
  @AfterAll
- @Test
  static void tearDown() {
  config.clear();
  Assertions.assertTrue(config.getFile().delete());
  }
 
+ @Getter
  static class AnnotationTests {
 
  @ConfigPath("annotation-test")
  public String annotationTest;
+
  @ConfigPath("section.annotations")
  public Integer annotationTest2;
-
- public String getAnnotationTest() {
- return annotationTest;
- }
-
- public int getAnnotationTest2() {
- return annotationTest2;
- }
  }
 }

src/test/java/de/leonhard/storage/JsonTest.java

@@ -17,12 +17,12 @@ public class JsonTest {
  @Test
  void setUp() {
  json = new Json("Example", "");
- Assertions.assertEquals("Example.yml", json.getName());
+ Assertions.assertEquals("Example.json", json.getName());
  }
 
  @Test
  void testGetDataType() {
- Assertions.assertEquals(json.getDataType(), DataType.SORTED);
+ Assertions.assertEquals(json.getDataType(), DataType.UNSORTED);
  }
 
  @Test
@@ -68,8 +68,7 @@ void testSet() {
  }
 
  @AfterAll
- @Test
- void tearDown() {
+ static void tearDown() {
  json.clear();
  Assertions.assertTrue(json.getFile().delete());
  }

src/test/java/de/leonhard/storage/TomlTest.java

@@ -17,12 +17,12 @@ public class TomlTest {
  @Test
  void setUp() {
  toml = new Toml("Example", "");
- Assertions.assertEquals("Example.yml", toml.getName());
+ Assertions.assertEquals("Example.toml", toml.getName());
  }
 
  @Test
  void testGetDataType() {
- Assertions.assertEquals(DataType.SORTED, toml.getDataType());
+ Assertions.assertEquals(DataType.UNSORTED, toml.getDataType());
  }
 
  @Test
@@ -68,8 +68,7 @@ void testSet() {
  }
 
  @AfterAll
- @Test
- void tearDown() {
+ static void tearDown() {
  toml.clear();
  Assertions.assertTrue(toml.getFile().delete());
  }

src/test/java/de/leonhard/storage/YamlTest.java

@@ -26,7 +26,7 @@ void setUp() {
 
  @Test
  void testGetDataType() {
- Assertions.assertEquals(DataType.SORTED, yaml.getDataType());
+ Assertions.assertEquals(DataType.UNSORTED, yaml.getDataType());
  }
 
  @Test
@@ -78,8 +78,7 @@ void testSet() {
  }
 
  @AfterAll
- @Test
- void tearDown() {
+ static void tearDown() {
  yaml.clear();
  Assertions.assertTrue(yaml.getFile().delete());
  }