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

[Time To Visualize] Add Functional Tests After Config Option Defaults to True #80584

Closed
33 tasks done
ThomThomson opened this issue Oct 14, 2020 · 3 comments
Closed
33 tasks done
Labels
enhancement New value added to drive a business result Feature:Dashboard Dashboard related features Project:TimeToVisualize Team:Presentation Presentation Team for Dashboard, Input Controls, and Canvas

Comments

@ThomThomson
Copy link
Contributor

ThomThomson commented Oct 14, 2020

Once the config option allowByValueEmbeddables is set to true by default, or removed. We will have to begin implementing functional test cases for testing all by value and by reference behaviours. This issue will track the functional tests that need to be implemented.

Panels By Value - Lens / Maps / Visualize

Add from app by Value - Lens / Maps / Visualize #89245

  • Create new visualization and add to new dashboard via new modal (@poffdeluxe)
  • Create new visualization and add to existing dashboard via new modal (@poffdeluxe)
  • Add existing visualization to new dashboard (@poffdeluxe)
  • Add existing visualization to existing dashboard (@poffdeluxe)
  • Verify that users that do not have certain dashboard privileges cannot add to dashboard (@poffdeluxe)

Remove State from URL #86939

  • Unsaved changes persist when user navigates to another app. (@ThomThomson )
  • View mode restores last saved panels (@ThomThomson )
  • Edit mode restores unsaved panels (@ThomThomson )
  • Unsaved edits do not show up after saving (@ThomThomson )
  • Unsaved edits do not show up when the edited dashboard is the same as the saved dashboard (@ThomThomson )
  • Unsaved changes show up properly in listing page (@ThomThomson )
  • Unsaved changes to new dashboard show up in listing page (@ThomThomson)
  • Delete unsaved changes from listing page. (@ThomThomson )
  • New dashboard in progress warning. (@ThomThomson )
  • Listing page restores unsaved edits (@ThomThomson )

Getting Started / Best Practices

Run the test server (to start ES/Kibana) from either root dir or x-pack dir: node scripts/functional_tests_server
Now you can run the runner to kick off the tests: node scripts/functional_test_runner

To limit the tests being run, you can add a --grep option to the runner command or change the describe of the test group to be describe.only

When writing tests, it might be necessary to add data-test-subj attributes to elements in the application that you need to interact with. You can then use the testSubjects service to easily select those elements.

@poffdeluxe
Copy link
Contributor

poffdeluxe commented Jan 14, 2021

Added a few tests to the list that specifically have to do with creating/editing visualizations from their app (Viz/Lens) and adding to a dashboard by-value using the new modal

@LeeDr
Copy link
Contributor

LeeDr commented Feb 2, 2021

  • test that the Saved Objects page shows the relationship from a dashboard to the index pattern(s) which are related to via by-value visualization(s).
  • test that exporting a dashboard with "Include related objects" exports the index pattern used by the by-value visualizations
  • test importing exported dashboards along with related index pattern

@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-presentation (Team:Presentation)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New value added to drive a business result Feature:Dashboard Dashboard related features Project:TimeToVisualize Team:Presentation Presentation Team for Dashboard, Input Controls, and Canvas
Projects
None yet
Development

No branches or pull requests

5 participants