Skip to content

Commit

Permalink
Skip miniflare integration tests until testing is supported in Minifl…
Browse files Browse the repository at this point in the history
  • Loading branch information
tygern committed Feb 13, 2024
1 parent a6dd92f commit ff42a3f
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 8 deletions.
7 changes: 2 additions & 5 deletions .github/workflows/pipeline.yml
Original file line number Diff line number Diff line change
Expand Up @@ -19,11 +19,8 @@ jobs:
cache: npm
- name: 'Install dependencies'
run: npm ci
- name: 'Migrate local database'
uses: cloudflare/wrangler-action@v3
with:
wranglerVersion: "3.28.2"
command: d1 migrations apply starter-db --local
- name: 'Migrate'
run: npm run migrate
- name: 'Run tests'
run: npm test
- name: 'Build app'
Expand Down
6 changes: 3 additions & 3 deletions test/index.test.ts
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,7 @@ describe('Starter app', () => {
await worker.stop();
});

it('loads the index', async () => {
test.skip('loads the index', async () => {
const response = await worker.fetch('/');

expect(response.status).toEqual(200)
Expand All @@ -22,7 +22,7 @@ describe('Starter app', () => {
expect(text).toContain('log in');
});

it('loads the dashboard', async () => {
test.skip('loads the dashboard', async () => {
const response = await worker.fetch('/dashboard', {
headers: {
'starter-proxied': 'true',
Expand All @@ -36,7 +36,7 @@ describe('Starter app', () => {
expect(text).toContain('Welcome, [email protected]!');
});

it('blocks requests to the dashboard without a proxied header', async () => {
test.skip('blocks requests to the dashboard without a proxied header', async () => {
const response = await worker.fetch('/dashboard');

expect(response.status).toEqual(403)
Expand Down

0 comments on commit ff42a3f

Please sign in to comment.