-
Notifications
You must be signed in to change notification settings - Fork 14
Update unittests #1784
Update unittests #1784
Conversation
@queengooborg sorry I neglected this. Can you rebase? |
|
@@ -249,6 +249,11 @@ describe('/tests/', () => { | |||
assert.equal(res.status, 200); | |||
}); | |||
|
|||
it('get all tests, ignore CSS', async () => { | |||
const res = await agent.get('/tests/?ignore=css'); | |||
assert.equal(res.status, 200); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should something be asserted here that's different from the previous step? As it is, this test would pass even if the ignore
parameter is unused.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We should probably also check to make sure we're loading the correct tests as well in the previous two steps. I'm not sure how to do it at this time though, so I've added some TODO comments to remind us later.
This PR updates the unittests to add more coverage, as well as properly ignore blocks of code that are too challenging/impossible/not worthwhile to test.