From 6cfc0fde089ea08c808d9cc75e56396fef792db2 Mon Sep 17 00:00:00 2001
From: a2geek <robgreene@gmail.com>
Date: Wed, 7 Oct 2026 12:08:42 -0500
Subject: [PATCH] More unit tests for GameDOS. It not sets the filename when a
 new file entry is created.

---
 .../exception/FileExistsException.java        | 18 +++++++
 .../os/gamedos/GamedosDirectory.java          |  9 +++-
 .../org/applecommander/os/GamedosTest.java    | 54 +++++++++++++------
 3 files changed, 64 insertions(+), 17 deletions(-)
 create mode 100644 lib/ac-api/src/main/java/org/applecommander/exception/FileExistsException.java

diff --git a/lib/ac-api/src/main/java/org/applecommander/exception/FileExistsException.java b/lib/ac-api/src/main/java/org/applecommander/exception/FileExistsException.java
new file mode 100644
index 00000000..f56e638b
--- /dev/null
+++ b/lib/ac-api/src/main/java/org/applecommander/exception/FileExistsException.java
@@ -0,0 +1,18 @@
+package org.applecommander.exception;
+
+public class FileExistsException extends RuntimeException {
+    public FileExistsException() {
+    }
+
+    public FileExistsException(String fmt, Object ...args) {
+        super(String.format(fmt, args));
+    }
+
+    public FileExistsException(String message, Throwable cause) {
+        super(message, cause);
+    }
+
+    public FileExistsException(Throwable cause) {
+        super(cause);
+    }
+}
diff --git a/lib/ac-api/src/main/java/org/applecommander/os/gamedos/GamedosDirectory.java b/lib/ac-api/src/main/java/org/applecommander/os/gamedos/GamedosDirectory.java
index d99ae462..8a80b6f6 100644
--- a/lib/ac-api/src/main/java/org/applecommander/os/gamedos/GamedosDirectory.java
+++ b/lib/ac-api/src/main/java/org/applecommander/os/gamedos/GamedosDirectory.java
@@ -22,9 +22,11 @@ package org.applecommander.os.gamedos;
 import org.applecommander.capability.Capability;
 import org.applecommander.device.TrackSectorDevice;
 import org.applecommander.exception.DirectoryFullException;
+import org.applecommander.exception.FileExistsException;
 import org.applecommander.filestore.*;
 import org.applecommander.util.DataBuffer;
 
+import java.nio.file.FileAlreadyExistsException;
 import java.util.ArrayList;
 import java.util.List;
 import java.util.Optional;
