Skip to content
This repository was archived by the owner on Jun 5, 2020. It is now read-only.

Comments

fix last test failure in buster-dev-tools/runtests#15

Merged
cjohansen merged 3 commits intobusterjs:masterfrom
meisl:master
Jan 3, 2013
Merged

fix last test failure in buster-dev-tools/runtests#15
cjohansen merged 3 commits intobusterjs:masterfrom
meisl:master

Conversation

@meisl
Copy link
Member

@meisl meisl commented Jan 3, 2013

We were comparing a string with a buffer. That's due to an API change in fs.readFileSync, the key thing in its doc being

If encoding is specified then this function returns a string. Otherwise it returns a buffer.

See also: thread on dev mailing list

meisl added 3 commits January 3, 2013 15:09
… when it's just not there

"catch any" is particularly annoying in case of errors *within* local.js
…n node 0.6 to 0.8.16 we were getting a buffer instead of a string. That's why assert.equals failed.
cjohansen added a commit that referenced this pull request Jan 3, 2013
fix last test failure in buster-dev-tools/runtests
@cjohansen cjohansen merged commit bdd99d8 into busterjs:master Jan 3, 2013
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants