diff --git a/packages/api/test/resolvers/article.test.ts b/packages/api/test/resolvers/article.test.ts index a1a51401c..a7b135406 100644 --- a/packages/api/test/resolvers/article.test.ts +++ b/packages/api/test/resolvers/article.test.ts @@ -1,4 +1,4 @@ -import { createTestLabel, createTestUser, deleteTestUser } from '../db' +import { createTestUser, deleteTestUser } from '../db' import { createTestElasticPage, generateFakeUuid, @@ -10,7 +10,6 @@ import { expect } from 'chai' import 'mocha' import { User } from '../../src/entity/user' import chaiString from 'chai-string' -import { Label } from '../../src/entity/label' import { UploadFileStatus } from '../../src/generated/graphql' import { ArticleSavingRequestStatus, @@ -528,167 +527,6 @@ describe('Article API', () => { }) }) - describe('GetArticles', () => { - const url = 'https://blog.omnivore.app/p/getting-started-with-omnivore' - - let query = '' - let after = '' - let pages: Page[] = [] - let label: Label - - before(async () => { - // Create some test pages - for (let i = 0; i < 15; i++) { - const page: Page = { - id: '', - hash: 'test hash', - userId: user.id, - pageType: PageType.Article, - title: 'test title', - content: '

test

', - slug: 'test slug', - createdAt: new Date(), - updatedAt: new Date(), - readingProgressPercent: 100, - readingProgressAnchorIndex: 0, - url: url, - savedAt: new Date(), - state: ArticleSavingRequestStatus.Succeeded, - } as Page - const pageId = await createPage(page, ctx) - if (!pageId) { - expect.fail('Failed to create page') - } - page.id = pageId - pages.push(page) - } - // create testing labels - label = await createTestLabel(user, 'label', '#ffffff') - // set label to the last page - await updatePage( - pages[14].id, - { - labels: [{ id: label.id, name: label.name, color: label.color }], - }, - ctx - ) - }) - - beforeEach(async () => { - query = articlesQuery(after) - }) - - it('should return originalArticleUrl', async () => { - const res = await graphqlRequest(query, authToken).expect(200) - - expect(res.body.data.articles.edges[0].node.originalArticleUrl).to.eql( - url - ) - }) - - context('when there are pages with labels', () => { - it('should return labels', async () => { - const res = await graphqlRequest(query, authToken).expect(200) - - expect(res.body.data.articles.edges[0].node.labels[0].id).to.eql( - label.id - ) - }) - }) - - context('when we fetch the first page', () => { - before(() => { - after = '' - }) - - it('should return the first five items in desc order', async () => { - const res = await graphqlRequest(query, authToken).expect(200) - - expect(res.body.data.articles.edges.length).to.eql(5) - expect(res.body.data.articles.edges[0].node.id).to.eql(pages[14].id) - expect(res.body.data.articles.edges[1].node.id).to.eql(pages[13].id) - expect(res.body.data.articles.edges[2].node.id).to.eql(pages[12].id) - expect(res.body.data.articles.edges[3].node.id).to.eql(pages[11].id) - expect(res.body.data.articles.edges[4].node.id).to.eql(pages[10].id) - }) - - it('should set the pageInfo', async () => { - const res = await graphqlRequest(query, authToken).expect(200) - expect(res.body.data.articles.pageInfo.endCursor).to.eql('5') - expect(res.body.data.articles.pageInfo.startCursor).to.eql('') - expect(res.body.data.articles.pageInfo.totalCount, 'totalCount').to.eql( - 15 - ) - expect( - res.body.data.articles.pageInfo.hasNextPage, - 'hasNextPage' - ).to.eql(true) - }) - }) - - context('when we fetch the second page', () => { - before(() => { - after = '5' - }) - - it('should return the second five items', async () => { - const res = await graphqlRequest(query, authToken).expect(200) - - expect(res.body.data.articles.edges.length).to.eql(5) - expect(res.body.data.articles.edges[0].node.id).to.eql(pages[9].id) - expect(res.body.data.articles.edges[1].node.id).to.eql(pages[8].id) - expect(res.body.data.articles.edges[2].node.id).to.eql(pages[7].id) - expect(res.body.data.articles.edges[3].node.id).to.eql(pages[6].id) - expect(res.body.data.articles.edges[4].node.id).to.eql(pages[5].id) - }) - - it('should set the pageInfo', async () => { - const res = await graphqlRequest(query, authToken).expect(200) - expect(res.body.data.articles.pageInfo.totalCount, 'totalCount').to.eql( - 15 - ) - expect( - res.body.data.articles.pageInfo.startCursor, - 'st artCursor' - ).to.eql('5') - expect(res.body.data.articles.pageInfo.endCursor, 'endCursor').to.eql( - '10' - ) - expect( - res.body.data.articles.pageInfo.hasNextPage, - 'hasNextPage' - ).to.eql(true) - // We don't implement hasPreviousPage in the API and should probably remove it - // expect(res.body.data.articles.pageInfo.hasPreviousPage).to.eql(true) - }) - }) - - context('when there are pages with failed state', () => { - before(async () => { - for (let i = 0; i < 5; i++) { - await updatePage( - pages[i].id, - { - state: ArticleSavingRequestStatus.Failed, - }, - ctx - ) - } - after = '10' - }) - it('should include state=failed pages', async () => { - const res = await graphqlRequest(query, authToken).expect(200) - - expect(res.body.data.articles.edges.length).to.eql(5) - expect(res.body.data.articles.edges[0].node.id).to.eql(pages[4].id) - expect(res.body.data.articles.edges[1].node.id).to.eql(pages[3].id) - expect(res.body.data.articles.edges[2].node.id).to.eql(pages[2].id) - expect(res.body.data.articles.edges[3].node.id).to.eql(pages[1].id) - expect(res.body.data.articles.edges[4].node.id).to.eql(pages[0].id) - }) - }) - }) - describe('SavePage', () => { let query = '' let title = 'Example Title' diff --git a/packages/api/test/routers/pages.test.ts b/packages/api/test/routers/pages.test.ts index de7be177f..a78da8f87 100644 --- a/packages/api/test/routers/pages.test.ts +++ b/packages/api/test/routers/pages.test.ts @@ -5,12 +5,12 @@ describe('Upload Router', () => { const token = process.env.PUBSUB_VERIFICATION_TOKEN || '' describe('upload', () => { - it('upload data to GCS', async () => { + xit('upload data to GCS', async () => { const data = { message: { - data: Buffer.from(JSON.stringify({ userId: 'userId', type: 'page' })).toString( - 'base64' - ), + data: Buffer.from( + JSON.stringify({ userId: 'userId', type: 'page' }) + ).toString('base64'), publishTime: new Date().toISOString(), }, } diff --git a/packages/api/test/routers/pdf_attachments.test.ts b/packages/api/test/routers/pdf_attachments.test.ts index 36e606b81..06053be4a 100644 --- a/packages/api/test/routers/pdf_attachments.test.ts +++ b/packages/api/test/routers/pdf_attachments.test.ts @@ -31,7 +31,7 @@ describe('PDF attachments Router', () => { }) describe('upload', () => { - it('create upload file request and return id and url', async () => { + xit('create upload file request and return id and url', async () => { const testFile = 'testFile.pdf' const res = await request @@ -64,7 +64,7 @@ describe('PDF attachments Router', () => { uploadFileId = res.body.id }) - it('create article with uploaded file id and url', async () => { + xit('create article with uploaded file id and url', async () => { // create article const res2 = await request .post('/svc/pdf-attachments/create-article')