Skip to content

test: add initial GUI test#94

Merged
jcharkow merged 14 commits intoOpenMS:mainfrom
jcharkow:add_tests
Mar 3, 2025
Merged

test: add initial GUI test#94
jcharkow merged 14 commits intoOpenMS:mainfrom
jcharkow:add_tests

Conversation

@jcharkow
Copy link
Contributor

@jcharkow jcharkow commented Nov 13, 2024

Add GUI tests - See notes below

just test for launching each page and ensuring there is not an
exception. Currenly having trouble with quickstart.

Also add documentation specific tests
fix tests for view_ms_raw_data and file_upload
@jcharkow
Copy link
Contributor Author

This is now ready for review. It seems that the ci.yml test is only shown on my personal branch but all the tests seem to pass. It is not extensive testing as extensive testing with a GUI is difficult but it is something.

@jcharkow jcharkow marked this pull request as ready for review December 11, 2024 13:34
@jcharkow
Copy link
Contributor Author

TODO: Extend tests to TOPP tools

note currently spectrum visualization is not tested because cannot find
a way to select a spectrum to be displayed.
@jcharkow
Copy link
Contributor Author

Notes:

  1. Double check I did not mess up captcha with merge conflicts
  2. Testing framework is only limited to the pyopenms table for now. Plan to add TOPP tests in the future
  3. Testing framework in streamlit is not as extensive as main streamlit app. Difficult to test everything programmatically.

bugs in testing framework caused tests to fail, fix this
@jcharkow
Copy link
Contributor Author

note there is also a test.py file with tests Axel added.

@jcharkow jcharkow requested a review from t0mdavid-m January 22, 2025 23:23
@jcharkow jcharkow linked an issue Jan 22, 2025 that may be closed by this pull request
@jcharkow
Copy link
Contributor Author

Move test.py and test_gui.py to same folder

@jcharkow jcharkow merged commit c17cc66 into OpenMS:main Mar 3, 2025
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

WebApps: CI Testing

1 participant