test: Add comprehensive tests for log_scan_run_info function

[Copilot]

Adds 42 test cases for log_scan_run_info to verify logging behavior across various input combinations.

Test Coverage

• Commit SHA handling: empty (logs "none"), valid hex formats, edge cases
• Branch variations: Some/None, special characters, unicode
• Parameter combinations and consistency validation
• Scan run ID format (UUID) and uniqueness
• Idempotency and boundary conditions

Implementation

• New test file: crates/mother-cli/src/commands/scan/tests/tests_log_scan_run_info.rs
• Uses create_scan_run() for integration testing
• Constant VALID_FULL_COMMIT_SHA for reusable 40-char hex SHA

Example test pattern:

#[test]
fn test_log_scan_run_info_with_empty_commit_sha() {
    let (scan_run, commit_sha) = create_scan_run(path, None);
    assert_eq!(commit_sha, "");
    log_scan_run_info(&scan_run, &commit_sha); // Should log "none"
}

All tests verify the function doesn't panic with various inputs while maintaining existing test patterns.

Warning

Firewall rules blocked me from connecting to one or more addresses (expand for details)

I tried to connect to the following addresses, but was blocked by firewall rules:

• 192.168.1.100
  • Triggering command: /home/REDACTED/work/mother/mother/target/debug/deps/mother_cli-d349731c4bc5c110 /home/REDACTED/work/mother/mother/target/debug/deps/mother_cli-d349731c4bc5c110 --emit=dep-info,metadata,link -C opt-level=1 -C embed-bitcode=no -C codegen-units=256 -C debuginfo=2 --warn=rust_2018_idioms --warn=clippy::zero_sized_map_values out/�� out/build/43a19b30e9e69d8f-commit_graph.o out/build/43a19b30e9e69d8f-commit_list.o rgo/bin/as e tp.o pointer cc (packet block)
  • Triggering command: /home/REDACTED/work/mother/mother/target/debug/deps/mother_cli-d349731c4bc5c110 /home/REDACTED/work/mother/mother/target/debug/deps/mother_cli-d349731c4bc5c110 --test-threads=4 --emit=dep-info,metadata -C opt-level=1 -C embed-bitcode=no .2.13/src/lib.rs codegen-units=256 -C debuginfo=2 -C targ�� ld/libgit2-sys-655f2dd85dfd3719/-g ld/libgit2-sys-655f2dd85dfd3719/-gdwarf-4 bin/cargo-clippy target/debug/dep/home/REDACTED/.rustup/toolchains/stable-x86_64-REDACTED-linux-gnu/bin/rustc target/debug/dep--crate-name (packet block)
  • Triggering command: /home/REDACTED/work/mother/mother/target/debug/deps/mother_cli-d349731c4bc5c110 /home/REDACTED/work/mother/mother/target/debug/deps/mother_cli-d349731c4bc5c110 --test-threads=4 pointer cc -O1 in.so /lto-wrapper cc it2 (packet block)
• invalid-host
  • Triggering command: /home/REDACTED/work/mother/mother/target/debug/deps/mother_cli-d349731c4bc5c110 /home/REDACTED/work/mother/mother/target/debug/deps/mother_cli-d349731c4bc5c110 --emit=dep-info,metadata,link -C opt-level=1 -C embed-bitcode=no -C codegen-units=256 -C debuginfo=2 --warn=rust_2018_idioms --warn=clippy::zero_sized_map_values out/�� out/build/43a19b30e9e69d8f-commit_graph.o out/build/43a19b30e9e69d8f-commit_list.o rgo/bin/as e tp.o pointer cc (dns block)
  • Triggering command: /home/REDACTED/work/mother/mother/target/debug/deps/mother_cli-d349731c4bc5c110 /home/REDACTED/work/mother/mother/target/debug/deps/mother_cli-d349731c4bc5c110 --test-threads=4 --emit=dep-info,metadata -C opt-level=1 -C embed-bitcode=no .2.13/src/lib.rs codegen-units=256 -C debuginfo=2 -C targ�� ld/libgit2-sys-655f2dd85dfd3719/-g ld/libgit2-sys-655f2dd85dfd3719/-gdwarf-4 bin/cargo-clippy target/debug/dep/home/REDACTED/.rustup/toolchains/stable-x86_64-REDACTED-linux-gnu/bin/rustc target/debug/dep--crate-name (dns block)
  • Triggering command: /home/REDACTED/work/mother/mother/target/debug/deps/mother_cli-d349731c4bc5c110 /home/REDACTED/work/mother/mother/target/debug/deps/mother_cli-d349731c4bc5c110 --test-threads=4 pointer cc -O1 in.so /lto-wrapper cc it2 (dns block)
