Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Test published applications #1084

Open
VMelnalksnis opened this issue Feb 11, 2024 · 2 comments · May be fixed by #1094
Open

Test published applications #1084

VMelnalksnis opened this issue Feb 11, 2024 · 2 comments · May be fixed by #1094
Labels
area:build Relating to deployment/releases type:enhancement New feature or request

Comments

@VMelnalksnis
Copy link
Owner

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.

@VMelnalksnis VMelnalksnis added type:enhancement New feature or request area:build Relating to deployment/releases labels Feb 11, 2024
This was referenced Feb 11, 2024
@VMelnalksnis
Copy link
Owner Author

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.

As for the desktop app, I'm not sure.

@VMelnalksnis
Copy link
Owner Author

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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area:build Relating to deployment/releases type:enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant