Matthias Riße
Matthias Riße
These deprecation warnings in the CI runs come from dependencies which still use this deprecated method. There is not much we can do about this, apart from making sure they...
In `test_join_collaborator_to_operation` on line 127 a Permission object is created and then asserts are used to check that its attributes match what was used to create it. This is essentially...
This came up in #2326. There are some locations in which attributes of classes are duplicated in a way in which it is easy for them to become out-of-sync. For...
Qt widgets that are not cleaned up after a test can affect the following tests. This change makes it so that tests that do not clean up after themselves will...
Currently MSS' test suite uses Xvfb and pyvirtualdisplay to run tests invisibly. This is fine, except that it introduces somewhat unnecessary overhead through the X server and that X is...
We have quite a few typos/misspellings in MSS. We could introduce a spellchecker as an additional linter. [codespell](https://github.com/codespell-project/codespell) is one that I am aware of. This is what it finds...
Right now tests may leave widgets open. This can affect other tests. Not closing all widgets should instead be a test failure and any instance where this happens needs to...
This removes the docker image logic from the CI setup and instead replaces it with a setup based on pixi. As part of that it moves the dependency specification from...
https://github.com/Open-MSS/MSS/blob/af7352611868f1e29421fde8ff5d84cbc295f029/mslib/utils/mssautoplot.py#L335-L342 The else branch in line 341 is unreachable because the preceding elif condition is always true. That is because ```python "404 Client Error" or "NOT FOUND for url" in...
There are some code paths that are never used. One file that comes to mind, for which we have no coverage and that we decided against actually using: https://github.com/Open-MSS/MSS/blob/af7352611868f1e29421fde8ff5d84cbc295f029/mslib/msui/msui_web_browser.py There...