Not overriden `message` property correctly. #3

Closed
dscape opened this Issue Mar 3, 2012 · 0 comments

Comments

Projects
None yet
1 participant
Contributor

dscape commented Mar 3, 2012

Found when a build failed on Travis in node 0.4.12

1 Using worker: nodejs1.worker.travis-ci.org:travis-nodejs-2
2
3$ cd ~/builds
4$ git clone --depth=100 --quiet git://github.com/flatiron/errs.git flatiron/errs
5$ cd flatiron/errs
6$ git checkout -qf bbc7f29058cda876a9000d75c788bebf04b163f0
7$ export TRAVIS_NODE_VERSION=0.4
8$ nvm use 0.4
9Now using node v0.4.12
10$ node --version
11v0.4.12
12$ npm --version
131.0.106
14$ npm install
15vows@0.6.2 ./node_modules/vows 
16└── eyes@0.1.7
17$ npm test
18
19> errs@0.1.2 test /home/vagrant/builds/flatiron/errs
20> vows test/*-test.js --spec
21
22 
23  ♢ errs 
24  
25  Using errs module the register() method
26    ✓ should register the prototype
27    ✓ should register an error without providing its name
28  Using errs module the create() method with a string
29    ✓ should create an error with the correct message
30  Using errs module the create() method with no parameters
31    ✓ should create an error with the correct message
32  Using errs module the create() method with an error
33    ✓ should return the error unmodified
34  Using errs module the create() method with a function
35    ✓ should create an error with the specified properties
36  Using errs module the handle() method with a callback
37    ✓ should invoke the callback with the error
38  Using errs module the handle() method with no callback
39    ✓ should invoke the callback with the error
40  Using errs module the create() method with an object that has no message
41    ✓ should create an error with the specified properties
42  Using errs module the create() method with an object that has a message
43    ✓ should create an error with the specified properties
44  Using errs module the create() method with a registered type that exists
45    ✓ should create an error of the correct type
46  Using errs module the create() method with a registered type that doesnt exist
47    ✓ should create an error of the correct type
48  Using errs module the unregister() method
49    ✓ should unregister the prototype
50  Using errs module the merge() method
51    ✓ should preserve custom properties
52    ✓ should have a stack trace
53    ✗ should preserve message specified in create 
54        » expected 'Override!', 
55      got  'Msg!' (==) // errs-test.js:107
56    ✓ should preserve properties specified
57 
58✗ Broken » 16 honored ∙ 1 broken (0.016s) 
59  
60npm ERR! errs@0.1.2 test: `vows test/*-test.js --spec`
61npm ERR! `sh "-c" "vows test/*-test.js --spec"` failed with 1
62npm ERR! 
63npm ERR! Failed at the errs@0.1.2 test script.
64npm ERR! This is most likely a problem with the errs package,
65npm ERR! not with npm itself.
66npm ERR! Tell the author that this fails on your system:
67npm ERR!     vows test/*-test.js --spec
68npm ERR! You can get their info via:
69npm ERR!     npm owner ls errs
70npm ERR! There is likely additional logging output above.
71npm ERR! 
72npm ERR! System Linux 2.6.38-8-generic
73npm ERR! command "node" "/home/vagrant/.nvm/v0.4.12/bin/npm" "test"
74npm ERR! cwd /home/vagrant/builds/flatiron/errs
75npm ERR! node -v v0.4.12
76npm ERR! npm -v 1.0.106
77npm ERR! code ELIFECYCLE
78npm ERR! 
79npm ERR! Additional logging details can be found in:
80npm ERR!     /home/vagrant/builds/flatiron/errs/npm-debug.log
81npm not ok
82
83Done. Build script exited with: 1

@dscape dscape closed this in 17d90b9 Mar 3, 2012

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment