#75 adjust the .say() method to match the spec

StellarFw · Jun 27, 2017 · 51c1a97 · 51c1a97
1 parent 8334a3e
commit 51c1a97
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 2 deletions.
diff --git a/src/client.js b/src/client.js
@@ -564,7 +564,14 @@ StellarClient.prototype.off = function (event, func) { return this.removeListene
  * @return Promise
  */
 StellarClient.prototype.say = function (room, message) {
-  return this.send({ event: 'say', room, message })
+  // set default room as target, and set message with the first argument
+  if (message === undefined) {
+    message = room
+    room = this.options.defaultRoom
+  }
+
+  // emit a 'say' event for the selected room
+  return this.to(room).emit('message', message)
 }
 
 /**

diff --git a/test/servers/websocket.js b/test/servers/websocket.js
@@ -307,7 +307,7 @@ describe('Servers: Web Socket', function () {
       const listener = response => {
         client1.removeListener('say', listener)
         response.context.should.equal('user')
-        response.message.should.equal('hello from client 2')
+        response.message.data.should.equal('hello from client 2')
         done()
       }