Skip to content
This repository has been archived by the owner on Dec 28, 2022. It is now read-only.

Add more server-side tests to increase coverage #35

Merged
merged 8 commits into from
Oct 12, 2016

Conversation

Gum-Joe
Copy link
Owner

@Gum-Joe Gum-Joe commented Aug 31, 2016

This is the PR for #33.

Coverage:

Coverage Status

✅ Added tests for GET /api/session/user when not logged
in and GET /api/info/version
🎨 Removed unused method Api.prototype.run()
@coveralls
Copy link

Coverage Status

Coverage increased (+0.7%) to 71.977% when pulling 3d80fa7 on v1-coverage-improvements into b03b954 on master.

✅ Added tests for app/util/schema.js
✅ Added test for api.addPlugin()
✅ Changed request timeout in test/auth.js
@coveralls
Copy link

coveralls commented Sep 1, 2016

Coverage Status

Coverage increased (+1.7%) to 72.937% when pulling 9c141d9 on v1-coverage-improvements into b03b954 on master.

✅ Added additional socket.io tests for
sockets.defaultListener();
@coveralls
Copy link

coveralls commented Sep 3, 2016

Coverage Status

Coverage increased (+2.8%) to 74.088% when pulling cb37c48 on v1-coverage-improvements into b03b954 on master.

✅ Ignore help() (app/cli/index.js)
@coveralls
Copy link

coveralls commented Sep 22, 2016

Coverage Status

Coverage increased (+7.6%) to 78.887% when pulling 077659b on v1-coverage-improvements into b03b954 on master.

✅ Added tests for cli.parse()
@coveralls
Copy link

coveralls commented Sep 26, 2016

Coverage Status

Coverage increased (+10.5%) to 81.766% when pulling 167c2a1 on v1-coverage-improvements into b03b954 on master.

✅ Start adding tests for helpers: added test for error
handler for 404
@coveralls
Copy link

coveralls commented Sep 28, 2016

Coverage Status

Coverage increased (+11.08%) to 82.342% when pulling c1f5f2c on v1-coverage-improvements into b03b954 on master.

✅ Added more tests for error handler
🚑 Fixed issue where error handler was not used
💄 Changed formatting of views/error.ejs
@coveralls
Copy link

coveralls commented Sep 29, 2016

Coverage Status

Coverage increased (+13.9%) to 85.135% when pulling 5feb5d0 on v1-coverage-improvements into b03b954 on master.

✅ Ignore some untestable code
@coveralls
Copy link

coveralls commented Sep 29, 2016

Coverage Status

Coverage increased (+14.2%) to 85.493% when pulling f6a7c8e on v1-coverage-improvements into b03b954 on master.

@Gum-Joe
Copy link
Owner Author

Gum-Joe commented Oct 12, 2016

We're going to leave it here for now. Merging.

@Gum-Joe Gum-Joe merged commit b76ad30 into master Oct 12, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants