Create test route task #102
Conversation
Mohammed-Sbeata
requested review from
nada-abuzaid,
Amal-Mousa,
KhaledToha and
AyaAljalees
__tests__/task.test.ts
Outdated
| import { | ||
| afterAll, beforeEach, describe, test, expect, | ||
| } from '@jest/globals'; |
There was a problem hiding this comment.
Jest puts each of these methods into the global environment so u don't have to import anything to use them.
There was a problem hiding this comment.
ok, thanks.
__tests__/task.test.ts
Outdated
| .send({ | ||
| title: 'add query tasks', | ||
| }) |
There was a problem hiding this comment.
read the taskSchema again and update the request body here.
There was a problem hiding this comment.
ok, updated it.
__tests__/task.test.ts
Outdated
|
|
||
| test('Should return 200 when add task', (done) => { | ||
| supertest(app) | ||
| .post('/project/10/task') |
There was a problem hiding this comment.
do u have project id 10 in ur database? if yes, do u have a manager role? please check to pass the test.
There was a problem hiding this comment.
yes, I'm sure
__tests__/task.test.ts
Outdated
| .send({ | ||
| title: 'add query tasks', | ||
| }) | ||
| .expect(200) |
There was a problem hiding this comment.
201 because it created or added task.
__tests__/task.test.ts
Outdated
| .end((err, res) => { | ||
| if (err) return done(err); | ||
| expect(res.body.message).toEqual('Task added successfully'); | ||
| expect(res.body.data).toBe('array'); |
There was a problem hiding this comment.
res.body.data does this return a text with value "array"?
There was a problem hiding this comment.
sorry, without single quote or deleted it
| } catch (err) { | ||
| console.log('DB Error: ', err); | ||
| } | ||
| }; |
There was a problem hiding this comment.
Why did you create this file? Khaled created it already, delete it please
There was a problem hiding this comment.
ok, deleted it.
Mohammed-Sbeata
added
awaiting-review
and removed
awaiting-review
request-change
labels
No description provided.