chore: fix integration success to always report status

block · Jul 23, 2024 · 4bf9e51 · 4bf9e51
1 parent 787444e
commit 4bf9e51
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 3 deletions.
diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml
@@ -207,6 +207,13 @@ jobs:
     name: Integration Success
     needs: [integration-run]
     runs-on: ubuntu-latest
+    if: ${{ always() }}
     steps:
-      - name: Ok
-        run: echo "Integration tests passed"
+      - name: Check integration tests result
+        run: |
+          if [[ "${{ needs.integration-run.result }}" == "failure" ]]; then
+            echo "Integration tests failed"
+            exit 1
+          else
+            echo "Integration tests passed"
+          fi
diff --git a/backend/controller/ingress/ingress_integration_test.go b/backend/controller/ingress/ingress_integration_test.go
@@ -19,7 +19,7 @@ func TestHttpIngress(t *testing.T) {
 		in.Deploy("httpingress"),
 		in.HttpCall(http.MethodGet, "/users/123/posts/456", nil, in.JsonData(t, in.Obj{}), func(t testing.TB, resp *in.HTTPResponse) {
 			assert.Equal(t, 200, resp.Status)
-			assert.Equal(t, []string{"Header from FTL"}, resp.Headers["Get"])
+			assert.Equal(t, []string{"Header from TL"}, resp.Headers["Get"])
 			assert.Equal(t, []string{"application/json; charset=utf-8"}, resp.Headers["Content-Type"])
 
 			message, ok := resp.JsonBody["msg"].(string)