muter-mutation-testing/muter

Reconsider defaultExcludeList values to be more specific

Closed this issue · 3 comments

This is the code we currently have:

private var defaultExcludeList: [FilePath] {
        [
            ".swiftpm",
            ".build",
            "Build",
            "Carthage",
            "muter_tmp",
            "Pods",
            "Spec",
            "Test",
            "fastlane"
        ]
    }

The problem with the above code is that if I have a file named SpecialFeatureViewModel it will be excluded.

So I am proposing we change them like \Spec\ which will exclude every file that is inside Spec folder. And of course this will have to work for both linux and macos (we can see how we can achieve this).

What do you think? @rakaramos @ZevEisenberg

Makes sense to me. We will probably need to add a few more entries to the default list, like Tests.

@ZevEisenberg I've added Tests folder as you mentioned.
If you have some time please give me a review on #258

@Nikoloutsos closing this one as it was fixed in #258