• invalid-uri
  • Triggering command: /home/REDACTED/work/mother/mother/target/debug/deps/mother_cli-d349731c4bc5c110 /home/REDACTED/work/mother/mother/target/debug/deps/mother_cli-d349731c4bc5c110 --emit=dep-info,metadata,link -C opt-level=1 -C embed-bitcode=no -C codegen-units=256 -C debuginfo=2 --warn=rust_2018_idioms --warn=clippy::zero_sized_map_values out/�� out/build/43a19b30e9e69d8f-commit_graph.o out/build/43a19b30e9e69d8f-commit_list.o rgo/bin/as e tp.o pointer cc (dns block)
  • Triggering command: /home/REDACTED/work/mother/mother/target/debug/deps/mother_cli-d349731c4bc5c110 /home/REDACTED/work/mother/mother/target/debug/deps/mother_cli-d349731c4bc5c110 --test-threads=4 --emit=dep-info,metadata -C opt-level=1 -C embed-bitcode=no .2.13/src/lib.rs codegen-units=256 -C debuginfo=2 -C targ�� ld/libgit2-sys-655f2dd85dfd3719/-g ld/libgit2-sys-655f2dd85dfd3719/-gdwarf-4 bin/cargo-clippy target/debug/dep/home/REDACTED/.rustup/toolchains/stable-x86_64-REDACTED-linux-gnu/bin/rustc target/debug/dep--crate-name (dns block)
  • Triggering command: /home/REDACTED/work/mother/mother/target/debug/deps/mother_cli-d349731c4bc5c110 /home/REDACTED/work/mother/mother/target/debug/deps/mother_cli-d349731c4bc5c110 --test-threads=4 pointer cc -O1 in.so /lto-wrapper cc it2 (dns block)
• neo4j.example.com
  • Triggering command: /home/REDACTED/work/mother/mother/target/debug/deps/mother_cli-d349731c4bc5c110 /home/REDACTED/work/mother/mother/target/debug/deps/mother_cli-d349731c4bc5c110 --emit=dep-info,metadata,link -C opt-level=1 -C embed-bitcode=no -C codegen-units=256 -C debuginfo=2 --warn=rust_2018_idioms --warn=clippy::zero_sized_map_values out/�� out/build/43a19b30e9e69d8f-commit_graph.o out/build/43a19b30e9e69d8f-commit_list.o rgo/bin/as e tp.o pointer cc (dns block)
  • Triggering command: /home/REDACTED/work/mother/mother/target/debug/deps/mother_cli-d349731c4bc5c110 /home/REDACTED/work/mother/mother/target/debug/deps/mother_cli-d349731c4bc5c110 --test-threads=4 --emit=dep-info,metadata -C opt-level=1 -C embed-bitcode=no .2.13/src/lib.rs codegen-units=256 -C debuginfo=2 -C targ�� ld/libgit2-sys-655f2dd85dfd3719/-g ld/libgit2-sys-655f2dd85dfd3719/-gdwarf-4 bin/cargo-clippy target/debug/dep/home/REDACTED/.rustup/toolchains/stable-x86_64-REDACTED-linux-gnu/bin/rustc target/debug/dep--crate-name (dns block)
  • Triggering command: /home/REDACTED/work/mother/mother/target/debug/deps/mother_cli-d349731c4bc5c110 /home/REDACTED/work/mother/mother/target/debug/deps/mother_cli-d349731c4bc5c110 --test-threads=4 pointer cc -O1 in.so /lto-wrapper cc it2 (dns block)
• neo4j.production.example.com
  • Triggering command: /home/REDACTED/work/mother/mother/target/debug/deps/mother_cli-d349731c4bc5c110 /home/REDACTED/work/mother/mother/target/debug/deps/mother_cli-d349731c4bc5c110 --emit=dep-info,metadata,link -C opt-level=1 -C embed-bitcode=no -C codegen-units=256 -C debuginfo=2 --warn=rust_2018_idioms --warn=clippy::zero_sized_map_values out/�� out/build/43a19b30e9e69d8f-commit_graph.o out/build/43a19b30e9e69d8f-commit_list.o rgo/bin/as e tp.o pointer cc (dns block)
  • Triggering command: /home/REDACTED/work/mother/mother/target/debug/deps/mother_cli-d349731c4bc5c110 /home/REDACTED/work/mother/mother/target/debug/deps/mother_cli-d349731c4bc5c110 --test-threads=4 --emit=dep-info,metadata -C opt-level=1 -C embed-bitcode=no .2.13/src/lib.rs codegen-units=256 -C debuginfo=2 -C targ�� ld/libgit2-sys-655f2dd85dfd3719/-g ld/libgit2-sys-655f2dd85dfd3719/-gdwarf-4 bin/cargo-clippy target/debug/dep/home/REDACTED/.rustup/toolchains/stable-x86_64-REDACTED-linux-gnu/bin/rustc target/debug/dep--crate-name (dns block)
  • Triggering command: /home/REDACTED/work/mother/mother/target/debug/deps/mother_cli-d349731c4bc5c110 /home/REDACTED/work/mother/mother/target/debug/deps/mother_cli-d349731c4bc5c110 --test-threads=4 pointer cc -O1 in.so /lto-wrapper cc it2 (dns block)
