PowerApps-Tooling icon indicating copy to clipboard operation
PowerApps-Tooling copied to clipboard
verified publisher icon microsoft

Tests should be run from the solution file

Open Youssef1313 opened this issue 1 year ago • 1 comments

why not running tests directly from the main .sln file?

Originally posted by @mizrael in https://github.com/microsoft/PowerApps-Tooling/issues/530#issuecomment-1914818728

Youssef1313 avatar Nov 28 '24 04:11 Youssef1313

@mizrael Your point is totally valid. CI shouldn't list the test projects and run on them one by one. A couple of reasons:

  1. When a new test project is added, you may forget to update the CI to run it. This is not hypothetical, it already happened and currently YamlValidator.Tests are not run in CI.
  2. If you (in the future) properly configured code coverage, you will want a single report to get the right/accurate results. Maybe two test projects are using the same production assembly and one of them is covering some part while the other doesn't. You don't want to see the results of only one of the test projects that would say some part isn't covered.

cc @Evangelink

Youssef1313 avatar Nov 28 '24 04:11 Youssef1313