Skip to content
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

Merged
merged 11 commits into from
Apr 22, 2020

Conversation

Marc-Andre-Rivet
Copy link
Contributor

Companion to plotly/dash#1203, related to original community PR plotly/dash#1125.

@Marc-Andre-Rivet Marc-Andre-Rivet marked this pull request as ready for review April 21, 2020 21:46
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 ../../..
Copy link
Contributor Author

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"
Copy link
Contributor Author

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.

@rpkyle
Copy link
Contributor

rpkyle commented Apr 22, 2020

LGTM! 💃

@Marc-Andre-Rivet Marc-Andre-Rivet merged commit 9f7d33e into dev Apr 22, 2020
@Marc-Andre-Rivet Marc-Andre-Rivet deleted the pr1125-dashR-tests branch April 22, 2020 00:11
rpkyle added a commit that referenced this pull request May 6, 2020
* 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>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants