From 352851f23f5600e05cb403188ac8de8b83c41ec6 Mon Sep 17 00:00:00 2001 From: Johann150 Date: Sun, 7 May 2023 23:34:34 +0200 Subject: [PATCH] tests: fix error codes and HTTP statuses --- packages/backend/test/block.ts | 8 ++++---- packages/backend/test/note.ts | 10 +++++----- 2 files changed, 9 insertions(+), 9 deletions(-) diff --git a/packages/backend/test/block.ts b/packages/backend/test/block.ts index b3343813c..c35999d5c 100644 --- a/packages/backend/test/block.ts +++ b/packages/backend/test/block.ts @@ -35,7 +35,7 @@ describe('Block', () => { const res = await request('/following/create', { userId: alice.id }, bob); assert.strictEqual(res.status, 400); - assert.strictEqual(res.body.error.id, 'c4ab57cc-4e41-45e9-bfd9-584f61e35ce0'); + assert.strictEqual(res.body.error.code, 'BLOCKED'); })); it('ブロックされているユーザーにリアクションできない', async(async () => { @@ -44,7 +44,7 @@ describe('Block', () => { const res = await request('/notes/reactions/create', { noteId: note.id, reaction: '👍' }, bob); assert.strictEqual(res.status, 400); - assert.strictEqual(res.body.error.id, '20ef5475-9f38-4e4c-bd33-de6d979498ec'); + assert.strictEqual(res.body.error.code, 'BLOCKED'); })); it('ブロックされているユーザーに返信できない', async(async () => { @@ -53,7 +53,7 @@ describe('Block', () => { const res = await request('/notes/create', { replyId: note.id, text: 'yo' }, bob); assert.strictEqual(res.status, 400); - assert.strictEqual(res.body.error.id, 'b390d7e1-8a5e-46ed-b625-06271cafd3d3'); + assert.strictEqual(res.body.error.code, 'BLOCKED'); })); it('ブロックされているユーザーのノートをRenoteできない', async(async () => { @@ -62,7 +62,7 @@ describe('Block', () => { const res = await request('/notes/create', { renoteId: note.id, text: 'yo' }, bob); assert.strictEqual(res.status, 400); - assert.strictEqual(res.body.error.id, 'b390d7e1-8a5e-46ed-b625-06271cafd3d3'); + assert.strictEqual(res.body.error.code, 'BLOCKED'); })); // TODO: ユーザーリストに入れられないテスト diff --git a/packages/backend/test/note.ts b/packages/backend/test/note.ts index b495d8b7b..9cd5c650b 100644 --- a/packages/backend/test/note.ts +++ b/packages/backend/test/note.ts @@ -158,7 +158,7 @@ describe('Note', () => { replyId: '000000000000000000000000', }; const res = await request('/notes/create', post, alice); - assert.strictEqual(res.status, 400); + assert.strictEqual(res.status, 404); })); it('存在しないrenote対象で怒られる', async(async () => { @@ -166,7 +166,7 @@ describe('Note', () => { renoteId: '000000000000000000000000', }; const res = await request('/notes/create', post, alice); - assert.strictEqual(res.status, 400); + assert.strictEqual(res.status, 404); })); it('不正なリプライ先IDで怒られる', async(async () => { @@ -175,7 +175,7 @@ describe('Note', () => { replyId: 'foo', }; const res = await request('/notes/create', post, alice); - assert.strictEqual(res.status, 400); + assert.strictEqual(res.status, 404); })); it('不正なrenote対象IDで怒られる', async(async () => { @@ -183,7 +183,7 @@ describe('Note', () => { renoteId: 'foo', }; const res = await request('/notes/create', post, alice); - assert.strictEqual(res.status, 400); + assert.strictEqual(res.status, 404); })); it('存在しないユーザーにメンションできる', async(async () => { @@ -286,7 +286,7 @@ describe('Note', () => { choice: 2, }, alice); - assert.strictEqual(res.status, 400); + assert.strictEqual(res.status, 409); })); it('許可されている場合は複数投票できる', async(async () => {