refactor: removed deprecated loadCursorImage()

AlmasB · AlmasB · commit ec3f7dcbc432 · 2023-03-22T23:00:19.000Z
diff --git a/fxgl/src/main/kotlin/com/almasb/fxgl/app/services/FXGLAssetLoaderService.kt b/fxgl/src/main/kotlin/com/almasb/fxgl/app/services/FXGLAssetLoaderService.kt
@@ -238,22 +238,6 @@ class FXGLAssetLoaderService : AssetLoaderService() {
         return load(MUSIC, url)
     }
 
-    // TODO: remove in future version
-    /**
-     * Loads cursor image with given name from /assets/ui/cursors/.
-     * Either returns a valid image or throws exception in case of errors.
-     *
-     * @param name image name without the /assets/ui/cursors/, e.g. "attack_cursor.png"
-     * @return cursor image
-     * @throws IllegalArgumentException if asset not found or loading error
-     */
-    @Deprecated("Place cursor under textures/ and use loadImage() instead")
-    fun loadCursorImage(name: String): Image {
-        val url = getURL(CURSORS_DIR + name)
-
-        return load(IMAGE, url)
-    }
-
     /**
      * Loads resource bundle with given [name] from "/assets/properties/".
      *
diff --git a/fxgl/src/test/kotlin/com/almasb/fxgl/app/AssetLoaderServiceTest.kt b/fxgl/src/test/kotlin/com/almasb/fxgl/app/AssetLoaderServiceTest.kt
@@ -274,19 +274,6 @@ class AssetLoaderServiceTest {
         assertThat(map.getString("testKey"), `is`("testValue"))
     }
 
-    @Test
-    fun loadCursorImage() {
-        var cursorImage = assetLoader.loadCursorImage("test_cursor.png")
-
-        assertThat(cursorImage, `is`(notNullValue()))
-        assertThat(cursorImage.width, `is`(64.0))
-        assertThat(cursorImage.height, `is`(64.0))
-
-        cursorImage = assetLoader.loadCursorImage("bla-bla")
-
-        assertThat(cursorImage, `is`(notNullValue()))
-    }
-
     @Test
     fun loadUI() {
         var count = 0
diff --git a/fxgl/src/test/resources/fxglassets/ui/cursors/test_cursor.png b/fxgl/src/test/resources/fxglassets/ui/cursors/test_cursor.png
