Patched CI

Author: jdjfisher

.github/workflows/ci.yml

@@ -15,7 +15,14 @@ jobs:
 
     steps:
       - uses: actions/checkout@v3
+
       - name: Build
         run: cargo build --verbose
+
+      - name: Set git config
+        run: |
+          git config --global user.email "<author@example.com>"
+          git config --global user.name "A U Thor"
+
       - name: Run tests
         run: cargo test --verbose

tests/common/mod.rs

@@ -1,22 +1,17 @@
+use assert_cmd::prelude::OutputAssertExt;
 use assert_fs::{
     prelude::{FileWriteStr, PathAssert, PathChild},
     TempDir,
 };
 use std::process::Command;
 
 fn run_git_command(dir: &TempDir, subcommand: &str, args: Vec<&str>) {
-    let output = Command::new("git")
+    Command::new("git")
         .current_dir(&dir)
         .arg(subcommand)
         .args(args)
-        .output()
-        .unwrap();
-
-    assert!(
-        output.status.success(),
-        "{}",
-        String::from_utf8_lossy(&output.stdout)
-    );
+        .assert()
+        .success();
 }
 
 // TODO: Tidy, builder pattern?

tests/integration.rs

@@ -45,12 +45,12 @@ fn propagates_invalid_git_target_error() {
     Command::cargo_bin("git-step")
         .unwrap()
         .args(["-C", temp_dir.to_str().unwrap()])
-        .arg("master")
+        .arg("lorem")
         .assert()
         .failure()
         .code(1)
         .stderr(contains(
-            "error: pathspec \'master\' did not match any file(s) known to git",
+            "error: pathspec \'lorem\' did not match any file(s) known to git",
         ));
 }