File exclusion added, but still analyzed?

zaat · September 19, 2019, 8:53am

Hi,

In one of our projects we want to exclude all css files currently stored in a folder called FranceFiles. When we add **/FranceFiles/*.css to the exclusion, they still get included in the analysis. Any ideas?
Folder where the css resides is: Src/XXX.YYY/Tests/XXX.YYY.Common.Tests/Helpers/FranceFiles

janos · September 20, 2019, 11:23am

That exclusion pattern looks good to me, as a value for the sonar.exclusions scanner parameters.

Can you please clarify:

How do you scan your project?
How exactly do you specify the exclusion pattern?

zaat · September 20, 2019, 12:19pm

Hi @janos,
We use Azure DevOps (cloud version), and have the Sonarcloud extension installed. Build is done by MSBuild.
We have the following Coverage exclusion:

and the following file exclusion:

zaat · September 25, 2019, 8:16am

Hi @janos if you need more details, please indicate.

janos · September 25, 2019, 11:36am

There are two types of “file” exclusions, source and test. Which one is it, can you please clarify?

Also, what do you use to compute coverage in css files?

zaat · September 26, 2019, 3:18pm

Hi,

It is the source file exclusion that we use. Right now we are not yet looking into the coverage of css files.

zaat · October 3, 2019, 9:28am

Hi @janos, if you have any info on what we can best do, please indicate. Right now we set all issues to resolve as won’t fix, but would be better if we got the file exclusion to work.

janos · October 3, 2019, 9:52am

Hi @zaat, sorry about the late response. When you say the files still get included in the analysis, how do you mean exactly? What are you observing that makes you think this is the case? For example, do you see those files on the Code tab on SonarCloud? (For the record, I was not able to reproduce the issue, files seem to get correctly excluded with that kind of pattern, regardless of directory depth.)

zaat · October 3, 2019, 11:30am

Hi @janos,
I just setup an additional (new) project for the team that is experiencing the issue. With the same code, same exclusions, same issue persists. I can send you the details of the team/org/appl if you want to investigate?

zaat · October 4, 2019, 11:56am

Yep, I see the code in the code-tab

janos · October 11, 2019, 9:48am

Files under folders named Tests are considered test files, which is different from source files. To exclude these CSS files, please use the Test File Exclusions setting.

Let me know if you need more help.

zaat · October 11, 2019, 12:17pm

Hi @janos,

Thanks for checking. I tried, when I put **/FranceFiles/*.css in the test file exclusion it only works if I add the same location in the source file exclusion.
So only as source file exclusion, or only as test file exclusion does not work. Then these files are analyzed. Only if I add the exclusion on both (Source file and test file) they are excluded. Does seem like something to fix I think ?
Anyway, works for me now. Thanks for your support!