can add item to item only if not already there

vextorspace · Jul 12, 2024 · 1fdc06e · 1fdc06e
1 parent 0349c2c
commit 1fdc06e
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 0 deletions.
diff --git a/composeApp/src/commonMain/kotlin/model/Item.kt b/composeApp/src/commonMain/kotlin/model/Item.kt
@@ -10,6 +10,8 @@ data class Item(var content: String = "New Item", val subItems: MutableList<Item
     constructor(content: String, subItems: Collection<Item>) : this(content, subItems.toMutableList())
 
     fun add(subItem: Item) {
+        if(subItems.contains(subItem))
+            return
         subItems.add(subItem)
     }
 

diff --git a/composeApp/src/commonTest/kotlin/model/SubItemOrderMattersTest.kt b/composeApp/src/commonTest/kotlin/model/SubItemOrderMattersTest.kt
@@ -1,5 +1,6 @@
 package model
 
+import io.kotest.matchers.collections.shouldContainExactly
 import io.kotest.matchers.collections.shouldContainInOrder
 import kotlin.test.Test
 
@@ -27,4 +28,13 @@ class SubItemOrderMattersTest {
         item.remove(subItem1)
         item.subItems.shouldContainInOrder(subItem3, subItem2)
     }
+
+    @Test
+    fun `same item cannot be added twice to an item`() {
+        val item = Item("item")
+        val subItem = Item("subItem")
+        item.add(subItem)
+        item.add(subItem)
+        item.subItems.shouldContainExactly(subItem)
+    }
 }