Fix coverage reports configuration

[hectorgomezv]

Summary

Since the rootDir change in this PR, the coverage reports are taking into consideration all the folders in the root, instead of /src only. This PR restores this behavior by ignoring non-relevant folders regarding test coverage.

Changes

• Adds non-relevant test coverage folders to coveragePathIgnorePatterns configuration.

[coveralls]

Pull Request Test Coverage Report for Build 10684918840

Details

• 0 of 0 changed or added relevant lines in 0 files are covered.
• 10 unchanged lines in 1 file lost coverage.
• Overall coverage decreased (-0.005%) to 46.327%

---

Files with Coverage Reduction	New Missed Lines	%
src/domain/staking/staking.repository.ts	10	32.26%

Totals
Change from base Build 10679610744:	-0.005%
Covered Lines:	4811
Relevant Lines:	8363

---

💛 - Coveralls

[iamacook]

The coverage report still shows 46%. Do we first need to merge it?

[hectorgomezv]

The coverage report still shows 46%. Do we first need to merge it?

You're right, I was under the impression that including the src reference into the collectCoverageFrom property worked, but that's not the case. I reverted to a previous commit where I was manually ignoring some folders: c9e9e82

This is the change: 9de1100

It's less clear, but works. Another idea would be to just restrict the coverage analysis to .ts files (instead of both .ts and .js). Any preference?

[iamacook]

The coverage report still shows 46%. Do we first need to merge it?

You're right, I was under the impression that including the src reference into the collectCoverageFrom property worked, but that's not the case. I reverted to a previous commit where I was manually ignoring some folders: c9e9e82

This is the change: 9de1100

It's less clear, but works. Another idea would be to just restrict the coverage analysis to .ts files (instead of both .ts and .js). Any preference?

As it's not touched often, I don't think the clarity is a huge concern. I'm still not seeing an updated coverage report though 🙈

[hectorgomezv]

The coverage report still shows 46%. Do we first need to merge it?

You're right, I was under the impression that including the src reference into the collectCoverageFrom property worked, but that's not the case. I reverted to a previous commit where I was manually ignoring some folders: c9e9e82
This is the change: 9de1100
It's less clear, but works. Another idea would be to just restrict the coverage analysis to .ts files (instead of both .ts and .js). Any preference?

As it's not touched often, I don't think the clarity is a huge concern. I'm still not seeing an updated coverage report though 🙈

I think Coveralls is not updating the report for some reason, but I see in this branch only src files are taken into consideration: https://github.com/safe-global/safe-client-gateway/actions/runs/10684954356/job/29616598722

And in other branches also other folders (like dist) are being considered: https://github.com/safe-global/safe-client-gateway/actions/runs/10698244987/job/29657195866

(See the Run yarn run test:cov coverage table at the end of the console output).