remove redaudant discovery check in automated test

aec58b3b36d85447b38369285b857c2864ff6ef8 · 2024-07-20 16:13:01 -0700 · Alex Cheema

Files touched

M .github/workflows/test.yml

Diff

commit aec58b3b36d85447b38369285b857c2864ff6ef8
Author: Alex Cheema <alexcheema123@gmail.com>
Date:   Sat Jul 20 16:13:01 2024 -0700

    remove redaudant discovery check in automated test
---
 .github/workflows/test.yml | 20 +++-----------------
 1 file changed, 3 insertions(+), 17 deletions(-)

diff --git a/.github/workflows/test.yml b/.github/workflows/test.yml
index 0e9c9335..d69b7909 100644
--- a/.github/workflows/test.yml
+++ b/.github/workflows/test.yml
@@ -148,6 +148,9 @@ jobs:
           }')
         echo "Response 2: $response_2"
 
+        # Stop both instances
+        kill $PID1 $PID2
+
         echo ""
         if ! echo "$response_1" | grep -q "Michael Jackson" || ! echo "$response_2" | grep -q "Michael Jackson"; then
           echo "Test failed: Response does not contain 'Michael Jackson'"
@@ -158,20 +161,3 @@ jobs:
         else
           echo "Test passed: Response from both nodes contains 'Michael Jackson'"
         fi
-
-        # Stop both instances
-        kill $PID1 $PID2
-
-        # Check outputs
-        if grep -q "Connected to peer" output1.log && grep -q "Connected to peer" output2.log; then
-          echo "Test passed: Both instances discovered each other"
-          exit 0
-        else
-          echo "Test failed: Devices did not discover each other"
-          echo "Output of first instance:"
-          cat output1.log
-          echo ""
-          echo "Output of second instance:"
-          cat output2.log
-          exit 1
-        fi
\ No newline at end of file

← 9785e250 formatting if · back to Exo · debug cached files in workflow 3d82338c →