Rework testing architecture #41
Comments
agraebe
added a commit
that referenced
this issue
Feb 26, 2017
* adding bluebird missing tests * adding tests for async methods * #33 implementation including tests and updated README.md * adding error handling to samples * resolving conflict for README * update dependency versions * remove publisher and fix gulpfile * #24 first piece with tests to come: Reindeers & Requests implementation * solving #24, #40, #41, #42 - added all methods for #24 including tests - updated READMEs & example files - reworked entire testing architecture for #41 - removed deprecated methods for #42 - added scope checks for #40 * adding tests for #40 * sync updates from master * Added mechanism for token refresh (#47) * added token refresh functionality * added unit test for oauth.js * added unit test for oauthAsync.js * added tests for complete code coverage * code cleanup * Updating dependencies and set engines * Setting tokens to fix #39 * Added implicit surge handling (#49) * added functionality for surge pricing * added implicit surge handling * Changed OAuth2 urls to v2 (#53) * Adding tests for #53 * Dropping node v.1x/iojs in favor of ES6 #30
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Right now, the testing architecture is not suitable for more and advanced tests. For instance, nock responses are stored in variables in testing files. Those files should be stored separately.
Also, all nock instructions should be encapsulated to avoid duplicated calls (e.g. authorization is needed in almost every case).
The text was updated successfully, but these errors were encountered: