-
Notifications
You must be signed in to change notification settings - Fork 16
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
test suite and constructor setup #1
Conversation
store.on('update', function(object, options) {}) | ||
store.on('remove', function(object, options) {}) | ||
store.on('change', function(eventName, object, options) {}) | ||
store.on('sync', function(object) {}) |
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.
instead of sync, we now have
- pull
- push
and also
- connect
- disconnect
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 I also add the .isConnected()
method as store.isConnected()
?
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.
There's also no clear
event in pouchdb-hoodie-sync, correct?
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.
ah good point, not yet, but there should! I'll add it to the issue, you can add it to the README already
This PR adds the same test suite used in the other PouchDB-Hoodie plugins:
It adds a spec for the plugin constructor.
Adds initial constructor index.js for plugin.
It also contains an updated README to more accurately the API covered by this plugin.