Skip to content

Emit error on failed resubscribe #174

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.

Sign up for GitHub

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

Open
gkubisa wants to merge 5 commits into
base: master
Choose a base branch
from
Open

Emit error on failed resubscribe #174

Changes from 1 commit
Commits
Show all changes
5 commits
Select commit Hold shift + click to select a range
ee59fa8
Add .editorconfig
gkubisa Oct 17, 2017
d130678
Emit error on failed resubscribe
gkubisa Oct 17, 2017
7f3229f
Add semicolons
gkubisa Jul 12, 2018
e97489c
Add another semicolon
gkubisa Jul 12, 2018
2e43940
Update tested nodejs versions
gkubisa Jul 12, 2018
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Add another semicolon
  • Loading branch information
@gkubisa
gkubisa committed Jul 12, 2018
commit e97489cd7143552994d5d3c822e6e0e27b50e8cf
2 changes: 1 addition & 1 deletion test/client/subscribe.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -246,7 +246,7 @@ describe('client subscribe', function() {
var doc2 = this.backend.connect().get('dogs', 'fido');
doc.create({age: 3}, function(err) {
if (err) return done(err);
doc2[method]()
doc2[method]();
doc2.on('error', function(err) {
expect(err.message).equal('Reject doc read');
expect(doc2.version).eql(null);
Expand Down