Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
- Notifications
You must be signed in to change notification settings - Fork 140
Check if figure has 'data' member set#224
Uh oh!
There was an error while loading. Please reload this page.
Changes from all commits
9d79206ee41fedc63362f4c56d62368ecc7e8b33b1File filter
Filter by extension
Conversations
Uh oh!
There was an error while loading. Please reload this page.
Jump to
Uh oh!
There was an error while loading. Please reload this page.
Diff view
Diff view
There are no files selected for viewing
Large diffs are not rendered by default.
Uh oh!
There was an error while loading. Please reload this page.
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1 +1 @@ | ||
| __version__ = '0.23.0' | ||
| __version__ = '0.23.1' |
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -13,10 +13,15 @@ | ||
| import dash_core_components as dcc | ||
| import dash_table_experiments as dt | ||
| from selenium import webdriver | ||
| from selenium.webdriver.common.by import By | ||
| from selenium.webdriver.common.keys import Keys | ||
| from selenium.common.exceptions import InvalidElementStateException | ||
| import time | ||
| from textwrap import dedent | ||
| from selenium.webdriver.support import expected_conditions | ||
| from selenium.webdriver.support.wait import WebDriverWait | ||
| try: | ||
| from urlparse import urlparse | ||
| except ImportError: | ||
| @@ -528,3 +533,22 @@ def update_graph(n_clicks): | ||
| button.click() | ||
| time.sleep(2) | ||
| self.snapshot('candlestick - 2 click') | ||
| def test_graph_without_data(self): | ||
| app = dash.Dash(__name__) | ||
| app.layout = html.Div([ | ||
| dcc.Graph(id='graph', figure={ | ||
| }) | ||
| ]) | ||
| self.startServer(app=app) | ||
| graph_rendered = WebDriverWait(self.driver, 10).until( | ||
Member There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Have you tried bumping this up to like, 20? Contributor There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Yes, didn't change, should put a percy snapshot before the wait so we can see if the graph is actually there. | ||
| expected_conditions.presence_of_element_located((By.ID, 'graph')) | ||
| ) | ||
| self.assertTrue(graph_rendered) | ||
| self.snapshot('graph without figure.data') | ||
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.
Similarly here - let's use the full link to the issue #223