-
-
Notifications
You must be signed in to change notification settings - Fork 195
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.
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
Add tests ideogram #182
Add tests ideogram #182
Conversation
8e624f4
to
36e21a8
Compare
@@ -500,6 +499,18 @@ Ideogram.propTypes = { | |||
}), | |||
}), | |||
|
|||
/** | |||
* Use perspective: 'comparative' to enable annotations between two chromosomes, | |||
* either within the same organism or different organisms. Used for homology. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
* either within the same organism or different organisms. Used for homology. | |
* either within the same organism or from different organisms. Used for homology. |
tests/test_ideogram.py
Outdated
'dash-bio/{}'.format(APP_NAME) | ||
|
||
|
||
# below, write tests for initial conditions; they will most likely |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I would take out this boilerplate.
tests/test_ideogram.py
Outdated
# verify the existence of the brush | ||
brush = selenium.find_elements_by_class_name('brush') | ||
assert len(brush) == 1 | ||
# selection = selenium.find_elements_by_class_name('selection')[0] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This test currently fails (issue #186), it has been commented out for the time being
|
||
brush = selenium.find_elements_by_class_name('brush') | ||
assert len(brush) == 1 | ||
# verify that the selection of the brush was updated |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Same as above (see #186)
Let's merge #187 into |
Remove deprecated selenium uses
|
||
/** | ||
* Use this prop in a dash callback to return annotationData when hovered. | ||
* It is read-only, i.e. it cannot be used with dash.dependencies.Output but only with |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good idea specifying this -- it will avoid a lot of confusion down the line! :)
Co-Authored-By: Bachibouzouk <pierre-francois.duc@netplus.ch>
Fullfills part of #103
Props covered by the test
About
Description of changes
Before merging