Feature Request: Dash Unit tests

lina-usc / pylossless

🧠 EEG Processing pipeline that annotates continuous data

https://pylossless.readthedocs.io/en/latest/

MIT License

20 stars 8 forks source link

Feature Request: Dash Unit tests #46

Open scott-huberty opened 1 year ago

scott-huberty commented 1 year ago

Making this a separate issue bc Dash unit tests will require a lot of development.

We need to integrate some basic unit tests for our QC dashboard into our CI pipeline. Some basic things like testing that there is a layout.shape where we know there should be a bad_pylossless mne.annotation (using a test file).

This documenation is a good place to start: https://dash.plotly.com/testing

Adding this to the 0.01 milestone.

christian-oreilly commented 1 year ago

FIrst Dash test was integrated in PR #117. We need to clarify the specific deliverables or objectives that we want to meet before we can close this issue. @scott-huberty

scott-huberty commented 1 year ago

After discussion, here are the tests @christian-oreilly and I think should exist before closing this PR:

[ ] Test that a file loads on dropdown
[ ] Test clicking a trace to change its color (becomes gray).
[ ] Test creating an annotations
[ ] Test moving an annotation
[ ] Test deleting an annotation
[ ] Test saving a file