ci: remove wait-on installation from integration and unit test workflows, update wait logic in E2E test workflow

This commit is contained in:
Carlos Santos 2025-05-30 11:27:05 +02:00
parent 333c7da5b2
commit 5884f9341e
3 changed files with 16 additions and 30 deletions

View File

@ -10,8 +10,6 @@ jobs:
uses: actions/setup-node@v4
with:
node-version: '20'
- name: Install wait-on
run: npm install -g wait-on
- name: Install LK CLI
run: curl -sSL https://get.livekit.io/cli | bash
- name: Setup OpenVidu Local Deployment
@ -44,8 +42,6 @@ jobs:
uses: actions/setup-node@v4
with:
node-version: '20'
- name: Install wait-on
run: npm install -g wait-on
- name: Install LK CLI
run: curl -sSL https://get.livekit.io/cli | bash
- name: Setup OpenVidu Local Deployment
@ -77,8 +73,6 @@ jobs:
uses: actions/setup-node@v4
with:
node-version: '20'
- name: Install wait-on
run: npm install -g wait-on
- name: Install LK CLI
run: curl -sSL https://get.livekit.io/cli | bash
- name: Setup OpenVidu Local Deployment
@ -110,8 +104,6 @@ jobs:
uses: actions/setup-node@v4
with:
node-version: '20'
- name: Install wait-on
run: npm install -g wait-on
- name: Install LK CLI
run: curl -sSL https://get.livekit.io/cli | bash
- name: Setup OpenVidu Local Deployment
@ -143,8 +135,6 @@ jobs:
uses: actions/setup-node@v4
with:
node-version: '20'
- name: Install wait-on
run: npm install -g wait-on
- name: Install LK CLI
run: curl -sSL https://get.livekit.io/cli | bash
- name: Setup OpenVidu Local Deployment
@ -176,8 +166,6 @@ jobs:
uses: actions/setup-node@v4
with:
node-version: '20'
- name: Install wait-on
run: npm install -g wait-on
- name: Install LK CLI
run: curl -sSL https://get.livekit.io/cli | bash
- name: Setup OpenVidu Local Deployment
@ -209,8 +197,6 @@ jobs:
uses: actions/setup-node@v4
with:
node-version: '20'
- name: Install wait-on
run: npm install -g wait-on
- name: Install LK CLI
run: curl -sSL https://get.livekit.io/cli | bash
- name: Setup OpenVidu Local Deployment

View File

@ -10,8 +10,6 @@ jobs:
uses: actions/setup-node@v4
with:
node-version: '20'
- name: Install wait-on
run: npm install -g wait-on
- name: Setup OpenVidu Meet
uses: OpenVidu/actions/start-openvidu-meet@main
- name: Run tests

View File

@ -10,27 +10,31 @@ jobs:
uses: actions/setup-node@v4
with:
node-version: '20'
- name: Install wait-on
run: npm install -g wait-on
- name: Setup OpenVidu Local Deployment
uses: OpenVidu/actions/start-openvidu-local-deployment@main
- name: Setup OpenVidu Meet
shell: bash
run: |
./prepare.sh
cd backend
npm run start:prod > ../backend.log 2>&1 &
uses: OpenVidu/actions/start-openvidu-meet@main
env:
MEET_WEBHOOK_ENABLED: true
- name: Wait for OpenVidu Meet to Start
run: wait-on --timeout 30000 http://localhost:6080/meet/health
- name: Start testapp
shell: bash
run: |
cd testapp
npm run start > ../testapp.log 2>&1 &
- name: Wait for testapp to Start
run: wait-on --timeout 30000 http://localhost:5080
shell: bash
run: |
echo "Waiting for testapp to start on http://localhost:5080..."
for i in {1..30}; do
if curl -s http://localhost:5080 >/dev/null 2>&1; then
echo "Testapp is ready!"
exit 0
fi
echo "Attempt $i/30: Testapp not ready yet, waiting 1 second..."
sleep 1
done
echo "Timeout: Testapp failed to start within 30 seconds"
exit 1
- name: Run tests
run: |
docker ps -a
@ -42,14 +46,12 @@ jobs:
env:
RUN_MODE: CI
PLAYWRIGHT_BROWSERS_PATH: /tmp/ms-playwright
- name: Upload OpenVidu Meet logs
- name: Upload OpenVidu Testapp logs
if: always()
uses: actions/upload-artifact@v4
with:
name: meet-logs
path: |
backend.log
testapp.log
path: testapp.log
retention-days: 2
- name: Upload failed test videos
if: always()