Add more e2e tests

Author: franciszekjob

crates/forge/tests/data/declare_from_file/data/invalid_contract_class.json

@@ -0,0 +1,3 @@
+{
+  "name": "not a Sierra contract class"
+}

crates/forge/tests/data/declare_from_file/tests/tests.cairo

@@ -24,3 +24,13 @@ fn already_declared() {
         _ => panic!("expected AlreadyDeclared"),
     }
 }
+
+#[test]
+fn missing_file() {
+    declare_from_file("data/missing.contract_class.json").unwrap();
+}
+
+#[test]
+fn invalid_json() {
+    declare_from_file("data/invalid_contract_class.json").unwrap();
+}

crates/forge/tests/e2e/declare_from_file.rs

@@ -5,7 +5,7 @@ use shared::test_utils::output_assert::assert_stdout_contains;
 use snapbox::cmd::Command as SnapboxCommand;
 
 #[test]
-fn declare_from_file() {
+fn simple() {
     let temp = setup_package("declare_from_file");
 
     SnapboxCommand::from_std(
@@ -17,21 +17,86 @@ fn declare_from_file() {
     .assert()
     .success();
 
-    let output = test_runner(&temp).assert().success();
+    let output = test_runner(&temp).arg("simple").assert().success();
 
     assert_stdout_contains(
         output,
         indoc! {r"
         [..]Compiling[..]
         [..]Finished[..]
 
-
-        Collected 2 test(s) from declare_from_file package
+        Collected 1 test(s) from declare_from_file package
         Running 0 test(s) from src/
-        Running 2 test(s) from tests/
+        Running 1 test(s) from tests/
         [PASS] declare_from_file_integrationtest::tests::simple [..]
+        Tests: 1 passed, 0 failed, 0 ignored, 1 filtered out
+        "},
+    );
+}
+
+#[test]
+fn already_declared() {
+    let temp = setup_package("declare_from_file");
+
+    SnapboxCommand::from_std(
+        ScarbCommand::new()
+            .current_dir(temp.path())
+            .arg("build")
+            .command(),
+    )
+    .assert()
+    .success();
+
+    let output = test_runner(&temp)
+        .arg("already_declared")
+        .assert()
+        .success();
+
+    assert_stdout_contains(
+        output,
+        indoc! {r"
+        [..]Compiling[..]
+        [..]Finished[..]
+
+        Collected 1 test(s) from declare_from_file package
+        Running 0 test(s) from src/
+        Running 1 test(s) from tests/
         [PASS] declare_from_file_integrationtest::tests::already_declared [..]
-        Tests: 2 passed, 0 failed, 0 ignored, 0 filtered out
+        Tests: 1 passed, 0 failed, 0 ignored, 1 filtered out
         "},
     );
 }
+
+#[test]
+fn missing_file() {
+    let temp = setup_package("declare_from_file_failures");
+
+    let output = test_runner(&temp).arg("missing_file").assert().code(1);
+
+    assert_stdout_contains(
+        output,
+        indoc! {r#"
+        [FAIL] declare_from_file_failures_integrationtest::tests::missing_file
+
+        Failure data:
+            "Failed to read Sierra file at data/missing.contract_class.json: [..]"
+        "#},
+    );
+}
+
+#[test]
+fn invalid_json() {
+    let temp = setup_package("declare_from_file_failures");
+
+    let output = test_runner(&temp).arg("invalid_json").assert().code(1);
+
+    assert_stdout_contains(
+        output,
+        indoc! {r#"
+        [FAIL] declare_from_file_failures_integrationtest::tests::invalid_json
+
+        Failure data:
+            "Failed to parse Sierra contract class JSON at data/invalid_contract_class.json: [..]"
+        "#},
+    );
+}