Merge pull request #2006 from plotly/fix-ci

Fix ci failures

Author: alexcjohnson

.circleci/config.yml

@@ -294,7 +294,7 @@ jobs:
             . venv/bin/activate && rm -rf components/dash-core-components/dash_core_components
             cd components/dash-core-components
             TESTFILES=$(circleci tests glob "tests/integration/**/test_*.py" | circleci tests split --split-by=timings)
-            pytest --headless --nopercyfinalize --junitxml=test-reports/junit_intg.xml --junitprefix="components.dash-core-components" ${TESTFILES}
+            pytest --headless --nopercyfinalize --junitxml=test-reports/junit_intg.xml --junitprefix="components.dash-core-components" ${TESTFILES} --reruns 3
       - store_artifacts:
           path: ~/dash/components/dash-core-components/test-reports
       - store_test_results:

components/dash-core-components/package.json

@@ -21,7 +21,7 @@
     "private::lint.prettier": "prettier --config .prettierrc src/**/*.js --list-different",
     "prepublishOnly": "rm -rf lib && babel src --out-dir lib --copy-files && rm -rf lib/jl/ lib/*.jl",
     "test": "run-s -c lint test:intg test:pyimport",
-    "test:intg": "pytest --nopercyfinalize --headless tests/integration",
+    "test:intg": "pytest --nopercyfinalize --headless tests/integration --reruns 3",
     "test:pyimport": "python -m unittest tests/test_dash_import.py",
     "prebuild:js": "cp node_modules/plotly.js-dist-min/plotly.min.js dash_core_components_base/plotly.min.js && cp node_modules/mathjax/es5/tex-svg.js dash_core_components_base/mathjax.js",
     "build:js": "webpack --mode production",

components/dash-core-components/tests/integration/calendar/test_date_picker_single.py

@@ -1,6 +1,8 @@
 from datetime import datetime, timedelta
 
 import pytest
+import werkzeug
+
 from dash import Dash, Input, Output, html, dcc, no_update
 
 
@@ -71,6 +73,11 @@ def test_dtps010_local_and_session_persistence(dash_dcc):
     assert dash_dcc.get_logs() == []
 
 
