[api] [test] fix streaming logs request along with tests.

nodejitsu · Apr 19, 2012 · 444ac71 · 444ac71
1 parent 56f472d
commit 444ac71
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 5 deletions.
diff --git a/node.js/lib/client/logs.js b/node.js/lib/client/logs.js
@@ -86,7 +86,7 @@ Logs.prototype.streamByApp = function (appName, stream) {
   var appName = defaultUser.call(this, appName),
       argv = ['logs'].concat(appName.split('/'), 'stream');
 
-  return this.stream('POST', argv, {}, stream);
+  return this.stream('GET', argv, null, stream);
 };
 
 //
@@ -98,5 +98,5 @@ Logs.prototype.streamByApp = function (appName, stream) {
 Logs.prototype.streamByUser = function (username, stream) {
   username = username || this.options.get('username');
 
-  return this.stream('POST', ['logs', username, 'stream'], {}, stream);
+  return this.stream('GET', ['logs', username, 'stream'], null, stream);
 };
diff --git a/node.js/test/commands/logs-test.js b/node.js/test/commands/logs-test.js
@@ -29,8 +29,12 @@ vows.describe('logs').addBatch(makeApiCall(
   'logs streamByUser myUser',
   function setup () {
     nock('http://api.mockjitsu.com')
-      .post('/logs/myUser/stream', {})
-      .reply(200, 'here are some logs\nenjoy them.',
-                  { 'x-powered-by': 'Nodejitsu' })
+      .get('/logs/myUser/stream')
+      .reply(200, {
+          app: 'example-app',
+          user: 'myUser',
+          data: 'Testing my app.1334857998687\n',
+          name: 'stdout'
+      }, { 'x-powered-by': 'Nodejitsu' })
   }
 )).export(module);