search

busterjs / buster

Abandoned - A powerful suite of automated test tools for JavaScript.
http://docs.busterjs.org
Other
448 stars 37 forks source link

buster-test-cli crash #367

Closed viking closed 11 years ago

viking commented 11 years ago

Getting this message almost every time I run buster test:

TypeError: Cannot call method 'send' of undefined
    at /usr/lib/node_modules/buster/node_modules/buster-test-cli/node_modules/ramp/node_modules/faye/node/faye-node.js:1856:45
    at Object.Faye.Deferrable.callback (/usr/lib/node_modules/buster/node_modules/buster-test-cli/node_modules/ramp/node_modules/faye/node/faye-node.js:285:23)
    at Object.Faye.Transport.WebSocket.Faye.extend.Faye.Class.request (/usr/lib/node_modules/buster/node_modules/buster-test-cli/node_modules/ramp/node_modules/faye/node/faye-node.js:1856:10)
    at Object.Faye.Transport.Faye.extend.Faye.Class.send (/usr/lib/node_modules/buster/node_modules/buster-test-cli/node_modules/ramp/node_modules/faye/node/faye-node.js:1073:37)
    at Object.<anonymous> (/usr/lib/node_modules/buster/node_modules/buster-test-cli/node_modules/ramp/node_modules/faye/node/faye-node.js:1012:23)
    at Object.Faye.Extensible.pipeThroughExtensions (/usr/lib/node_modules/buster/node_modules/buster-test-cli/node_modules/ramp/node_modules/faye/node/faye-node.js:497:44)
    at Object.Faye.Client.Faye.Class._send (/usr/lib/node_modules/buster/node_modules/buster-test-cli/node_modules/ramp/node_modules/faye/node/faye-node.js:1010:10)
    at Object.Faye.Client.Faye.Class.connect (/usr/lib/node_modules/buster/node_modules/buster-test-cli/node_modules/ramp/node_modules/faye/node/faye-node.js:818:10)
    at Object.<anonymous> (/usr/lib/node_modules/buster/node_modules/buster-test-cli/node_modules/ramp/node_modules/faye/node/faye-node.js:804:47)
    at Object.<anonymous> (/usr/lib/node_modules/buster/node_modules/buster-test-cli/node_modules/ramp/node_modules/faye/node/faye-node.js:780:32)
viking commented 11 years ago

Below is my buster configuration. I'm using the buster-amd extension.

var config = module.exports;

config["tests"] = {
  rootPath: "../",
  environment: "browser",
  libs: [
    "lib/require.js",
    "test/requirejs-config.js"
  ],
  sources: [
    "lib/maria.js",
    "src/js/**/*.js"
  ],
  tests: [
    "test/js/**/test_*.js"
  ],
  testHelpers: [
    "test/lib/helpers.js"
  ],
  extensions: [require("buster-amd")]
}
viking commented 11 years ago

This issue may be related: https://github.com/faye/faye/issues/233

augustl commented 11 years ago

Interesting, thanks for reporting. A rewrite of the "ramp" module is in progress, I'll make sure that I don't re-implement this issue :)

augustl commented 11 years ago

Fixed in the latest Faye versions, seems it's safe to close this one. The Faye code in ramp is also completely rewritten. Open a new issue if a similar error occurs.