From ea9621389bfa62cb4e63688249c52ac0e41ff282 Mon Sep 17 00:00:00 2001
From: Joel Klinghed <the_jk@spawned.biz>
Date: Mon, 14 Oct 2024 21:41:06 +0200
Subject: Add tests for create dir/file/link that already exists

Fix implementations to work as expected
(that createDirectory/File/Link fails if an entry with that name
already exists).
---
 .../java/org/the_jk/cleversync/TreeAbstractTest.kt | 59 ++++++++++++++++++++++
 1 file changed, 59 insertions(+)

(limited to 'libs/test-utils/src')

diff --git a/libs/test-utils/src/main/java/org/the_jk/cleversync/TreeAbstractTest.kt b/libs/test-utils/src/main/java/org/the_jk/cleversync/TreeAbstractTest.kt
index 2555195..84f4ed6 100644
--- a/libs/test-utils/src/main/java/org/the_jk/cleversync/TreeAbstractTest.kt
+++ b/libs/test-utils/src/main/java/org/the_jk/cleversync/TreeAbstractTest.kt
@@ -8,6 +8,7 @@ import org.the_jk.cleversync.io.Directory
 import org.the_jk.cleversync.io.Link
 import org.the_jk.cleversync.io.ModifiableLink
 import org.the_jk.cleversync.io.ModifiableTree
+import java.io.IOException
 
 abstract class TreeAbstractTest {
     protected lateinit var tree: ModifiableTree
@@ -45,6 +46,26 @@ abstract class TreeAbstractTest {
         assertThat(content.links).isEmpty()
     }
 
+    @Test
+    open fun createDirectoryAlreadyExists() {
+        tree.createDirectory("foo")
+        tree.createFile("bar").write().use { it.write(0) }
+        if (supportSymlinks()) {
+            tree.createLink("fum", "does-not-exist")
+        }
+        Assert.assertThrows(IOException::class.java) {
+            tree.createDirectory("foo")
+        }
+        Assert.assertThrows(IOException::class.java) {
+            tree.createDirectory("bar")
+        }
+        if (supportSymlinks()) {
+            Assert.assertThrows(IOException::class.java) {
+                tree.createDirectory("fum")
+            }
+        }
+    }
+
     @Test(timeout = 10000)
     open fun observeCreateDirectory() {
         val content = tree.liveList()
@@ -79,6 +100,26 @@ abstract class TreeAbstractTest {
         }
     }
 
+    @Test
+    open fun createFileAlreadyExists() {
+        tree.createDirectory("foo")
+        tree.createFile("bar").write().use { it.write(0) }
+        if (supportSymlinks()) {
+            tree.createLink("fum", "does-not-exist")
+        }
+        Assert.assertThrows(IOException::class.java) {
+            tree.createFile("foo")
+        }
+        Assert.assertThrows(IOException::class.java) {
+            tree.createFile("bar")
+        }
+        if (supportSymlinks()) {
+            Assert.assertThrows(IOException::class.java) {
+                tree.createFile("fum")
+            }
+        }
+    }
+
     @Test
     open fun overwriteFile() {
         val foo = tree.createFile("foo")
@@ -158,6 +199,24 @@ abstract class TreeAbstractTest {
         assertThat(tree.openFile("link")?.name).isAnyOf("file", "link")
     }
 
+    @Test
+    open fun createLinkAlreadyExists() {
+        Assume.assumeTrue(supportSymlinks())
+
+        tree.createDirectory("foo")
+        tree.createFile("bar").write().use { it.write(0) }
+        tree.createLink("fum", "does-not-exist")
+        Assert.assertThrows(IOException::class.java) {
+            tree.createLink("foo", "test")
+        }
+        Assert.assertThrows(IOException::class.java) {
+            tree.createLink("bar", "test")
+        }
+        Assert.assertThrows(IOException::class.java) {
+            tree.createLink("fum", "test")
+        }
+    }
+
     @Test
     open fun createLinkSubdir() {
         Assume.assumeTrue(supportSymlinks())
-- 
cgit v1.2.3-70-g09d2