Mocha: When using --delay, .only() no longer works

[wlingke]

When I run a test with --delay, and I specify only(), the entire test suite gets run, not just the only block.

[boneskull]

@wlingke can you please provide some example code

[wlingke]

run below with --delay will fail yield in test two being run.

var assert = require('assert')
setTimeout(function(){

    describe.only('Should only run this', function(){
        it('passing test', function(){
            assert(true)
        })
    });

    describe('Should not run this', function(){
        it('failing test', function(){
            assert(false)
        })
    });

    run()
}, 1000);

[trueter]

Can confirm.

[stale[bot]]

I am a bot that watches issues for inactivity.
This issue hasn't had any recent activity, and I'm labeling it stale. In 14 days, if there are no further comments or activity, I will close this issue.
Thanks for contributing to Mocha!

[JustinLivi]

Can confirm to keep this from closing.

[ScottFreeCode]

Yup, tried it out and this is still an issue.

Surprising, because only got redesigned and rewritten in version 3.

[ScottFreeCode]

Fix: move https://github.com/mochajs/mocha/blob/16ffca210267b06300c9c54f886a031ea6a9030d/lib/runner.js#L813-L816 to between https://github.com/mochajs/mocha/blob/16ffca210267b06300c9c54f886a031ea6a9030d/lib/runner.js#L824 and https://github.com/mochajs/mocha/blob/16ffca210267b06300c9c54f886a031ea6a9030d/lib/runner.js#L825

How to write tests for it though... probably will have to use the test/integration model with test/integration/fixtures...

[silviom]

Hi, @ScottFreeCode. I have created the following PR for the fix. Thanks for the instructions.
https://github.com/mochajs/mocha/pull/3215

[juergba]

This issue can be closed, see PR #3215