Skip to content

Commit 7598a8a

Browse files
committed
fix(socket.io): align [SNTA] expected event-list with Plan 66 accessUpdated
Plan 66 added the `accessUpdated` socket.io event server-side (open-pryv.io commit c13a773 et al.); the lib's allowed-events list was updated to include it, but the corresponding test fixture in components/pryv-socket.io/test/socket.io.test.js:138 still asserted the pre-Plan-66 list (eventsChanged,streamsChanged,accessesChanged,disconnect,error). CI has been red on master since the Plan 66 monitor+socket.io 3.1.0 bump landed (25787864547) and again on the Plan 68 3.2.0 release (25914436096). Update the expected string to match.
1 parent a8e2d8f commit 7598a8a

1 file changed

Lines changed: 1 addition & 1 deletion

File tree

components/pryv-socket.io/test/socket.io.test.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -135,7 +135,7 @@ describe('[SKIX] Socket.IO', function () {
135135
try {
136136
conn.socket.on('Bogus', () => {});
137137
} catch (e) {
138-
return expect(e.message).to.equal('Unkown event [Bogus]. Allowed events are: eventsChanged,streamsChanged,accessesChanged,disconnect,error');
138+
return expect(e.message).to.equal('Unkown event [Bogus]. Allowed events are: eventsChanged,streamsChanged,accessesChanged,accessUpdated,disconnect,error');
139139
}
140140
throw new Error('Should fail');
141141
});

0 commit comments

Comments
 (0)