Remove the filter excluding quality.cpp from the tests

[costamag]

The quality.cpp file was previously excluded from the test workflow, which prevented the CI from detecting issues introduced in earlier PRs ( #668 ). This PR ensures that quality.cpp is properly included, allowing the workflow to catch potential errors that would otherwise only be noticed during local compilation.

It would be good to hear from previous contributors if there is a good reason why quality.cpp should be excluded from the workflow run ( @lee30sonia, @aletempiac )

[costamag]

Hi @changmg,
@lee30sonia mentioned offline that the filter was introduced to speed up the test run. If we decide to restore the filter, I'm happy to work on increasing the test coverage so that it still catches the issue of the PR #668.

[aletempiac]

Hi @costamag,
We can try restoring it again and check the runtime increase in running the tests. It should be affordable. It is definitely a good idea to have quality checks on commits, especially when getting PRs from outside the lsils group.

[costamag]

Hi @aletempiac,
Thank you for your feedback! Sounds good. Let’s go ahead and restore them for now.