Merge pull request #926 from g-r-a-n-t/string-tests-bug

Test assertion string bugfix

crates/codegen/src/yul/isel/test.rs

@@ -8,12 +8,12 @@ pub fn lower_test(db: &dyn CodegenDb, test: FunctionId) -> yul::Object {
     let test = db.mir_lowered_func_signature(test);
     context.function_dependency.insert(test);
 
-    let dep_constants = context.resolve_constant_dependency(db);
     let dep_functions: Vec<_> = context
         .resolve_function_dependency(db)
         .into_iter()
         .map(yul::Statement::FunctionDefinition)
         .collect();
+    let dep_constants = context.resolve_constant_dependency(db);
     let dep_contracts = context.resolve_contract_dependency(db);
     let runtime_funcs: Vec<_> = context
         .runtime

crates/tests/fixtures/files/assert_with_string.fe

@@ -0,0 +1,4 @@
+#test 
+fn foo() {
+    assert true, "oops"
+}

newsfragments/926.bugfix.md

@@ -0,0 +1,12 @@
+Yul codegen was failing to include string literals used in test assertions. This resulted in a compiler error.
+
+Example:
+
+```
+#test
+fn foo() {
+    assert false, "oops"
+}
+```
+
+The example code above was failing to compile, but now it compiles and executes as expected.