Add tests

Author: Dimitar Grigorov

filetoolsserver/handler/copy_file_test.go

@@ -5,6 +5,7 @@ import (
 	"os"
 	"path/filepath"
 	"testing"
+	"time"
 )
 
 func TestHandleCopyFile_Success(t *testing.T) {
@@ -15,6 +16,10 @@ func TestHandleCopyFile_Success(t *testing.T) {
 	dst := filepath.Join(tempDir, "dest.txt")
 	os.WriteFile(src, []byte("content"), 0644)
 
+	// Set a specific mod time on source to verify preservation
+	fixedTime := time.Date(2020, 6, 15, 12, 0, 0, 0, time.UTC)
+	os.Chtimes(src, fixedTime, fixedTime)
+
 	result, _, err := h.HandleCopyFile(context.Background(), nil, CopyFileInput{Source: src, Destination: dst})
 	if err != nil {
 		t.Fatal(err)
@@ -23,19 +28,23 @@ func TestHandleCopyFile_Success(t *testing.T) {
 		t.Error("expected success")
 	}
 
-	// Source should still exist
 	if _, err := os.Stat(src); err != nil {
 		t.Error("source should still exist")
 	}
 
-	// Destination should exist with same content
 	content, err := os.ReadFile(dst)
 	if err != nil {
 		t.Error("destination should exist")
 	}
 	if string(content) != "content" {
 		t.Errorf("wrong content: %s", content)
 	}
+
+	// Verify mod time is preserved
+	dstInfo, _ := os.Stat(dst)
+	if !dstInfo.ModTime().Equal(fixedTime) {
+		t.Errorf("mod time not preserved: got %v, want %v", dstInfo.ModTime(), fixedTime)
+	}
 }
 
 func TestHandleCopyFile_SourceNotExists(t *testing.T) {

filetoolsserver/handler/directory_test.go

@@ -39,6 +39,30 @@ func TestHandleListDirectory_WithPattern(t *testing.T) {
 	}
 }
 
+func TestHandleListDirectory_EmptyDirectory(t *testing.T) {
+	tempDir := t.TempDir()
+	h := NewHandler([]string{tempDir})
+
+	emptyDir := filepath.Join(tempDir, "empty")
+	os.Mkdir(emptyDir, 0755)
+
+	input := ListDirectoryInput{Path: emptyDir}
+
+	result, output, err := h.HandleListDirectory(context.Background(), nil, input)
+	if err != nil {
+		t.Fatal(err)
+	}
+	if result.IsError {
+		t.Error("expected success")
+	}
+	if output.Files == nil {
+		t.Error("expected non-nil empty slice, got nil")
+	}
+	if len(output.Files) != 0 {
+		t.Errorf("expected 0 files, got %d", len(output.Files))
+	}
+}
+
 func TestHandleListDirectory_WithSubdirectory(t *testing.T) {
 	tempDir := t.TempDir()
 	h := NewHandler([]string{tempDir})

filetoolsserver/handler/edit_file_test.go

@@ -195,37 +195,6 @@ func TestHandleEditFile_CRLFPreservation(t *testing.T) {
 	}
 }
 
-func TestHandleEditFile_LFPreservation(t *testing.T) {
-	tempDir := t.TempDir()
-	h := NewHandler([]string{tempDir})
-
-	testFile := filepath.Join(tempDir, "test.txt")
-	// Write file with LF line endings
-	os.WriteFile(testFile, []byte("line1\nline2\nline3"), 0644)
-
-	input := EditFileInput{
-		Path:  testFile,
-		Edits: []EditOperation{{OldText: "line1\nline2", NewText: "new1\nnew2"}},
-	}
-
-	result, _, err := h.HandleEditFile(context.Background(), nil, input)
-	if err != nil {
-		t.Fatal(err)
-	}
-	if result.IsError {
-		t.Errorf("expected success")
-	}
-
-	// Verify LF line endings are preserved (no CRLF introduced)
-	content, _ := os.ReadFile(testFile)
-	if string(content) != "new1\nnew2\nline3" {
-		t.Errorf("LF line endings should be preserved, got %q", content)
-	}
-	if strings.Contains(string(content), "\r\n") {
-		t.Errorf("should not contain CRLF, got %q", content)
-	}
-}
-
 func TestHandleEditFile_ValidationErrors(t *testing.T) {
 	tempDir := t.TempDir()
 	h := NewHandler([]string{tempDir})

filetoolsserver/roots_test.go

@@ -103,9 +103,7 @@ func TestFileURIToPath(t *testing.T) {
 		skip string // GOOS to skip on
 	}{
 		{name: "Windows drive letter", uri: "file:///C:/Users/test", want: "C:/Users/test", skip: "linux"},
-		{name: "Windows forward slashes", uri: "file:///D:/Projects/app", want: "D:/Projects/app", skip: "linux"},
 		{name: "Unix absolute path", uri: "file:///home/user/project", want: "/home/user/project", skip: "windows"},
-		{name: "Unix tmp path", uri: "file:///tmp/test", want: "/tmp/test", skip: "windows"},
 		{name: "not a file URI", uri: "/some/path", want: "/some/path"},
 		{name: "empty string", uri: "", want: ""},
 	}