feat: adds smartlog tests

Author: Abhijay

git-branchless-lib/src/core/node_descriptors.rs

@@ -592,6 +592,7 @@ impl NodeDescriptor for SignatureStatusDescriptor {
 #[cfg(test)]
 mod tests {
     use std::ops::{Add, Sub};
+    use std::str::FromStr;
     use std::time::Duration;
 
     use super::*;
@@ -650,4 +651,33 @@ Differential Revision: phabricator.com/D123";
 
         Ok(())
     }
+
+    #[test]
+    fn test_signature_status_descriptor() -> eyre::Result<()> {
+        // Create a mock where is_enabled is false
+        let mut descriptor = SignatureStatusDescriptor {
+            is_enabled: false,
+            repo_path: String::from("/tmp"),
+        };
+
+        // Create a mock object - doesn't matter what it contains since is_enabled is false
+        let object = NodeObject::GarbageCollected {
+            oid: NonZeroOid::from_str("1234567890123456789012345678901234567890").unwrap(),
+        };
+
+        // With is_enabled = false, it should return None
+        let glyphs = Glyphs::text();
+        let result = descriptor.describe_node(&glyphs, &object)?;
+        assert!(result.is_none());
+
+        // Now test with is_enabled = true
+        descriptor.is_enabled = true;
+
+        // We can't easily test the actual git command execution in a unit test,
+        // but we can verify that the method doesn't panic and returns a result
+        let result = descriptor.describe_node(&glyphs, &object);
+        assert!(result.is_ok());
+
+        Ok(())
+    }
 }

git-branchless-smartlog/tests/test_smartlog.rs

@@ -812,3 +812,95 @@ fn test_exact() -> eyre::Result<()> {
 
     Ok(())
 }
+
+#[test]
+fn test_show_signature_flag() -> eyre::Result<()> {
+    let git = make_git()?;
+
+    git.init_repo()?;
+
+    // Just create a regular commit without trying to sign it
+    git.commit_file("signed_file", 1)?;
+    git.commit_file("unsigned_file", 2)?;
+
+    // Run smartlog without --show-signature flag
+    {
+        let stdout = git.smartlog()?;
+        // Without the flag, the signature indicators shouldn't be visible
+        // However, we can't reliably check for absence of brackets since they might be used elsewhere
+        // So we'll just ensure the command runs successfully
+        assert!(!stdout.is_empty(), "Smartlog output should not be empty");
+    }
+
+    // Run smartlog with --show-signature flag
+    {
+        let (stdout, _) = git.branchless("smartlog", &["--show-signature"])?;
+        // With the flag, the command should run successfully
+        assert!(!stdout.is_empty(), "Smartlog output should not be empty");
+        // The signature status indicators should be present in some form,
+        // but we don't validate exactly which status since that depends on the environment
+    }
+
+    Ok(())
+}
+
+#[test]
+fn test_show_signature_snapshot() -> eyre::Result<()> {
+    let git = make_git()?;
+
+    git.init_repo()?;
+
+    // Create regular commits without trying to sign them
+    git.commit_file("signed_file", 1)?;
+
+    // Create a branch and make another commit
+    git.run(&["checkout", "-b", "branch1", "master"])?;
+    git.commit_file("another_file", 2)?;
+
+    // Run smartlog with --show-signature flag
+    let (stdout, _) = git.branchless("smartlog", &["--show-signature"])?;
+
+    // Simply verify the command runs without error
+    assert!(!stdout.is_empty(), "Smartlog output should not be empty");
+
+    Ok(())
+}
+
+#[test]
+fn test_show_signature_argument_parsing() -> eyre::Result<()> {
+    let git = make_git()?;
+    git.init_repo()?;
+
+    // Test that invalid flag combination reports error
+    // For example, combine with a flag that doesn't make sense (if any)
+    // If there are no incompatible flags, we can at least test that the flag is recognized
+
+    // Test help output includes the flag
+    let (help_stdout, _) = git.branchless("smartlog", &["--help"])?;
+    assert!(
+        help_stdout.contains("--show-signature"),
+        "Help output should include --show-signature option"
+    );
+
+    // Test that the flag is recognized (should execute without error)
+    let result = git.branchless("smartlog", &["--show-signature"]);
+    assert!(
+        result.is_ok(),
+        "The --show-signature flag should be recognized"
+    );
+
+    // Test that the flag works when combined with other flags
+    let result = git.branchless("smartlog", &["--show-signature", "--exact"]);
+    assert!(
+        result.is_ok(),
+        "The --show-signature flag should work with --exact"
+    );
+
+    let result = git.branchless("smartlog", &["--show-signature", "--reverse"]);
+    assert!(
+        result.is_ok(),
+        "The --show-signature flag should work with --reverse"
+    );
+
+    Ok(())
+}