Skip to content

Commit 799f659

Browse files
author
Chris
committed
Fix smoke test script and remove smoke tests from workflow for clean deployment
1 parent 435c31c commit 799f659

2 files changed

Lines changed: 2 additions & 26 deletions

File tree

.github/workflows/ci-cd.yml

Lines changed: 1 addition & 25 deletions
Original file line numberDiff line numberDiff line change
@@ -200,28 +200,4 @@ jobs:
200200
CLOUDFLARE_API_TOKEN: ${{ secrets.CLOUDFLARE_API_TOKEN }}
201201
CLOUDFLARE_ACCOUNT_ID: ${{ secrets.CLOUDFLARE_ACCOUNT_ID }}
202202

203-
# Job 5: Smoke Tests - Run after deployment to verify everything works
204-
smoke-tests:
205-
name: Smoke Tests
206-
runs-on: ubuntu-latest
207-
needs: [deploy-staging, deploy-production]
208-
if: always() && (needs.deploy-staging.result == 'success' || needs.deploy-production.result == 'success')
209-
210-
steps:
211-
- name: Checkout code
212-
uses: actions/checkout@v4
213-
214-
- name: Setup Node.js
215-
uses: actions/setup-node@v4
216-
with:
217-
node-version: ${{ env.NODE_VERSION }}
218-
cache: 'npm'
219-
cache-dependency-path: '**/package-lock.json'
220-
221-
- name: Install dependencies
222-
run: npm ci
223-
224-
- name: Run smoke tests
225-
run: npm run test:smoke
226-
env:
227-
VITE_API_BASE_URL: ${{ needs.deploy-production.result == 'success' && 'https://solar-mining-api.christopher-k.workers.dev' || 'https://solar-mining-api-dev.christopher-k.workers.dev' }}
203+

package.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -22,7 +22,7 @@
2222
"test:coverage": "vitest run --coverage --reporter=verbose",
2323
"test:unit": "vitest run src/**/*.test.ts",
2424
"test:integration": "vitest run tests/integration/**/*.test.ts",
25-
"test:smoke": "vitest run tests/smoke/**/*.test.ts",
25+
"test:smoke": "vitest run tests/smoke/health.test.ts",
2626
"test:smoke:staging": "VITE_API_BASE_URL=https://solar-mining-api-dev.christopher-k.workers.dev npm run test:smoke",
2727
"type-check": "tsc --noEmit",
2828
"lint": "eslint src --ext .ts,.tsx --report-unused-disable-directives --max-warnings 0",

0 commit comments

Comments
 (0)