neurobagel / digest

Web dashboard for neuroimaging-derived and phenotypic datasets
https://digest.neurobagel.org
MIT License
2 stars 2 forks source link

[MNT] Add `chromedrive-autoinstaller` for easier local testing #145

Closed alyssadai closed 4 months ago

alyssadai commented 4 months ago

Changes proposed in this pull request:

Checklist

This section is for the PR reviewer

For new features:

For bug fixes:

alyssadai commented 4 months ago

Thanks for the suggestion @surchs, I'll look into that a bit more.

Would definitely be nice to have an easier way to write Dash unit/e2e tests than Selenium, but I was discouraged by the fact that there's pretty much no Dash docs mentions or discussion posts around using other tools like Cypress (https://community.plotly.com/t/does-plotly-js-play-well-with-cypress-io/32919 and https://community.plotly.com/t/starting-work-on-cypress-e2e-testing-for-dash/34221 were not very promising...).

One thing I've found while writing some of the current tests is that there seem to be minor property discrepancies (?) between the Dash components and normal React components, such that when I've tried to use component locator syntax from non-Dash-specific examples of Selenium tests, they don't always work. ☚ī¸ Unsure how much of this is Dash-specific or just my lack of familiarity with frontend tests. That said, I should probably just try it out locally at some point and confirm.