fixat mer från coderabbit

Author: Boppler12

src/main/java/org/example/FileCache.java

@@ -5,6 +5,8 @@
 public class FileCache {
     private static final ConcurrentHashMap<String, byte[]> cache = new ConcurrentHashMap<>();
 
+    private FileCache() {}
+
     public static byte[] get(String key) {
         return cache.get(key);
     }
@@ -16,9 +18,5 @@ public static void put(String key, byte[] content) {
     public static void clear() {
         cache.clear();
     }
-    public static void clearCache() {
-        FileCache.clear();
-    }
-
 }
 

src/main/java/org/example/StaticFileHandler.java

@@ -8,6 +8,8 @@
 import java.nio.file.Files;
 import java.nio.file.Path;
 import java.nio.file.Paths;
+import java.nio.file.NoSuchFileException;
+
 
 public class StaticFileHandler {
     private static final String DEFAULT_WEB_ROOT = "www";
@@ -44,11 +46,14 @@ public void sendGetRequest(OutputStream outputStream, String uri) throws IOExcep
             outputStream.write(response.build());
             outputStream.flush();
 
-        } catch (IOException e) {
+        } catch (NoSuchFileException e) {
             writeResponse(outputStream, 404, "Not Found");
+        } catch (IOException e) {
+            writeResponse(outputStream, 500, "Internal Server Error");
         }
     }
 
+
     private String sanitizeUri(String uri) {
         if (uri == null || uri.isEmpty()) return "index.html";
 

src/test/java/org/example/StaticFileHandlerTest.java

@@ -87,7 +87,7 @@ void testConcurrent_MultipleReads() throws InterruptedException, IOException {
 
         handler.sendGetRequest(new ByteArrayOutputStream(), "shared.html");
 
-        // Act - 10 trådar läser samma fil 50 gånger varje
+        // Act - 10 threads reading same file 50 times each
         Thread[] threads = new Thread[10];
         final Exception[] threadError = new Exception[1];
 
@@ -100,7 +100,7 @@ void testConcurrent_MultipleReads() throws InterruptedException, IOException {
                         String response = out.toString();
 
                         if (!response.contains("HTTP/1.1 200") || !response.contains("Data")) {
-                            throw new AssertionError("Oväntad response");
+                            throw new AssertionError("Unexpected response");
                         }
                     }
                 } catch (Exception e) {
@@ -111,7 +111,15 @@ void testConcurrent_MultipleReads() throws InterruptedException, IOException {
             });
             threads[i].start();
         }
+    
+    // Wait for all threads to complete
+    for (Thread thread : threads) {
+        thread.join();
     }
+    
+    // Assert
+    assertNull(threadError[0], "Thread threw exception: " + (threadError[0] != null ? threadError[0].getMessage() : ""));
+}
 
     @Test
     void test_file_that_exists_should_return_200() throws IOException {