New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Display highscores #4
Conversation
|
||
app.listen(port, () => console.log(`Example app listening on port ${port}!`)) | ||
|
||
module.exports = app.listen(port) |
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.
This is needed so testing is easier
it("should fail when not provided a username", (done) => { | ||
chai.request(app) | ||
.get('/') | ||
.end((err, res) => { | ||
res.should.have.status(404) | ||
done() | ||
}) | ||
}) |
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.
This will most likely need to be changed in the future.
Once we support no entering a username, this test will need to be updated
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.
I didn't see this right away, would have saved me some troubles. Whoops.
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.
Wow, this is incredible!
Functional changes:
-displays api output
Testing:
-added tests around the api call
-manually tested and saw the output as expected
Readme:
-updated readme explaining how to build/run this app