Few small test fixes #30

Merged
merged 2 commits into from Sep 16, 2012

Conversation

Projects
None yet
3 participants
Contributor

pkrefta commented Jun 28, 2012

Hi guys,

I tried Filatiron and it turned out that two tests didn't pass - both required to add index.js because I've encoutered "Cannot find module" message. I've also removed console.log from one of the because it waa whole request object into console.

Contributor

jfhbrook commented Jun 28, 2012

Which tests weren't passing? I want to understand why we need these index.js files.

Contributor

pkrefta commented Jun 28, 2012

Sorry for alarm - I didn't noticed that tests have passed despite this two errors - vows sad 2 errored. These messages were generated by ecstatic-test.js when I was whole test suite with vows test/* -i - howewer running only this test doesn't raise any errors.

@pksunkara pksunkara added a commit that referenced this pull request Sep 16, 2012

@pksunkara pksunkara Merge pull request #30 from pkrefta/test_fixes
Few small test fixes
f771e13

@pksunkara pksunkara merged commit f771e13 into flatiron:master Sep 16, 2012

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