+@pytest.mark.xfail(
+    condition=werkzeug.__version__ in ("2.1.0", "2.1.1"),
+    reason="Bug with 204 and Transfer-Encoding",
+    strict=False,
+)
 def test_dtps011_memory_persistence(dash_dcc):
     app = Dash(__name__)
     app.layout = html.Div(

components/dash-core-components/tests/integration/dropdown/test_dynamic_options.py

@@ -24,8 +24,7 @@ def update_options(search_value):
     dash_dcc.start_server(app)
 
     # Get the inner input used for search value.
-    dropdown = dash_dcc.find_element("#my-dynamic-dropdown")
-    input_ = dropdown.find_element_by_css_selector("input")
+    input_ = dash_dcc.find_element("#my-dynamic-dropdown input")
 
     # Focus on the input to open the options menu
     input_.send_keys("x")
@@ -36,15 +35,15 @@ def update_options(search_value):
     input_.clear()
     input_.send_keys("o")
 
-    options = dropdown.find_elements_by_css_selector(".VirtualizedSelectOption")
+    options = dash_dcc.find_elements("#my-dynamic-dropdown .VirtualizedSelectOption")
 
     # Should show all options.
     assert len(options) == 3
 
     # Searching for `on`
     input_.send_keys("n")
 
-    options = dropdown.find_elements_by_css_selector(".VirtualizedSelectOption")
+    options = dash_dcc.find_elements("#my-dynamic-dropdown .VirtualizedSelectOption")
 
     assert len(options) == 1
     print(options)

components/dash-core-components/tests/integration/graph/test_graph_varia.py

@@ -2,6 +2,9 @@
 import pytest
 import time
 import json
+
+import werkzeug
+
 from dash import Dash, Input, Output, State, dcc, html
 from dash.exceptions import PreventUpdate
 from selenium.webdriver.common.by import By
@@ -128,7 +131,7 @@ def show_relayout_data(data):
 
     # use this opportunity to test restyleData, since there are multiple
     # traces on this graph
-    legendToggle = dash_dcc.driver.find_element_by_css_selector(
+    legendToggle = dash_dcc.find_element(
         "#example-graph .traces:first-child .legendtoggle"
     )
     legendToggle.click()
@@ -142,7 +145,7 @@ def show_relayout_data(data):
     )
 
     # and test relayoutData while we're at it
-    autoscale = dash_dcc.driver.find_element_by_css_selector("#example-graph .ewdrag")
+    autoscale = dash_dcc.find_element("#example-graph .ewdrag")
     autoscale.click()
     autoscale.click()
     dash_dcc.wait_for_text_to_equal("#relayout-data", '{"xaxis.autorange": true}')
@@ -185,6 +188,10 @@ def render_content(click, prev_graph):
     assert dash_dcc.get_logs() == []
 
 
+@pytest.mark.skipif(
+    werkzeug.__version__ in ("2.1.0", "2.1.1"),
+    reason="Bug with no_update 204 responses get Transfer-Encoding header.",
+)
 @pytest.mark.parametrize("is_eager", [True, False])
 def test_grva004_graph_prepend_trace(dash_dcc, is_eager):
     app = Dash(__name__, eager_loading=is_eager)
@@ -357,6 +364,10 @@ def display_data(trigger, fig):
     assert dash_dcc.get_logs() == []
 
 
+@pytest.mark.skipif(
+    werkzeug.__version__ in ("2.1.0", "2.1.1"),
+    reason="Bug with no_update 204 responses get Transfer-Encoding header.",
+)
 @pytest.mark.parametrize("is_eager", [True, False])
 def test_grva005_graph_extend_trace(dash_dcc, is_eager):
     app = Dash(__name__, eager_loading=is_eager)

components/dash-core-components/tests/integration/link/test_absolute_path.py

@@ -49,7 +49,7 @@ def extras(t):
             dcc.Link(
                 children="Absolute Path",
                 id="link1",
-                href=dash_dcc.server.url + "/extra/eseehc",
+                href="/extra/eseehc",
                 refresh=True,
             ),
             dcc.Location(id="url", refresh=False),

components/dash-core-components/tests/integration/link/test_link_event.py

@@ -93,7 +93,9 @@ def display_page(pathname):
     dash_dcc.wait_for_text_to_equal("#page-content", "You are on page /test-link")
 
     wait.until(
-        lambda: test_link.get_attribute("href") == "http://localhost:8050/test-link", 3
+        lambda: test_link.get_attribute("href")
+        == "http://localhost:{}/test-link".format(dash_dcc.server.port),
+        3,
     )
     wait.until(lambda: call_count.value == 2, 3)
 

components/dash-core-components/tests/integration/location/test_location_callback.py

@@ -93,7 +93,9 @@ def update_pathname(n_clicks, current_pathname):
     # Check that link updates pathname
     dash_dcc.find_element("#test-link").click()
     until(
-        lambda: dash_dcc.driver.current_url.replace("http://localhost:8050", "")
+        lambda: dash_dcc.driver.current_url.replace(
+            "http://localhost:{}".format(dash_dcc.server.port), ""
+        )
         == "/test/pathname",
         3,
     )

components/dash-core-components/tests/integration/store/test_data_lifecycle.py

@@ -1,6 +1,14 @@
+import pytest
+import werkzeug
+
 import dash.testing.wait as wait
 
 
+@pytest.mark.xfail(
+    condition=werkzeug.__version__ in ("2.1.0", "2.1.1"),
+    reason="Bug with 204 and Transfer-Encoding",
+    strict=False,
+)
 def test_stdl001_data_lifecycle_with_different_condition(store_app, dash_dcc):
     dash_dcc.start_server(store_app)
 

components/dash-core-components/tests/integration/tab/test_tabs_with_graphs.py

@@ -1,3 +1,5 @@
+import werkzeug
+
 from dash import Dash, Input, Output, dcc, html
 from dash.exceptions import PreventUpdate
 import json
@@ -119,6 +121,11 @@ def render_content(tab):
     assert dash_dcc.get_logs() == []
 
 
+@pytest.mark.xfail(
+    condition=werkzeug.__version__ in ("2.1.0", "2.1.1"),
+    reason="Bug with 204 and Transfer-Encoding",
+    strict=False,
+)
 @pytest.mark.parametrize("is_eager", [True, False])
 def test_tabs_render_without_selected(dash_dcc, is_eager):
     app = Dash(__name__, eager_loading=is_eager)