• nonexistent.host.that.does.not.exist
  • Triggering command: /home/REDACTED/work/mother/mother/target/debug/deps/mother_cli-d349731c4bc5c110 /home/REDACTED/work/mother/mother/target/debug/deps/mother_cli-d349731c4bc5c110 --emit=dep-info,metadata,link -C opt-level=1 -C embed-bitcode=no -C codegen-units=256 -C debuginfo=2 --warn=rust_2018_idioms --warn=clippy::zero_sized_map_values out/�� out/build/43a19b30e9e69d8f-commit_graph.o out/build/43a19b30e9e69d8f-commit_list.o rgo/bin/as e tp.o pointer cc (dns block)
  • Triggering command: /home/REDACTED/work/mother/mother/target/debug/deps/mother_cli-d349731c4bc5c110 /home/REDACTED/work/mother/mother/target/debug/deps/mother_cli-d349731c4bc5c110 --test-threads=4 --emit=dep-info,metadata -C opt-level=1 -C embed-bitcode=no .2.13/src/lib.rs codegen-units=256 -C debuginfo=2 -C targ�� ld/libgit2-sys-655f2dd85dfd3719/-g ld/libgit2-sys-655f2dd85dfd3719/-gdwarf-4 bin/cargo-clippy target/debug/dep/home/REDACTED/.rustup/toolchains/stable-x86_64-REDACTED-linux-gnu/bin/rustc target/debug/dep--crate-name (dns block)
  • Triggering command: /home/REDACTED/work/mother/mother/target/debug/deps/mother_cli-d349731c4bc5c110 /home/REDACTED/work/mother/mother/target/debug/deps/mother_cli-d349731c4bc5c110 --test-threads=4 pointer cc -O1 in.so /lto-wrapper cc it2 (dns block)
• not-a-valid-uri
  • Triggering command: /home/REDACTED/work/mother/mother/target/debug/deps/mother_cli-d349731c4bc5c110 /home/REDACTED/work/mother/mother/target/debug/deps/mother_cli-d349731c4bc5c110 --emit=dep-info,metadata,link -C opt-level=1 -C embed-bitcode=no -C codegen-units=256 -C debuginfo=2 --warn=rust_2018_idioms --warn=clippy::zero_sized_map_values out/�� out/build/43a19b30e9e69d8f-commit_graph.o out/build/43a19b30e9e69d8f-commit_list.o rgo/bin/as e tp.o pointer cc (dns block)
  • Triggering command: /home/REDACTED/work/mother/mother/target/debug/deps/mother_cli-d349731c4bc5c110 /home/REDACTED/work/mother/mother/target/debug/deps/mother_cli-d349731c4bc5c110 --test-threads=4 --emit=dep-info,metadata -C opt-level=1 -C embed-bitcode=no .2.13/src/lib.rs codegen-units=256 -C debuginfo=2 -C targ�� ld/libgit2-sys-655f2dd85dfd3719/-g ld/libgit2-sys-655f2dd85dfd3719/-gdwarf-4 bin/cargo-clippy target/debug/dep/home/REDACTED/.rustup/toolchains/stable-x86_64-REDACTED-linux-gnu/bin/rustc target/debug/dep--crate-name (dns block)
  • Triggering command: /home/REDACTED/work/mother/mother/target/debug/deps/mother_cli-d349731c4bc5c110 /home/REDACTED/work/mother/mother/target/debug/deps/mother_cli-d349731c4bc5c110 --test-threads=4 pointer cc -O1 in.so /lto-wrapper cc it2 (dns block)

If you need me to access, download, or install something from one of these locations, you can either:

• Configure Actions setup steps to set up my environment, which run before the firewall is enabled
• Add the appropriate URLs or hosts to the custom allowlist in this repository's Copilot coding agent settings (admins only)

Original prompt

Add comprehensive tests for the scan-log-scan-run-info module.

Issues to Resolve

• test: Add tests for function mother::commands::scan::log_scan_run_info #35: test: Add tests for function mother::commands::scan::log_scan_run_info

CRITICAL REQUIREMENTS

1. TEST THROUGH PUBLIC API ONLY

• Do NOT make private methods public just for testing
• Test internal logic through public entrypoint functions
• If a function is private, test it via the public function that calls it

2. VERIFY LOGIC CORRECTNESS

• READ and UNDERSTAND the implementation before writing tests
• Check if the logic makes sense and is correct
• If you find bugs, note them but still test current behavior

3. TEST FILE LOCATION AND NAMING

• Unit tests: scan_log_scan_run_info/tests/tests_*.rs (tests folder inside module)
• Integration tests: tests/scan_log_scan_run_info/tests_*.rs (module folder inside root tests/)
• Use prefix tests_ for all test files (e.g., tests_selection_range.rs)
• All tests for this batch go in the appropriate folder
• Example unit test: server/selection_range.rs → server/tests/tests_selection_range.rs
• Example integration test: server/selection_range.rs → tests/server/tests_selection_range.rs

4. QUALITY TESTS ONLY

• NO TODO comments or placeholder tests
• Test edge cases: empty inputs, error conditions, boundaries
• Descriptive test names explaining what's tested

5. CLOSE ALL ISSUES IN ONE COMMIT

Commit message: test: Add comprehensive tests for scan-log-scan-run-info (closes #35)

6. VERIFY BEFORE PR

• make run-guidelines must pass
• Then create PR

Create a single PR resolving all 1 issues.

---

✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.