Skip to content

Improve interactions tests #38

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
xhluca opened this issue Mar 8, 2019 · 0 comments
Open

Improve interactions tests #38

xhluca opened this issue Mar 8, 2019 · 0 comments
Assignees
@kinimesi
Labels
good first issue Good for newcomers suggestion

Comments

@xhluca
Copy link

xhluca commented Mar 8, 2019

Future works for tests.test_interactions

  • make the nested functions (save_screenshot, perform_dragging, etc.) class methods of Test, and initialize the app in setUpClass (inherited from IntegrationTests).
  • break down the interactions tests into separate tests, one for dragging, one for clicking, one for hovering, etc.
  • programmatically set the renderedPositions of the nodes, rather than hardcoded.

Those suggestions do not currently affect the testing procedures, but shall we add supplementary tests or make substantial changes to the dash-cytoscape API, they could end up becoming useful. They might take substantial effort to implement.
@xhluca xhluca added the future work A feature to be added in future release. Community contribution welcome label Mar 8, 2019
@xhluca xhluca added suggestion and removed future work A feature to be added in future release. Community contribution welcome labels Jun 18, 2020
@xhluca xhluca added the good first issue Good for newcomers label Aug 5, 2020
kinimesi added a commit that referenced this issue Aug 18, 2021
@kinimesi
make the nested functions class methods of Test #38
8635417
kinimesi added a commit that referenced this issue Aug 19, 2021
@kinimesi
initialize the app in setUpClass #38
dae20a6
kinimesi added a commit that referenced this issue Aug 20, 2021
@kinimesi
break down the interactions tests into separate tests #38
f612bac
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@kinimesi kinimesi

Labels
good first issue Good for newcomers suggestion
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@kinimesi @xhluca and others