Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Warn if use test files included in files config section #35

Closed
marcalj opened this issue Mar 5, 2015 · 1 comment
Closed

Warn if use test files included in files config section #35

marcalj opened this issue Mar 5, 2015 · 1 comment

Comments

@marcalj
Copy link

marcalj commented Mar 5, 2015

I had a hard bug when running Wallaby with a config like this:

'use strict';

module.exports = function () {
    return {
        files: [
...
            'app/scripts/**/*.js',
            'test/mock/**/*.js',
            //'test/spec/**/*.js',
        ],
        tests: [
            'test/spec/*/*.js'
        ],
        testFramework: 'jasmine@2.1.3'
    };
};

I've just copy the karma.conf files section into wallaby's files config section, and I realized afterwards I was adding spec files in both files and tests config sections.

Maybe a warning or something would be fine to detect this errors.

PD: It was working fine, but the IDE shows spec files as source code, no tests warnings.

@ArtemGovorov
Copy link
Member

Good point, three different people had the same issue, I will have a look what can be done about it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants