refactor: Change pub to pub(crate) for file_manager module items

Author: reddevilmidzy

src/file_manager/mod.rs

@@ -1,5 +1,5 @@
-pub mod handlers;
-pub mod models;
+mod handlers;
+mod models;
 
-pub use handlers::*;
+pub(crate) use handlers::*;
 pub use models::*;

tests/file_manager/handlers.rs

@@ -4,12 +4,12 @@ use tokio::fs;
 struct TestSetup {
     problem_id: u32,
     file_content: Vec<u8>,
-    file_name: String,
+    filename: String,
     multipart_body: String,
     port: u16,
 }
 
-async fn setup_test(problem_id: u32, file_content: &[u8], file_name: &str) -> TestSetup {
+async fn setup_test(problem_id: u32, file_content: &[u8], filename: &str) -> TestSetup {
     // 테스트용 디렉토리만 정리 (uploads 전체가 아닌 특정 problem_id만)
     let test_upload_dir = format!("uploads/{}", problem_id);
     if Path::new(&test_upload_dir).exists() {
@@ -18,7 +18,7 @@ async fn setup_test(problem_id: u32, file_content: &[u8], file_name: &str) -> Te
 
     let multipart_body = format!(
         "--boundary\r\n\
-         Content-Disposition: form-data; name=\"file\"; filename=\"{file_name}\"\r\n\
+         Content-Disposition: form-data; name=\"file\"; filename=\"{filename}\"\r\n\
          Content-Type: text/plain\r\n\r\n\
          {}\r\n\
          --boundary--\r\n",
@@ -38,7 +38,7 @@ async fn setup_test(problem_id: u32, file_content: &[u8], file_name: &str) -> Te
     TestSetup {
         problem_id,
         file_content: file_content.to_vec(),
-        file_name: file_name.to_string(),
+        filename: filename.to_string(),
         multipart_body,
         port,
     }
@@ -81,14 +81,14 @@ async fn upload_file_success() {
         .json::<coduck_backend::file_manager::FileMetadata>()
         .await
         .unwrap();
-    assert_eq!(response_json.filename, setup.file_name);
+    assert_eq!(response_json.filename, setup.filename);
     assert_eq!(
         response_json.language,
         coduck_backend::file_manager::Language::Python
     );
     assert_eq!(response_json.category, "solution");
 
-    let expected_file_path = format!("uploads/{}/solution/{}", setup.problem_id, setup.file_name);
+    let expected_file_path = format!("uploads/{}/solution/{}", setup.problem_id, setup.filename);
     assert!(Path::new(&expected_file_path).exists());
 
     let saved_content = fs::read(&expected_file_path).await.unwrap();
@@ -119,7 +119,7 @@ async fn upload_file_handles_duplicate_filename() {
         .contains("already exists"));
 
     // 파일이 실제로 하나만 존재하는지 확인
-    let expected_file_path = format!("uploads/{}/solution/{}", setup.problem_id, setup.file_name);
+    let expected_file_path = format!("uploads/{}/solution/{}", setup.problem_id, setup.filename);
     assert!(Path::new(&expected_file_path).exists());
 
     cleanup_test(setup.problem_id).await;
@@ -140,7 +140,7 @@ async fn get_file_success() {
             "http://127.0.0.1:{port}/problems/{problem_id}/solution/{filename}",
             port = setup.port,
             problem_id = setup.problem_id,
-            filename = setup.file_name
+            filename = setup.filename
         ))
         .send()
         .await
@@ -189,7 +189,7 @@ async fn get_files_by_category_success() {
     let files: Vec<String> = response.json().await.unwrap();
 
     assert_eq!(files.len(), 1);
-    assert_eq!(files[0], setup.file_name);
+    assert_eq!(files[0], setup.filename);
 
     cleanup_test(setup.problem_id).await;
 }
@@ -203,7 +203,7 @@ async fn delete_file_success() {
     assert_eq!(upload_response.status(), reqwest::StatusCode::CREATED);
 
     // 파일이 존재하는지 확인
-    let expected_file_path = format!("uploads/{}/solution/{}", setup.problem_id, setup.file_name);
+    let expected_file_path = format!("uploads/{}/solution/{}", setup.problem_id, setup.filename);
     assert!(Path::new(&expected_file_path).exists());
 
     // 파일 삭제
@@ -213,7 +213,7 @@ async fn delete_file_success() {
             "http://127.0.0.1:{port}/problems/{problem_id}/solution/{filename}",
             port = setup.port,
             problem_id = setup.problem_id,
-            filename = setup.file_name
+            filename = setup.filename
         ))
         .send()
         .await
@@ -282,7 +282,7 @@ async fn update_file_content_success() {
             "http://127.0.0.1:{port}/problems/{problem_id}/solution/{filename}",
             port = setup.port,
             problem_id = setup.problem_id,
-            filename = setup.file_name
+            filename = setup.filename
         ))
         .json(&update_data)
         .send()
@@ -299,7 +299,7 @@ async fn update_file_content_success() {
         .contains("updated successfully"));
 
     // 파일 내용이 실제로 업데이트되었는지 확인
-    let expected_file_path = format!("uploads/{}/solution/{}", setup.problem_id, setup.file_name);
+    let expected_file_path = format!("uploads/{}/solution/{}", setup.problem_id, setup.filename);
     let updated_content = fs::read_to_string(&expected_file_path).await.unwrap();
     assert_eq!(updated_content, "print(int(input()) + int(input()))");
 
@@ -364,7 +364,7 @@ async fn update_file_missing_content() {
             "http://127.0.0.1:{port}/problems/{problem_id}/solution/{filename}",
             port = setup.port,
             problem_id = setup.problem_id,
-            filename = setup.file_name
+            filename = setup.filename
         ))
         .json(&update_data)
         .send()
@@ -387,7 +387,7 @@ async fn update_file_filename_success() {
     let client = reqwest::Client::new();
     let new_filename = "aplusb-AC.py";
     let update_data = serde_json::json!({
-        "old_filename": setup.file_name,
+        "old_filename": setup.filename,
         "new_filename": new_filename
     });
 

tests/file_manager/mod.rs

@@ -1 +1 @@
-pub mod handlers;
+mod handlers;

tests/mod.rs

@@ -1 +1 @@
-pub mod file_manager;
+mod file_manager;