Remove qapplication from tests - Githubissues

TomographicImaging / CILViewer

A simple Viewer for 3D data built with VTK

Apache License 2.0

8 stars 6 forks source link

Remove qapplication from tests #394

Closed DanicaSTFC closed 1 month ago

DanicaSTFC commented 2 months ago

Describe your changes

removed the qapplication instance from the test

Describe any testing you have performed

Consider adding example code to examples

I run the pytests before and after. I can see a new window appearing on the screen which indicates new tests are being performed.

Link relevant issues

Closes #367

Checklist when you are ready to request a review

[x] I have performed a self-review of my code
[ ] I have added docstrings in line with the guidance in the CIL developer guide
[ ] I have implemented unit tests that cover any new or modified functionality
[x] CHANGELOG.md has been updated with any functionality change
[x] Request review from all relevant developers
[x] Change pull request label to 'waiting for review'

Contribution Notes

[x] The content of this Pull Request (the Contribution) is intentionally submitted for inclusion in CILViewer (the Work) under the terms and conditions of the Apache-2.0 License
[x] I confirm that the contribution does not violate any intellectual property rights of third parties

Qt contributions should follow Qt naming conventions i.e. camelCase method names.

VTK contributions should follow VTK naming conventions i.e. PascalCase method names.