Skip to content

Commit

Permalink
Fixed slow browser tests. Closes #135
Browse files Browse the repository at this point in the history
  • Loading branch information
tj committed Dec 10, 2011
1 parent e1f0b1a commit b5ff18a
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 7 deletions.
6 changes: 2 additions & 4 deletions mocha.js
Original file line number Diff line number Diff line change
Expand Up @@ -737,7 +737,7 @@ exports.colors = {
, 'bright fail': 91
, 'bright yellow': 93
, 'pending': 36
, 'suite': '40'
, 'suite': 0
, 'error title': 0
, 'error message': 31
, 'error stack': 90
Expand Down Expand Up @@ -2618,9 +2618,7 @@ process.exit = function(status){};
process.stdout = {};
global = this;

process.nextTick = function(fn){
setTimeout(fn, 0);
};
process.nextTick = function(fn){ fn(); };

process.removeListener = function(ev){
if ('uncaughtException' == ev) {
Expand Down
4 changes: 1 addition & 3 deletions support/tail.js
Original file line number Diff line number Diff line change
Expand Up @@ -13,9 +13,7 @@ process.exit = function(status){};
process.stdout = {};
global = this;

process.nextTick = function(fn){
setTimeout(fn, 0);
};
process.nextTick = function(fn){ fn(); };

process.removeListener = function(ev){
if ('uncaughtException' == ev) {
Expand Down

3 comments on commit b5ff18a

@TooTallNate
Copy link
Contributor

Choose a reason for hiding this comment

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

Maybe you should use one of those setImmediate impls for this.
https://github.com/NobleJS/setImmediate

@rauchg
Copy link
Contributor

@rauchg rauchg commented on b5ff18a Dec 10, 2011

Choose a reason for hiding this comment

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

+1

@tj
Copy link
Contributor Author

@tj tj commented on b5ff18a Dec 10, 2011

Choose a reason for hiding this comment

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

yeah im down, we could definitely do that, it's crazy how bad the resolution is for setTimeout(, 0)

Please sign in to comment.