Skip to content

Commit

Permalink
Don't set the interface in Mocha test harness
Browse files Browse the repository at this point in the history
For some reason this was causing the tests to not run. Either way, BDD
is the default in the current version of Mocha, so it wasn't needed.
  • Loading branch information
Trevor Cook committed Aug 7, 2013
1 parent df899aa commit 14ff2c5
Showing 1 changed file with 0 additions and 3 deletions.
3 changes: 0 additions & 3 deletions test/mocha/test-runner.js
Original file line number Diff line number Diff line change
Expand Up @@ -33,9 +33,6 @@ function(config, mocha) {
// Compatible libraries: http://visionmedia.github.io/mocha/#assertions
window.expect = require("chai").expect;

// Prefer the BDD testing style.
mocha.setup("bdd");

require(["specs"], function(specs) {
// Load all specs.
require(specs.specs, function() {
Expand Down

2 comments on commit 14ff2c5

@tdcook
Copy link
Contributor

@tdcook tdcook commented on 14ff2c5 Aug 8, 2013

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks like this also breaks the in-browser test, but leaving it in breaks the command line.

@tbranyen
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think I got it in 101ae48

Please sign in to comment.