diff --git a/.github/workflows/run-tests.yaml b/.github/workflows/run-tests.yaml index 12399533e..3c735a777 100644 --- a/.github/workflows/run-tests.yaml +++ b/.github/workflows/run-tests.yaml @@ -73,14 +73,6 @@ jobs: source ~/.nvm/nvm.sh yarn build yarn lint - env: - PG_HOST: localhost - PG_PORT: ${{ job.services.postgres.ports[5432] }} - PG_USER: app_user - PG_PASSWORD: app_pass - PG_DB: omnivore_test - PG_POOL_MAX: 10 - REDIS_URL: redis://localhost:${{ job.services.redis.ports[6379] }} - name: Tests run: | source ~/.nvm/nvm.sh @@ -91,7 +83,6 @@ jobs: PG_USER: app_user PG_PASSWORD: app_pass PG_DB: omnivore_test - PG_POOL_MAX: 10 PG_LOGGER: debug REDIS_URL: redis://localhost:${{ job.services.redis.ports[6379] }} build-docker-images: diff --git a/packages/api/src/events/user/user_created.ts b/packages/api/src/events/user/user_created.ts index 25c5e15cc..eb0fa1fc1 100644 --- a/packages/api/src/events/user/user_created.ts +++ b/packages/api/src/events/user/user_created.ts @@ -50,15 +50,15 @@ export class PublishNewUserEvent implements EntitySubscriberInterface { } } -@EventSubscriber() -export class AddPopularReadsToNewUser - implements EntitySubscriberInterface -{ - listenTo() { - return Profile - } +// @EventSubscriber() +// export class AddPopularReadsToNewUser +// implements EntitySubscriberInterface +// { +// listenTo() { +// return Profile +// } - async afterInsert(event: InsertEvent): Promise { - await addPopularReadsForNewUser(event.entity.user.id, event.manager) - } -} +// async afterInsert(event: InsertEvent): Promise { +// await addPopularReadsForNewUser(event.entity.user.id, event.manager) +// } +// } diff --git a/packages/api/test/resolvers/webhooks.test.ts b/packages/api/test/resolvers/webhooks.test.ts index 0c5cce41f..989ab214b 100644 --- a/packages/api/test/resolvers/webhooks.test.ts +++ b/packages/api/test/resolvers/webhooks.test.ts @@ -80,7 +80,7 @@ describe('Webhooks API', () => { } ` - const res = await graphqlRequest(query, authToken) + const res = await graphqlRequest(query, authToken).expect(200) expect(res.body.data.webhook.webhook.id).to.eql(webhook.id) expect(res.body.data.webhook.webhook.url).to.eql(webhook.url) @@ -108,7 +108,7 @@ describe('Webhooks API', () => { } ` - const res = await graphqlRequest(query, authToken) + const res = await graphqlRequest(query, authToken).expect(200) const webhooks = await findWebhooks(user.id) expect(res.body.data.webhooks.webhooks).to.eql( @@ -165,7 +165,7 @@ describe('Webhooks API', () => { }) it('should create a webhook', async () => { - const res = await graphqlRequest(query, authToken) + const res = await graphqlRequest(query, authToken).expect(200) expect(res.body.data.setWebhook.webhook).to.be.an('object') expect(res.body.data.setWebhook.webhook.url).to.eql(webhookUrl) @@ -195,7 +195,7 @@ describe('Webhooks API', () => { }) it('should update a webhook', async () => { - const res = await graphqlRequest(query, authToken) + const res = await graphqlRequest(query, authToken).expect(200) expect(res.body.data.setWebhook.webhook).to.be.an('object') expect(res.body.data.setWebhook.webhook.url).to.eql(webhookUrl) @@ -240,7 +240,7 @@ describe('Webhooks API', () => { }) it('should delete a webhook', async () => { - const res = await graphqlRequest(query, authToken) + const res = await graphqlRequest(query, authToken).expect(200) const webhook = await findWebhookById(webhookId, user.id) expect(res.body.data.deleteWebhook.webhook).to.be.an('object')