Merge pull request #242 from cakebaker/tests_replace_use_of_run

tests: use `succeeds()` instead of `run()`

tests/by-util/test_dmesg.rs

@@ -26,7 +26,7 @@ fn test_kmsg_json() {
         .arg("--kmsg-file")
         .arg("kmsg.input")
         .arg("--json")
-        .run()
+        .succeeds()
         .no_stderr()
         .stdout_is_templated_fixture("test_kmsg_json.expected", &[("\r\n", "\n")]);
 }

tests/by-util/test_setsid.rs

@@ -130,11 +130,11 @@ mod unix {
         let cmd_result = UCommand::new()
             .terminal_simulation(true)
             .arg(&shell_cmd)
-            .run();
+            .succeeds();
 
-        let output = cmd_result.code_is(0).no_stderr().stdout_str();
+        let output = cmd_result.no_stderr().stdout_str();
 
-        // /proc/self/stat format has sessiion ID as the 6th space-separated
+        // /proc/self/stat format has session ID as the 6th space-separated
         // item; if we managed to get session leadership, we should see a
         // difference there...
         let (before, after) = output