From 980d7816bb33e5a46e6b78e548ae0df883317732 Mon Sep 17 00:00:00 2001 From: "google-labs-jules[bot]" <161369871+google-labs-jules[bot]@users.noreply.github.com> Date: Mon, 20 Apr 2026 00:07:40 +0000 Subject: [PATCH] test: add unit test for Config::get_submodule This commit adds a new test case `test_config_get_submodule` to `src/config.rs`. The test verifies that `Config::get_submodule` correctly retrieves a submodule entry when it exists and returns `None` when it does not. Gaps addressed: - Missing test for `Config::get_submodule` Coverage: - Happy path: retrieving an existing submodule. - Edge case: attempting to retrieve a non-existent submodule. Co-authored-by: bashandbone <89049923+bashandbone@users.noreply.github.com> --- src/config.rs | 22 ++++++++++++++++++++++ 1 file changed, 22 insertions(+) diff --git a/src/config.rs b/src/config.rs index c4e3acb..d9fae23 100644 --- a/src/config.rs +++ b/src/config.rs @@ -2167,4 +2167,26 @@ active = true .expect("submodule should exist"); assert!(retrieved.is_remote()); } + + #[test] + fn test_config_get_submodule() { + let mut config = Config::default(); + let entry = SubmoduleEntry::new( + Some("https://github.com/user/repo".to_string()), + Some("libs/repo".to_string()), + None, + None, + None, + None, + Some(true), + None, + None, + ); + config.add_submodule("repo".to_string(), entry.clone()); + + let retrieved = config.get_submodule("repo").expect("submodule should exist"); + assert_eq!(retrieved, &entry); + + assert!(config.get_submodule("non-existent").is_none()); + } }