From a3248cdc6fc418a6979fd5b7feb4d7baf51d0f62 Mon Sep 17 00:00:00 2001
From: Callum Stott <callum@seadowg.com>
Date: Mon, 8 May 2023 11:25:12 +0300
Subject: [PATCH 1/2] Reword test

---
 .../collect/android/formmanagement/FormMediaDownloaderTest.kt   | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/collect_app/src/test/java/org/odk/collect/android/formmanagement/FormMediaDownloaderTest.kt b/collect_app/src/test/java/org/odk/collect/android/formmanagement/FormMediaDownloaderTest.kt
index ea95992beef..390bb5b4612 100644
--- a/collect_app/src/test/java/org/odk/collect/android/formmanagement/FormMediaDownloaderTest.kt
+++ b/collect_app/src/test/java/org/odk/collect/android/formmanagement/FormMediaDownloaderTest.kt
@@ -57,7 +57,7 @@ class FormMediaDownloaderTest {
     }
 
     @Test
-    fun `returns false when there is an existing copy of a media file and an older one and server hash doesn't match existing copy`() {
+    fun `returns false when there is an existing copy of a media file and an older one and media file list hash doesn't match existing copy`() {
         // Save forms
         val formsRepository = InMemFormsRepository()
         val form1 = FormFixtures.form(

From dfdd358b48a17098f15aaee4de6fee943455cf7d Mon Sep 17 00:00:00 2001
From: Callum Stott <callum@seadowg.com>
Date: Mon, 8 May 2023 14:52:22 +0300
Subject: [PATCH 2/2] Move temp files to their own dir and prevent duplicate
 creation

---
 .../java/org/odk/collect/shared/TempFiles.kt  | 45 ++++++++++++++-----
 1 file changed, 33 insertions(+), 12 deletions(-)

diff --git a/shared/src/main/java/org/odk/collect/shared/TempFiles.kt b/shared/src/main/java/org/odk/collect/shared/TempFiles.kt
index 159c88cc04c..494f46b1e18 100644
--- a/shared/src/main/java/org/odk/collect/shared/TempFiles.kt
+++ b/shared/src/main/java/org/odk/collect/shared/TempFiles.kt
@@ -7,14 +7,16 @@ object TempFiles {
 
     @JvmStatic
     fun createTempFile(name: String, extension: String): File {
-        val tempFile = File.createTempFile(name, extension)
-        tempFile.deleteOnExit()
-        return tempFile
+        val tmpDir = getTempDir()
+        return File(tmpDir, name + getRandomName(tmpDir) + extension).also {
+            it.createNewFile()
+            it.deleteOnExit()
+        }
     }
 
     @JvmStatic
     fun createTempFile(parent: File, name: String): File {
-        return File(parent, name).also {
+        return File(parent, name + getRandomName(parent)).also {
             it.createNewFile()
             it.deleteOnExit()
         }
@@ -30,14 +32,15 @@ object TempFiles {
 
     @JvmStatic
     fun createTempFile(parent: File): File {
-        val tempFile = File.createTempFile(getRandomString(), ".temp", parent)
-        tempFile.deleteOnExit()
-        return tempFile
+        return File(parent, getRandomName(parent)).also {
+            it.createNewFile()
+            it.deleteOnExit()
+        }
     }
 
     @JvmStatic
     fun getPathInTempDir(name: String, extension: String): String {
-        val tmpDir = System.getProperty("java.io.tmpdir", ".")
+        val tmpDir = getTempDir()
         val file = File(tmpDir, name + extension)
         file.deleteOnExit()
         return file.absolutePath
@@ -45,15 +48,15 @@ object TempFiles {
 
     @JvmStatic
     fun getPathInTempDir(): String {
-        val tmpDir = System.getProperty("java.io.tmpdir", ".")
-        return File(tmpDir, getRandomString()).absolutePath
+        val tmpDir = getTempDir()
+        return File(tmpDir, getRandomName(tmpDir)).absolutePath
     }
 
     @JvmStatic
     @JvmOverloads
     fun createTempDir(parent: File? = null): File {
         val dir = if (parent != null) {
-            File(parent, getRandomString())
+            File(parent, getRandomName(parent))
         } else {
             File(getPathInTempDir())
         }
@@ -62,5 +65,23 @@ object TempFiles {
         return dir
     }
 
-    private fun getRandomString() = RandomString.randomString(16)
+    private fun getTempDir(): File {
+        val tmpDir = File(System.getProperty("java.io.tmpdir", "."), " org.odk.collect.shared.TempFiles")
+        if (!tmpDir.exists()) {
+            tmpDir.mkdir()
+        }
+
+        return tmpDir
+    }
+
+    private fun getRandomName(parent: File): String {
+        val existing = parent.listFiles()
+
+        var candiate = RandomString.randomString(16)
+        while (existing!!.any { it.name.contains(candiate) }) {
+            candiate = RandomString.randomString(16)
+        }
+
+        return candiate
+    }
 }