Remove tools folder from coverate report

[rawagner]

No description provided.

[coveralls]

Pull Request Test Coverage Report for Build 287

• 0 of 0 changed or added relevant lines in 0 files are covered.
• No unchanged relevant lines lost coverage.
• Overall coverage increased (+2.7%) to 63.627%

---

Totals
Change from base Build 283:	2.7%
Covered Lines:	441
Relevant Lines:	621

---

💛 - Coveralls

[coveralls]

Pull Request Test Coverage Report for Build 367

• 0 of 0 changed or added relevant lines in 0 files are covered.
• No unchanged relevant lines lost coverage.
• Overall coverage increased (+2.7%) to 63.627%

---

Totals
Change from base Build 283:	2.7%
Covered Lines:	441
Relevant Lines:	621

---

💛 - Coveralls

[atiratree]

Even though we have tests in tools I agree that we do not have to collect coverage on that.

[vojtechszocs]

I've just found this out. What's the reasoning behind this change? Tools are used to validate the project during the build, why aren't their tests important enough? 😕

[rawagner]

I wanted to collect coverage from the bits that we are actually shipping, but you are right, this is being used during the build so we should re-enable coverage report from tools dir.

[atiratree]

@vojtechszocs the tests are important (their coverage is also important).
This is just a scope issue. If we want our coverage to include just the application or the application and the tools.

The best solution IMO is to have two coverage reports (one special for tools).

I don't mind having the coverage or not having the coverage of the tools. After all, it is up to the people who look at these reports and what they want to see.