fix: Pass `--no-gpg-sign` to `git commit` in tests (#1718)

Some users have commit.gpgSign set to true in their global git config,
causing tests which run `git commit` to fail if the configured
user.signingKey is not present (as is often the case with PGP smart
cards). Passing this flag overrides the global git configuration,
preventing git from attempting sign the specified commit. This change
has no effect on users who haven't set commit.gpgSign to true.

2 files changed, 11 insertions, 7 deletions

diff --git a/src/modules/git_state.rs b/src/modules/git_state.rs
index 20089f838..a7bfaf4f4 100644
--- a/src/modules/git_state.rs
+++ b/src/modules/git_state.rs
@@ -334,13 +334,17 @@ mod tests {
         // Write a file on master and commit it
         write_file("Version A")?;
         run_git_cmd(&["add", "the_file"], Some(path), true)?;
-        run_git_cmd(&["commit", "--message", "Commit A"], Some(path), true)?;
+        run_git_cmd(
+            &["commit", "--message", "Commit A", "--no-gpg-sign"],
+            Some(path),
+            true,
+        )?;
 
         // Switch to another branch, and commit a change to the file
         run_git_cmd(&["checkout", "-b", "other-branch"], Some(path), true)?;
         write_file("Version B")?;
         run_git_cmd(
-            &["commit", "--all", "--message", "Commit B"],
+            &["commit", "--all", "--message", "Commit B", "--no-gpg-sign"],
             Some(path),
             true,
         )?;
@@ -349,7 +353,7 @@ mod tests {
         run_git_cmd(&["checkout", "master"], Some(path), true)?;
         write_file("Version C")?;
         run_git_cmd(
-            &["commit", "--all", "--message", "Commit C"],
+            &["commit", "--all", "--message", "Commit C", "--no-gpg-sign"],
             Some(path),
             true,
         )?;
diff --git a/src/modules/git_status.rs b/src/modules/git_status.rs
index 9daf94c05..621d3e608 100644
--- a/src/modules/git_status.rs
+++ b/src/modules/git_status.rs
@@ -813,7 +813,7 @@ mod tests {
             .current_dir(&repo_dir.path())
             .output()?;
         Command::new("git")
-            .args(&["commit", "-m", "add new files"])
+            .args(&["commit", "-m", "add new files", "--no-gpg-sign"])
             .current_dir(&repo_dir.path())
             .output()?;
 
@@ -842,7 +842,7 @@ mod tests {
         File::create(repo_dir.join("readme.md"))?.sync_all()?;
 
         Command::new("git")
-            .args(&["commit", "-am", "Update readme"])
+            .args(&["commit", "-am", "Update readme", "--no-gpg-sign"])
             .current_dir(&repo_dir)
             .output()?;
         barrier();
@@ -870,7 +870,7 @@ mod tests {
         fs::write(repo_dir.join("Cargo.toml"), " ")?;
 
         Command::new("git")
-            .args(&["commit", "-am", "Update readme"])
+            .args(&["commit", "-am", "Update readme", "--no-gpg-sign"])
             .current_dir(repo_dir)
             .output()?;
         barrier();
@@ -894,7 +894,7 @@ mod tests {
         barrier();
 
         Command::new("git")
-            .args(&["commit", "-m", "Change readme"])
+            .args(&["commit", "-m", "Change readme", "--no-gpg-sign"])
             .current_dir(repo_dir)
             .output()?;
         barrier();