@@ -78,8 +80,13 @@ public class GamedosDirectory implements WritableDirectory, GamedosConstants {
         for (int i=0; i<DIRECTORY_SIZE; i++) {
             DataBuffer data = device.readSector(DIRECTORY_TRACK, DIRECTORY_SECTOR+i);
             for (int offset=0; offset<data.limit(); offset+= GamedosFileEntry.ENTRY_SIZE) {
+                GamedosFileEntry fileEntry = new GamedosFileEntry(this, DIRECTORY_SECTOR+i, offset);
                 if (data.getUnsignedByte(offset) == 0) {
-                    return new GamedosFileEntry(this, DIRECTORY_SECTOR+i, offset);
+                    fileEntry.setName(fileName);
+                    return fileEntry;
+                }
+                else if (fileName.equalsIgnoreCase(fileEntry.getName())) {
+                    throw new FileExistsException("file '%s' already exists", fileName);
                 }
             }
         }
diff --git a/lib/ac-api/src/test/java/org/applecommander/os/GamedosTest.java b/lib/ac-api/src/test/java/org/applecommander/os/GamedosTest.java
index 886198e2..8ffd9a1f 100644
--- a/lib/ac-api/src/test/java/org/applecommander/os/GamedosTest.java
+++ b/lib/ac-api/src/test/java/org/applecommander/os/GamedosTest.java
@@ -88,26 +88,48 @@ public class GamedosTest {
         assertNotNull(directory);
         assertEquals(0, directory.getFiles().size());
 
-        // Create our file
-        GamedosFileEntry fileEntry = (GamedosFileEntry) directory.createFile("This is a text file.");
-        assertNotNull(fileEntry);
-        final String textContents = "THIS IS A THE FILE DATA.";
-        DataBuffer fileData = DataBuffer.wrap(textContents.getBytes());
-        fileEntry.setFiletype("T");
-        fileEntry.setDataFork(fileData);
+        // Create and verify the first file.
+        final byte[] textContents = "THIS IS A THE FILE DATA.".getBytes();
+        final String firstFileName = "This is the FIRST file.";
+        GamedosFileEntry firstFile = createFile(directory, firstFileName, "T", textContents);
+        verifyFile(firstFile, firstFileName, "T", 1, 0, 1, textContents);
 
-        // Verify the file went where we expected it to
-        assertEquals("T", fileEntry.getFiletype());
-        assertEquals(1, fileEntry.getFirstTrack());
-        assertEquals(0, fileEntry.getFirstSector());
-        assertEquals(1, fileEntry.getSectorCount());
-        assertEquals(textContents, fileEntry.getDataFork().getFixedLengthString(0, textContents.length()));
+        // Create and verify a second file.
+        final byte[] binaryContents = { 0, 1, 2, 3, 4, 5, 6, 7, 8, 9, 10 };
+        final String secondFileName = "This is the SECOND file.";
+        GamedosFileEntry secondFile = createFile(directory, secondFileName, "B", binaryContents);
+        verifyFile(secondFile, secondFileName, "B", 1, 1, 1, binaryContents);
 
-        // Read back to file entry from the directory and then delete it.
-        assertEquals(1, directory.getFiles().size());
+        // Read back the first file entry from the directory and then delete it.
+        assertEquals(2, directory.getFiles().size());
         GamedosFileEntry firstFileEntry = (GamedosFileEntry) directory.getFiles().getFirst();
         assertNotNull(firstFileEntry);
         directory.deleteFile(firstFileEntry);
-        assertEquals(0, directory.getFiles().size());
+        assertEquals(1, directory.getFiles().size());
+
+        // Finally, create a 3rd file and expect it to be placed where the FIRST file was.
+        final byte[] thirdFile = "THIS IS THE THIRD FILE.".getBytes();
+        final String thirdFileName = "This is the THIRD file.";
+        GamedosFileEntry thirdFileEntry = createFile(directory, thirdFileName, "T", thirdFile);
+        verifyFile(thirdFileEntry, thirdFileName, "T", 1, 0, 1, thirdFile);
+    }
+
+    public GamedosFileEntry createFile(GamedosDirectory directory, String fileName, String fileType, byte[] dataFork) {
+        GamedosFileEntry fileEntry = (GamedosFileEntry) directory.createFile(fileName);
+        assertNotNull(fileEntry);
+        fileEntry.setFiletype(fileType);
+        fileEntry.setDataFork(DataBuffer.wrap(dataFork));
+        return fileEntry;
+    }
+
+    public void verifyFile(GamedosFileEntry fileEntry, String expectedFileName, String expectedFileType,
+                           int expectedFirstTrack, int expectedFirstSector, int expectedSectorCount, byte[] expectedDataFork) {
+        assertEquals(expectedFileName, fileEntry.getName());
+        assertEquals(expectedFileType, fileEntry.getFiletype());
+        assertEquals(expectedFirstTrack, fileEntry.getFirstTrack());
+        assertEquals(expectedFirstSector, fileEntry.getFirstSector());
+        assertEquals(expectedSectorCount, fileEntry.getSectorCount());
+        assertArrayEquals(expectedDataFork,
+                fileEntry.getDataFork().getFixedLengthString(0, expectedDataFork.length).getBytes());
     }
 }