fix: Update API routes to use plural 'solutions' instead of singular 'solution'

Author: reddevilmidzy

tests/file_manager/handlers.rs

@@ -36,7 +36,7 @@ async fn setup_test() -> TestSetup {
 async fn upload_file_request(setup: &TestSetup, multipart_body: String) -> reqwest::Response {
     reqwest::Client::new()
         .post(&format!(
-            "http://127.0.0.1:{port}/problems/{problem_id}/solution",
+            "http://127.0.0.1:{port}/problems/{problem_id}/solutions",
             port = setup.port,
             problem_id = setup.problem_id
         ))
@@ -90,10 +90,10 @@ async fn upload_file_success() {
         response_json.language,
         coduck_backend::file_manager::Language::Python
     );
-    assert_eq!(response_json.category, "solution");
+    assert_eq!(response_json.category, "solutions");
 
     let expected_file_path = format!(
-        "{}/{}/solution/{}",
+        "{}/{}/solutions/{}",
         setup.temp_dir.path().to_str().unwrap(),
         setup.problem_id,
         filename
@@ -126,7 +126,7 @@ async fn upload_file_handles_duplicate_filename() {
         .contains("already exists"));
 
     let expected_file_path = format!(
-        "{}/{}/solution/{}",
+        "{}/{}/solutions/{}",
         setup.temp_dir.path().to_str().unwrap(),
         setup.problem_id,
         filename
@@ -149,7 +149,7 @@ async fn get_file_success() {
     let client = reqwest::Client::new();
     let response = client
         .get(&format!(
-            "http://127.0.0.1:{port}/problems/{problem_id}/solution/{filename}",
+            "http://127.0.0.1:{port}/problems/{problem_id}/solutions/{filename}",
             port = setup.port,
             problem_id = setup.problem_id,
             filename = filename
@@ -187,10 +187,9 @@ async fn get_files_by_category_success() {
     let client = reqwest::Client::new();
     let response = client
         .get(&format!(
-            "http://127.0.0.1:{port}/problems/{problem_id}/{category}",
+            "http://127.0.0.1:{port}/problems/{problem_id}/solutions",
             port = setup.port,
             problem_id = setup.problem_id,
-            category = "solution"
         ))
         .send()
         .await
@@ -216,7 +215,7 @@ async fn delete_file_success() {
 
     // Check if file exists
     let expected_file_path = format!(
-        "{}/{}/solution/{}",
+        "{}/{}/solutions/{}",
         setup.temp_dir.path().to_str().unwrap(),
         setup.problem_id,
         filename
@@ -227,7 +226,7 @@ async fn delete_file_success() {
     let client = reqwest::Client::new();
     let response = client
         .delete(&format!(
-            "http://127.0.0.1:{port}/problems/{problem_id}/solution/{filename}",
+            "http://127.0.0.1:{port}/problems/{problem_id}/solutions/{filename}",
             port = setup.port,
             problem_id = setup.problem_id,
             filename = filename
@@ -257,7 +256,7 @@ async fn delete_file_not_found() {
     let client = reqwest::Client::new();
     let response = client
         .delete(&format!(
-            "http://127.0.0.1:{port}/problems/{problem_id}/solution/{filename}",
+            "http://127.0.0.1:{port}/problems/{problem_id}/solutions/{filename}",
             port = setup.port,
             problem_id = setup.problem_id,
             filename = "non-existent-file.py"
@@ -295,7 +294,7 @@ async fn update_file_content_success() {
 
     let response = client
         .put(&format!(
-            "http://127.0.0.1:{port}/problems/{problem_id}/solution/{filename}",
+            "http://127.0.0.1:{port}/problems/{problem_id}/solutions/{filename}",
             port = setup.port,
             problem_id = setup.problem_id,
             filename = filename
@@ -316,7 +315,7 @@ async fn update_file_content_success() {
 
     // Verify that file content was actually updated
     let expected_file_path = format!(
-        "{}/{}/solution/{}",
+        "{}/{}/solutions/{}",
         setup.temp_dir.path().to_str().unwrap(),
         setup.problem_id,
         filename
@@ -338,7 +337,7 @@ async fn update_file_not_found() {
 
     let response = client
         .put(&format!(
-            "http://127.0.0.1:{port}/problems/{problem_id}/solution/{filename}",
+            "http://127.0.0.1:{port}/problems/{problem_id}/solutions/{filename}",
             port = setup.port,
             problem_id = setup.problem_id,
             filename = "non-existent-file.py"
@@ -377,7 +376,7 @@ async fn update_file_missing_content() {
 
     let response = client
         .put(&format!(
-            "http://127.0.0.1:{port}/problems/{problem_id}/solution/{filename}",
+            "http://127.0.0.1:{port}/problems/{problem_id}/solutions/{filename}",
             port = setup.port,
             problem_id = setup.problem_id,
             filename = filename
@@ -412,7 +411,7 @@ async fn update_filename_success() {
 
     let response = client
         .put(&format!(
-            "http://127.0.0.1:{port}/problems/{problem_id}/solution",
+            "http://127.0.0.1:{port}/problems/{problem_id}/solutions",
             port = setup.port,
             problem_id = setup.problem_id
         ))
@@ -432,7 +431,7 @@ async fn update_filename_success() {
 
     // Verify that filename was actually updated
     let expected_file_path = format!(
-        "{}/{}/solution/{}",
+        "{}/{}/solutions/{}",
         setup.temp_dir.path().to_str().unwrap(),
         setup.problem_id,
         new_filename