You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Is your feature request related to a problem? Please describe.
There have already been a few cases where issues are spotted only after release, because the published applications differ from the ones tested.
Describe the solution you'd like
Published applications are tested as well.
Describe alternatives you've considered
Additional context
Most likely will have to run all tests twice in CI - once as it is currently for code coverage, and once with published applications.
The text was updated successfully, but these errors were encountered:
One of the possible solutions was to publish the test projects, and the run tests from the published dll. However, it was not possible to reproduce the issues encountered in the last release so it would not be useful. I'm guessing the issue here is that the test project is referencing code from the applications, so it doesn't get trimmed in tests but would be trimmed when publishing.
For the API there is another solution - run the tests against the docker image. It will mean that either a new test project needs to be created, or the existing ones will need to be modified since they use the application DI container for setup/asserts.
I think that, at least for now, automated UI tests will be too much. However, instead of testing against the in-memory DesignTimeGnomeshadeClient, the desktop app tests could be run against a docker container built from the current commit. Need to try it out to know which approach would be the best - just testing the view models or moving to headless testing; testing the project as is, or testing a published .dll.
Is your feature request related to a problem? Please describe.
There have already been a few cases where issues are spotted only after release, because the published applications differ from the ones tested.
Describe the solution you'd like
Published applications are tested as well.
Describe alternatives you've considered
Additional context
Most likely will have to run all tests twice in CI - once as it is currently for code coverage, and once with published applications.
The text was updated successfully, but these errors were encountered: