You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@superset.apache.org by GitBox <gi...@apache.org> on 2020/10/20 18:14:06 UTC

[GitHub] [incubator-superset] adam-stasiak commented on a change in pull request #11207: feat: Added setup for running Cypress tests in docker locally

adam-stasiak commented on a change in pull request #11207:
URL: https://github.com/apache/incubator-superset/pull/11207#discussion_r508738444



##########
File path: CONTRIBUTING.md
##########
@@ -608,18 +608,34 @@ We use [Cypress](https://www.cypress.io/) for integration tests. Tests can be ru
 ```bash
 export SUPERSET_CONFIG=tests.superset_test_config
 export CYPRESS_BASE_URL="http://localhost:8081"
+# ENABLE_REACT_CRUD_VIEWS should be enabled for Cypress testing
+sed -i '/ENABLE_REACT_CRUD_VIEWS = False/c\ENABLE_REACT_CRUD_VIEWS = True' superset/config.py
+sed -i '/ENABLE_REACT_CRUD_VIEWS = os.environ.get("ENABLE_REACT_CRUD_VIEWS", False)/c\ENABLE_REACT_CRUD_VIEWS = True' tests/superset_test_config.py
+superset drop_db --yes
+superset create_db
 superset db upgrade
 superset init
 superset load_test_users
 superset load_examples
 superset run --port 8081
 ```
 
-Run Cypress tests:
+Build Frontend:
 
 ```bash
 cd superset-frontend
 npm run build
+```
+
+Alternatively you can proceed with Superset and Frontend running in Docker (hot reload for frontend not supported yet):
+
+```bash
+SUPERSET_CYPRESS_CONFIG=yes SUPERSET_CONFIG=tests.superset_test_config docker-compose up

Review comment:
       In 609 instruction it is said how to run everything without docker usage - I added this to be clear in required steps.




----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For additional commands, e-mail: notifications-help@superset.apache.org