YahooArchive / arrow

FE Test framework designed to promote TDD
http://yahoo.github.io/arrow/arrow_intro.html
BSD 3-Clause "New" or "Revised" License
55 stars 59 forks source link

Add Test Engine to support BDD style test #71

Closed wangjungithub closed 11 years ago

mridgway commented 11 years ago

The engineConfig path looks like it's working now, but I am now getting the following error in my tests that use nodejs driver:

/Users/mridgway/Projects/salcap/test/unit/public/js/data-adapters.js:7
describe('dataAdapters', function () {
^
ReferenceError: describe is not defined
    at Object.<anonymous> (/Users/mridgway/Projects/salcap/test/unit/public/js/data-adapters.js:7:1)
    at Module._compile (module.js:449:26)
    at Object.Module._extensions..js (module.js:467:10)
    at Module.load (module.js:356:32)
    at Function.Module._load (module.js:312:12)
    at Module.require (module.js:362:17)
    at require (module.js:378:17)
    at Mocha.loadFiles (/Users/mridgway/Projects/arrow/node_modules/mocha/lib/mocha.js:137:27)
    at Array.forEach (native)
    at Mocha.loadFiles (/Users/mridgway/Projects/arrow/node_modules/mocha/lib/mocha.js:134:14)
[2013-04-19 08:10:21.426] [ERROR] NodeDriver - Failed to get the test report
[2013-04-19 08:10:21.428] [ERROR] Arrow - Failed to get the test report, Controller: /Users/mridgway/Projects/arrow/lib/controller/default
liangji101 commented 11 years ago

Hi Michael ,are you using https://git.corp.yahoo.com/sports/salcap/blob/arrowMocha/test/unit/mocha-config.json for mocha's config ? Default we use ui : "bdd" in mocha, if you use "tdd", then there should be no "describe" method...

mridgway commented 11 years ago

Yep, that was it. Not sure how I did that... Thanks!

proverma commented 11 years ago

All these changes are part of pull request #74 , closing this PR.