-
-
Notifications
You must be signed in to change notification settings - Fork 402
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
Testing DynamicMaps #498
Comments
Anything else we want here? We now have a fair number of unit tests. |
True but that isn't the same as testing dynamic maps in the notebook. I'm still not sure what we can do about this, or whether there is anything sane we can do about this... |
Not sure what "testing dynamic maps in the notebook" adds over unit tests or were you talking about testing the widget code with DynamicMaps? Otherwise they don't behave any differently in a notebook and a unit test so I don't see what notebook tests would add. |
Unless we mirror the code in the notebooks against the unit tests (painful!) we have no way of knowing whether the dynamicmaps in the notebooks are behaving properly. That said, I don't mind seeing this issue closed as 'wontfix' as I think the problem is too difficult to solve properly. |
This issue has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs. |
This includes testing the DynamicMap tutorial which needs reference data.
The general issue is that DynamicMaps tend to be empty, leaving nothing to test. And to generate data, you need a live server making the issue even more complicated...
The text was updated successfully, but these errors were encountered: