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

Create Cypress E2E tests based on the test coverage report #5208

Open
1 of 17 tasks
gabrieldutra opened this issue Oct 13, 2020 · 5 comments
Open
1 of 17 tasks

Create Cypress E2E tests based on the test coverage report #5208

gabrieldutra opened this issue Oct 13, 2020 · 5 comments
Labels
Frontend hacktoberfest Hacktoberfest Friendly Issues

Comments

@gabrieldutra
Copy link
Member

gabrieldutra commented Oct 13, 2020

Description

Cypress is the tool we've used for End-to-end testing for the past couple of years. Recently, we generated a test coverage report (#5137), which gives us a better view of areas that could benefit from some testing. Existing tests can be found in client/cypress/integration and docs for local testing can be found here.

cypress-example

Tests to be created

  • Chart Visualization
    • Line
    • Bar
    • Area
    • Pie
    • Scatter
    • Bubble
    • Heatmap
    • Box
  • List Pages
    • Queries List
    • Dashboard List
    • Alerts List
  • Query "Add to dashboard" feature
  • Details Visualization
  • Edit Alert Destination
  • Databricks Schema Browser
@gabrieldutra gabrieldutra added Frontend hacktoberfest Hacktoberfest Friendly Issues labels Oct 13, 2020
@ask-imran
Copy link

ask-imran commented Oct 16, 2020

I would love it give it a go

@gabrieldutra
Copy link
Member Author

Thanks @ask-imran, feel free to pick any of the ones I mentioned 👍

The "Edit Alert Destination" (very similar to the existing edit_data_source_spec.js) should be a simple start if you are not used to the Redash interface, but LMK if you got interest in anything else. The docs should be of great help, but if you have any questions just send them in here 🙂.

@rafawendel rafawendel mentioned this issue Oct 19, 2020
5 tasks
@ask-imran
Copy link

ask-imran commented Oct 20, 2020

I have been having trouble running the app/cypress tests. Followed the guide

after the dependencies installed running npm build throws error (on windows machine)
running cypress tests fail during login, the login page as well looks a bit broken (may be due to forntend assets not being built)

Am sorry if it has been documented somewhere, and Instead of spamming the PR with the comments I can take it to the forum.

Please let me know if I have missed something.

@gabrieldutra
Copy link
Member Author

on windows machine

Oh 😅, I'm afraid we didn't officially support Windows for development, so this can be it, but it should be possible with Windows Subsystem for Linux (I saw this post in the forum). Does npm run watch show the same error?

I suppose it makes sense to move this to the forum under the Development category.

@ask-imran
Copy link

Thanks @gabrieldutra
I was trying it on my work laptop, it didn't have the specific build for WSL. I'll try it on my personal laptop. 🤞

@rafawendel rafawendel mentioned this issue Nov 20, 2020
1 task
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Frontend hacktoberfest Hacktoberfest Friendly Issues
Projects
None yet
Development

No branches or pull requests

2 participants