winstonjs / node-loggly

A client implementation for Loggly in node.js
http://github.com/winstonjs/node-loggly
Other
233 stars 80 forks source link

Tests are broken #13

Closed mmalecki closed 12 years ago

mmalecki commented 12 years ago

Tests are currently broken, both on node v0.4.12 and 0.6.0.


♢ node-loggly/common

  When using the common module the clone() method
    ✓ should return a deep clone of the object

♢ node-loggly/devices

  When using the node-loggly client the getDevices() method
    ✓ should return a list of valid devices
  When using the node-loggly client the addDeviceToInput() method
    ✗ should respond with 200 status code
      » expected null, got {
    stack: 'Error: Loggly Error (400): Bad Request\n    at Request.callback (/Users/maciej/Programowanie/JavaScript/node-loggly/lib/loggly/common.js:130:25)\n    at Request.<anonymous> (/Users/maciej/Programowanie/JavaScript/node-loggly/node_modules/request/main.js:314:21)\n    at Request.emit (events.js:64:17)\n    at IncomingMessage.<anonymous> (/Users/maciej/Programowanie/JavaScript/node-loggly/node_modules/request/main.js:295:19)\n    at IncomingMessage.emit (events.js:81:20)\n    at HTTPParser.onMessageComplete (http.js:133:23)\n    at CleartextStream.ondata (http.js:1231:22)\n    at CleartextStream._push (tls.js:303:27)\n    at SecurePair.cycle (tls.js:577:20)\n    at EncryptedStream.write (tls.js:96:13)',
    arguments: undefined,
    type: undefined,
    message: 'Loggly Error (400): Bad Request'
} // common.js:130

♢ node-loggly/inputs

  When using the node-loggly client the getInputs() method
    ✓ should return a list of valid inputs
  When using the node-loggly client the getInput method when called with a json input
    ✓ should return a valid input
    ✗ of the format 'json'
      » expected 'json',
    got  'text' (==) // input-test.js:64
    ✗ that matches the second input in the test configuration
      » expected 'efe9d37e-a9f1-4e28-a30f-a489ab545d36',
    got  undefined (==) // input-test.js:67
  When using the node-loggly client the getInput method when called with a plaintext input
    ✓ should return a valid input
    ✓ of the format 'text'
    ✗ that matches the first input in the test configuration
      » expected 'f67ef8d2-9482-4aba-9a82-2c5740f5f18e',
    got  undefined (==) // input-test.js:49
  When using the node-loggly client the log() method to a 'text' input when passed a callback
    ✗ should log messages to loggly
      » expected null, got {
    stack: 'Error: Loggly Error (500): Internal Server Error\n    at Request.callback (/Users/maciej/Programowanie/JavaScript/node-loggly/lib/loggly/common.js:130:25)\n    at Request.<anonymous> (/Users/maciej/Programowanie/JavaScript/node-loggly/node_modules/request/main.js:314:21)\n    at Request.emit (events.js:64:17)\n    at IncomingMessage.<anonymous> (/Users/maciej/Programowanie/JavaScript/node-loggly/node_modules/request/main.js:295:19)\n    at IncomingMessage.emit (events.js:81:20)\n    at HTTPParser.onMessageComplete (http.js:133:23)\n    at CleartextStream.ondata (http.js:1231:22)\n    at CleartextStream._push (tls.js:303:27)\n    at SecurePair.cycle (tls.js:577:20)\n    at EncryptedStream.write (tls.js:96:13)',
    arguments: undefined,
    type: undefined,
    message: 'Loggly Error (500): Internal Server Error'
} // common.js:130
  When using the node-loggly client the log() method to a 'json' input when passed a callback
    ✗ should log messages to loggly
      » expected null, got {
    stack: 'Error: Loggly Error (500): Internal Server Error\n    at Request.callback (/Users/maciej/Programowanie/JavaScript/node-loggly/lib/loggly/common.js:130:25)\n    at Request.<anonymous> (/Users/maciej/Programowanie/JavaScript/node-loggly/node_modules/request/main.js:314:21)\n    at Request.emit (events.js:64:17)\n    at IncomingMessage.<anonymous> (/Users/maciej/Programowanie/JavaScript/node-loggly/node_modules/request/main.js:295:19)\n    at IncomingMessage.emit (events.js:81:20)\n    at HTTPParser.onMessageComplete (http.js:133:23)\n    at CleartextStream.ondata (http.js:1231:22)\n    at CleartextStream._push (tls.js:303:27)\n    at SecurePair.cycle (tls.js:577:20)\n    at EncryptedStream.write (tls.js:96:13)',
    arguments: undefined,
    type: undefined,
    message: 'Loggly Error (500): Internal Server Error'
} // common.js:130

✗ Errored » Asynchronous Error
    in node-loggly/inputs
    in test/input-test.js

✗ Errored » callback not fired
    in When using the node-loggly client the log() method to a 'text' input when not passed a callback
    in node-loggly/inputs
    in test/input-test.js
✗ Errored » callback not fired
    in When using the node-loggly client the log() method to a 'json' input when not passed a callback
    in node-loggly/inputs
    in test/input-test.js
✗ Errored » 6 honored ∙ 6 broken ∙ 1 errored ∙ 2 dropped
indexzero commented 12 years ago

This is fixed.