Task 2: Add 5 integration tests (#180)

* docs: add 5 integration tests for R2CN task 2

* refactor: split integration tests into 5 separate files per reviewer request

* style: fix formatting using cargo fmt nightly

* refactor: follow established test patterns using internal API

Author: xwwcyber

Cargo.lock

@@ -81,3 +81,4 @@ testcontainers = { version = "0.25.2", features = [
     "reusable-containers",
 ] }
 reqwest = { version = "0.12.24", features = ["blocking"] }
+assert_cmd = "2.1.2"

Cargo.toml

@@ -617,3 +617,78 @@ async fn test_add_duplicate_file_should_not_duplicate_index() {
         );
     }
 }
+
+#[tokio::test]
+#[serial]
+/// Tests adding an empty file to the repository
+///
+/// Ensures that Libra can handle adding empty files without errors
+async fn test_add_empty_file() {
+    let test_dir = tempdir().unwrap();
+    test::setup_with_new_libra_in(test_dir.path()).await;
+    let _guard = test::ChangeDirGuard::new(test_dir.path());
+
+    // Create an empty file
+    let file_path = "empty.txt";
+    fs::File::create(file_path).unwrap();
+
+    // Execute add command
+    add::execute(AddArgs {
+        pathspec: vec![String::from(file_path)],
+        all: false,
+        update: false,
+        refresh: false,
+        force: false,
+        verbose: false,
+        dry_run: false,
+        ignore_errors: false,
+    })
+    .await;
+
+    // Verify the empty file was added to index
+    let changes = changes_to_be_committed().await;
+    assert!(
+        changes.new.iter().any(|x| x.to_str().unwrap() == file_path),
+        "Empty file should be added to index"
+    );
+}
+
+#[tokio::test]
+#[serial]
+/// Tests adding a file in a nested subdirectory
+///
+/// Ensures that Libra correctly handles files in deep directory structures
+async fn test_add_sub_directory_file() {
+    let test_dir = tempdir().unwrap();
+    test::setup_with_new_libra_in(test_dir.path()).await;
+    let _guard = test::ChangeDirGuard::new(test_dir.path());
+
+    // Create nested subdirectory structure
+    let sub_dir = "a/b/c";
+    fs::create_dir_all(sub_dir).unwrap();
+    let file_path = "a/b/c/deep.txt";
+    fs::write(file_path, "hello deep").unwrap();
+
+    // Execute add command
+    add::execute(AddArgs {
+        pathspec: vec![String::from(file_path)],
+        all: false,
+        update: false,
+        refresh: false,
+        force: false,
+        verbose: false,
+        dry_run: false,
+        ignore_errors: false,
+    })
+    .await;
+
+    // Verify the file in nested directory was added to index
+    let changes = changes_to_be_committed().await;
+    assert!(
+        changes
+            .new
+            .iter()
+            .any(|x| x.to_str().unwrap().replace("\\", "/") == file_path),
+        "File in nested subdirectory should be added to index"
+    );
+}

tests/command/add_test.rs

@@ -1792,3 +1792,78 @@ async fn test_status_porcelain_v2_deleted_file() {
         deleted_line
     );
 }
+
+#[tokio::test]
+#[serial]
+/// Tests status command after adding a file
+///
+/// Verifies that the status command correctly reports added files with proper formatting
+async fn test_status_after_add() {
+    let test_dir = tempdir().unwrap();
+    test::setup_with_new_libra_in(test_dir.path()).await;
+    let _guard = test::ChangeDirGuard::new(test_dir.path());
+
+    // Create a new file
+    let file_path = "test.txt";
+    fs::write(file_path, "content").unwrap();
+
+    // Add the file
+    add::execute(AddArgs {
+        pathspec: vec![String::from(file_path)],
+        all: false,
+        update: false,
+        refresh: false,
+        force: false,
+        verbose: false,
+        dry_run: false,
+        ignore_errors: false,
+    })
+    .await;
+
+    // Test porcelain output
+    let mut output = Vec::new();
+    status_execute(
+        StatusArgs {
+            porcelain: Some(PorcelainVersion::V1),
+            ..Default::default()
+        },
+        &mut output,
+    )
+    .await;
+
+    let output_str = String::from_utf8(output).unwrap();
+    assert!(
+        output_str
+            .lines()
+            .any(|l| l.starts_with("A ") && l.contains(file_path)),
+        "Porcelain status should show 'A ' prefix for added file: {}",
+        output_str
+    );
+
+    // Test short output
+    let mut output = Vec::new();
+    status_execute(
+        StatusArgs {
+            short: true,
+            ..Default::default()
+        },
+        &mut output,
+    )
+    .await;
+
+    let output_str = String::from_utf8(output).unwrap();
+    assert!(
+        output_str
+            .lines()
+            .any(|l| l.starts_with("A ") && l.contains(file_path)),
+        "Short status should show 'A ' prefix for added file: {}",
+        output_str
+    );
+
+    // Verify via changes_to_be_committed
+    let changes = changes_to_be_committed().await;
+    assert!(
+        changes.new.iter().any(|x| x.to_str().unwrap() == file_path),
+        "Added file should appear in changes_to_be_committed"
+    );
+}