Update YamlIterator to implement Iterator, not MutableIterator

krzema12 · May 11, 2024 · b8c9da7 · b8c9da7
1 parent a5c7e22
commit b8c9da7
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 4 deletions.
diff --git a/src/jvmMain/java/it/krzeminski/snakeyaml/engine/kmp/api/Load.kt b/src/jvmMain/java/it/krzeminski/snakeyaml/engine/kmp/api/Load.kt
@@ -156,7 +156,7 @@ class Load @JvmOverloads constructor(
     private class YamlIterator(
         private val composer: Composer,
         private val constructor: BaseConstructor,
-    ) : MutableIterator<Any?> {
+    ) : Iterator<Any?> {
         private var composerInitiated = false
         override fun hasNext(): Boolean {
             composerInitiated = true
@@ -170,7 +170,5 @@ class Load @JvmOverloads constructor(
             val node = composer.next()
             return constructor.constructSingleDocument(node)
         }
-
-        override fun remove(): Unit = throw UnsupportedOperationException("Removing is not supported.")
     }
 }
diff --git a/src/jvmTest/java/org/snakeyaml/engine/v2/api/LoadTest.java b/src/jvmTest/java/org/snakeyaml/engine/v2/api/LoadTest.java
@@ -197,6 +197,6 @@ void loadAllFromStringWithUnsupportedOperationException() {
     Iterable<Object> v = load.loadAllFromString("bbb");
     UnsupportedOperationException exception =
         assertThrows(UnsupportedOperationException.class, () -> v.iterator().remove());
-    assertEquals("Removing is not supported.", exception.getMessage());
+    assertEquals("Operation is not supported for read-only collection", exception.getMessage());
   }
 }