fix CI test spec and print output correctly #69

.github/workflows/e2e-test.yml

@@ -52,7 +52,7 @@ jobs:
       - name: Run sandbox
         run: docker run --rm --name my-sandbox --detach -p 20000:20000 tqtezos/flextesa:20210602 flobox start
 
-      - name: Configure tezos client and run integration tests
+      - name: Configure tezos client and run all test targets
         uses: addnab/docker-run-action@v3
         with:
           image: ${{ steps.tags.outputs.tagged_image }}

.github/workflows/rust.yml

@@ -41,8 +41,8 @@ jobs:
       - name: Build
         run: cargo build --all-features --all-targets --verbose
 
-      - name: Run all unit + library tests (not doc or integration tests)
-        run: cargo test --all-features --all-targets --lib --verbose 
+      - name: Run all unit + library tests (not doc, integration, benchmark, or example tests)
+        run: cargo test --all-features --lib --bins
 
       - name: Run all doctests
         run: cargo test --all-features --doc --verbose

integration_tests/main.rs

@@ -35,10 +35,12 @@ pub async fn main() {
     println!("Executing {} tests", tests.len());
     let mut results = Vec::with_capacity(tests.len());
     for test in tests {
-        println!("\nNow running: {}", test.name);
+        eprintln!("\n\ntest integration_tests::{} ... ", test.name);
         let result = test.execute(&rng, &customer_config, &merchant_config).await;
         if let Err(error) = &result {
-            eprintln!("Test failed: {}\n{:?}", test.name, error)
+            eprintln!("failed\n{:?}", error)
+        } else {
+            eprintln!("ok")
         }
         results.push(result);
 
@@ -53,8 +55,10 @@ pub async fn main() {
     common::teardown(server_future).await;
 
     // Fail if any test failed. This is separate from evaluation to enforce that _every_ test must
-    // run -- otherwise, any will short-circuit the execution
-    assert!(results.iter().all(|result| result.is_ok()));
+    // run without short-circuiting the execution at first failure
+    if !results.iter().all(|result| result.is_ok()) {
+        std::process::exit(101);
+    }
 }
 
 /// Get a list of tests to execute.