-
-
Notifications
You must be signed in to change notification settings - Fork 31
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
Nested and standard component folder structure test #191
Conversation
pip install -e git+https://github.com/plotly/dash.git#egg=dash[testing] | ||
git clone --depth 1 https://github.com/plotly/dash.git dash-main | ||
cd dash-main && pip install -e .[dev,testing] --progress-bar off && cd .. | ||
cd dash-main/\@plotly/dash-generator-test-component-nested && npm ci && npm run build && sudo R CMD INSTALL . && cd ../../.. |
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.
Using git clone
and local pip install as we now also need to build the test components
dashr.start_server(app) | ||
|
||
assert dashr.wait_for_element("#standard").text == "Standard" | ||
assert dashr.wait_for_element("#nested").text == "Nested" |
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 test as in Python: Both the nested and standard components load as expected.
LGTM! 💃 |
* Add unit tests for index customization (#176) * Send status code of 1 when unit tests fail (#177) * Support for arbitrary file extensions (#186) * Add assortment of unit tests to Dash for R (#179) * Nested and standard components test (#191) * Support inline clientside callbacks in Dash for R (#140) * Speed up CI, use image from plotly * Update to dash-renderer 1.4.1 Co-authored-by: HammadTheOne <hammadkhan@plotly.com> Co-authored-by: Marc-André Rivet <marc-andre.rivet@plotly.com>
Companion to plotly/dash#1203, related to original community PR plotly/dash#1125.