Revert "chore: update test"

This reverts commit 81b35e6.

crates/forge/tests/cli/cmd.rs

@@ -536,10 +536,31 @@ contract Greeter {
     cmd.arg("build");
 
     let output = cmd.stdout_lossy();
-    assert!(output.contains("Warning"), "{output}");
+    assert!(output.contains(
+        "
+Compiler run successful with warnings:
+Warning (5667): Warning: Unused function parameter. Remove or comment out the variable name to silence this warning.
+",
+    ));
 });
 
 // Tests that direct import paths are handled correctly
+//
+// NOTE(onbjerg): Disabled for Windows -- for some reason solc fails with a bogus error message
+// here: error[9553]: TypeError: Invalid type for argument in function call. Invalid implicit
+// conversion from struct Bar memory to struct Bar memory requested.   --> src\Foo.sol:12:22:
+//    |
+// 12 |         FooLib.check(b);
+//    |                      ^
+//
+//
+//
+// error[9553]: TypeError: Invalid type for argument in function call. Invalid implicit conversion
+// from contract Foo to contract Foo requested.   --> src\Foo.sol:15:23:
+//    |
+// 15 |         FooLib.check2(this);
+//    |                       ^^^^
+#[cfg(not(target_os = "windows"))]
 forgetest!(can_handle_direct_imports_into_src, |prj, cmd| {
     prj.add_source(
         "Foo",
@@ -1551,12 +1572,8 @@ forgetest_init!(can_install_missing_deps_build, |prj, cmd| {
     cmd.arg("build");
 
     let output = cmd.stdout_lossy();
-    assert!(output.contains("Missing dependencies found. Installing now"), "{output}");
-
-    // re-run
-    let output = cmd.stdout_lossy();
-    assert!(!output.contains("Missing dependencies found. Installing now"), "{output}");
-    assert!(output.contains("No files changed, compilation skipped"), "{output}");
+    assert!(output.contains("Missing dependencies found. Installing now"), "{}", output);
+    assert!(output.contains("No files changed, compilation skipped"), "{}", output);
 });
 
 // checks that extra output works

crates/forge/tests/cli/config.rs

@@ -380,7 +380,7 @@ contract Foo {}
 // test to ensure yul optimizer can be set as intended
 forgetest!(can_set_yul_optimizer, |prj, cmd| {
     prj.add_source(
-        "foo.sol",
+        "Foo",
         r"
 contract Foo {
     function bar() public pure {
@@ -405,7 +405,12 @@ contract Foo {
         ..Default::default()
     };
     prj.write_config(config);
-    cmd.assert_success();
+
+    assert!(cmd.stdout_lossy().ends_with(
+        "
+Compiler run successful!
+",
+    ));
 });
 
 // tests that the lib triple can be parsed

crates/forge/tests/fixtures/can_set_yul_optimizer.stderr

@@ -1,8 +1,9 @@
 Error: 
 Compiler run failed:
-Error (6553): The msize instruction cannot be used when the Yul optimizer is activated because it can change its semantics. Either disable the Yul optimizer or do not use the instruction.
+Error (6553): SyntaxError: The msize instruction cannot be used when the Yul optimizer is activated because it can change its semantics. Either disable the Yul optimizer or do not use the instruction.
  --> src/Foo.sol:6:8:
   |
 6 |        assembly {
   |        ^ (Relevant source part starts here and spans across multiple lines).
 
+