refactor: tests: add remaining tests (multi-command ones) besides ones with select (#1619)

* Fixes #1453.

This adds all multi-command refactor tests besides tests with `select`.

This runs multiple refactor commands with just multiple
`refactor(...).test()`s. This generates a separate snapshot for each
command, but this should be fine, and gives us more data to inspect if
anything goes wrong.

The old tests used `;` for this, which avoids running the refactorer
again. Now we run the refactorer from scratch for each command. This
avoids the need to `commit` in between certain commands in order to
re-evaluate the AST, as `commit` is broken (see #1605).

Author: kkysen

c2rust-refactor/tests/abstract/run.sh

@@ -82,7 +82,6 @@ impl<'a> RefactorTest<'a> {
         }
     }
 
-    #[allow(unused)] // TODO remove when used
     pub fn expect_compile_error(self, expect_error: bool) -> Self {
         self.old_expect_compile_error(expect_error)
             .new_expect_compile_error(expect_error)
@@ -216,6 +215,45 @@ fn test_refactor(
 
 // NOTE: Tests should be listed in alphabetical order.
 
+/// TODO Broken.
+/// The generated `fn add` is marked `unsafe` when it doesn't appear it should be.
+#[test]
+fn test_abstract() {
+    refactor("abstract")
+        .command_args(&["add(x: i32, y: i32) -> i32", "x + y"])
+        .named("abstract.rs")
+        .new_expect_compile_error(true)
+        .test();
+    // no commit
+    refactor("abstract")
+        .command_args(&[
+            "sub<T: Sub<T, Result=T>>(x: T, y: T) -> T",
+            "typed!(x, T) - y",
+            "x - y",
+        ])
+        .named("abstract.new")
+        .expect_compile_error(true)
+        .test();
+}
+
+#[test]
+fn test_autoretype_array() {
+    refactor("rewrite_expr")
+        .command_args(&["1 + 1", "2"])
+        .named("autoretype_array.rs")
+        .test();
+    refactor("autoretype").named("autoretype_array.new").test();
+}
+
+#[test]
+fn test_autoretype_method() {
+    refactor("rewrite_expr")
+        .command_args(&["1 + 1", "2"])
+        .named("autoretype_method.rs")
+        .test();
+    refactor("autoretype").named("autoretype_method.new").test();
+}
+
 #[test]
 fn test_bitcast_retype() {
     refactor("bitcast_retype")
@@ -289,6 +327,28 @@ fn test_matcher_typed() {
         .test();
 }
 
+/// This test was supposed to test if changes are visible across `commit`s,
+/// even when those changes aren't written to the original file
+/// (like with the `--rewrite-mode alongside` used by [`refactor`],
+/// and unlike with `--rewrite-mode inplace`).
+/// However, `commit` is currently broken (see #1605),
+/// so this test is not actually testing what it's meant to.
+/// The places where `commit`s are supposed to go
+/// are left as comments for now until we fix `commit`.
+#[test]
+fn test_multi_rewrite() {
+    refactor("rewrite_expr")
+        .command_args(&["1", "2"])
+        .named("multi_rewrite.rs")
+        .test();
+    // commit
+    refactor("rewrite_expr")
+        .command_args(&["2", "3"])
+        .named("multi_rewrite.new")
+        .test();
+    // commit
+}
+
 /// TODO Broken.
 /// This panics with the error:
 ///