Remove deprecated static methods (#30)

* Remove deprecated static methods

* Update test cases

---------

Co-authored-by: theEvilReaper <theEvilReaper@users.noreply.github.com>

Author: theEvilReaper

src/main/java/net/theevilreaper/aves/map/BaseMap.java

@@ -46,45 +46,6 @@ public BaseMap(@NotNull String name, Pos spawn, String... builders) {
         this.spawn = spawn;
     }
 
-    /**
-     * Creates a new instance from the {@link BaseMap} with all given values.
-     * Deprecated since 1.9.0, use {@link #builder()} instead.
-     *
-     * @param name the name from the map
-     */
-    @Deprecated(forRemoval = true, since = "1.9.0")
-    @Contract(value = "_ -> new", pure = true)
-    public static @NotNull BaseMap of(@NotNull String name) {
-        return new BaseMap(name, null);
-    }
-
-    /**
-     * The constructor sets all relevant values for a map.
-     * Deprecated since 1.9.0, use {@link #builder()} instead.
-     *
-     * @param name  the name from the map
-     * @param spawn the spawn location from the map
-     */
-    @Deprecated(forRemoval = true, since = "1.9.0")
-    @Contract(value = "_, _ -> new", pure = true)
-    public static @NotNull BaseMap of(@NotNull String name, Pos spawn) {
-        return new BaseMap(name, spawn, "team");
-    }
-
-    /**
-     * The constructor sets all relevant values for a map.
-     * Deprecated since 1.9.0, use {@link #builder()} instead.
-     *
-     * @param name     the name from the map
-     * @param builders the builders from the map
-     * @param spawn    the spawn location from the map
-     */
-    @Contract(value = "_, _, _ -> new", pure = true)
-    @Deprecated(forRemoval = true, since = "1.9.0")
-    public static @NotNull BaseMap of(@NotNull String name, Pos spawn, String... builders) {
-        return new BaseMap(name, spawn, builders);
-    }
-
     /**
      * Creates a new instance of the {@link BaseMapBuilder} to build a new map.
      * The builder can be used to set all values that are required for a map.

src/test/java/net/theevilreaper/aves/file/FileHandlerTest.java

@@ -41,7 +41,7 @@ void testOtherConstructor() {
     @Test
     void testGsonFileHandlerWrite() {
         var path = tempDir.toPath().resolve(testMap);
-        var baseMap = BaseMap.of("TestMap");
+        var baseMap = new BaseMap("TestMap", null);
         baseMap.setBuilders("Builder1", "Builder2");
         fileHandler.save(path, baseMap);
         assertTrue(Files.exists(path));

src/test/java/net/theevilreaper/aves/file/ModernFileHandlerTest.java

@@ -46,7 +46,7 @@ void testOtherConstructor() {
     @Test
     void testGsonFileHandlerWrite() {
         var path = tempDir.toPath().resolve(testMap);
-        var baseMap = BaseMap.of("TestMap");
+        var baseMap = new BaseMap("TestMap", null);
         baseMap.setBuilders("Builder1", "Builder2");
         fileHandler.save(path, baseMap, TypeToken.get(BaseMap.class));
         assertTrue(Files.exists(path));
@@ -77,7 +77,7 @@ void testFileNotExistsRead() {
     @Test
     void testInvalidPathSave() {
         var path = tempDir.toPath();
-        var baseMap = BaseMap.of("TestMap");
+        var baseMap = new BaseMap("TestMap", null);
         var exception = assertThrows(
                 IllegalArgumentException.class,
                 () -> fileHandler.save(path, baseMap, TypeToken.get(BaseMap.class))

src/test/java/net/theevilreaper/aves/map/BaseMapTest.java

@@ -19,13 +19,13 @@ class BaseMapTest {
     @BeforeAll
     void init() {
         this.builders = new String[]{"theEvilReaper, Tresson", "SeelenRetterin"};
-        this.firstMap = BaseMap.of("Test");
-        this.secondMap = BaseMap.of("Test", new Pos(120, 51, 23), builders[0], builders[1]);
+        this.firstMap = new BaseMap("Test", null);
+        this.secondMap = new BaseMap("Test", new Pos(120, 51, 23), builders[0], builders[1]);
     }
 
     @Test
     void testOtherConstructor() {
-        var baseMap = BaseMap.of("Test", Pos.ZERO);
+        var baseMap = new BaseMap("Test", Pos.ZERO);
         assertSame("Test", baseMap.getName());
         assertSame(Pos.ZERO, baseMap.getSpawn());
     }
@@ -64,7 +64,7 @@ void testNameSetForMaps() {
 
     @Test
     void testEmptyNameConstructor() {
-        var exception = assertThrows(IllegalArgumentException.class, () -> BaseMap.of(""));
+        var exception = assertThrows(IllegalArgumentException.class, () -> new BaseMap("", null));
         assertEquals("The name can not be null or empty", exception.getMessage());
     }
 
@@ -94,7 +94,7 @@ void testSetSpawn() {
 
     @Test
     void testGetSpawnOrDefault() {
-        BaseMap baseMap = BaseMap.of("Test");
+        BaseMap baseMap = new BaseMap("Test", null);
         assertNull(baseMap.getSpawn());
 
         Pos defaultPos = new Pos(1, 2, 3);