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

[Backport 2.12] Updating app analytics jest and cypress tests #1417

Merged
merged 1 commit into from
Feb 6, 2024

Conversation

opensearch-trigger-bot[bot]
Copy link
Contributor

Backport 3733eeb from #1398.

* some updates

Signed-off-by: Paul Sebastian <[email protected]>

* beginning assertion

Signed-off-by: Paul Sebastian <[email protected]>

* fix typo

Signed-off-by: Paul Sebastian <[email protected]>

* TEMPORARY: only run app analytics github action

Signed-off-by: Paul Sebastian <[email protected]>

* included wait for all requests

Signed-off-by: Paul Sebastian <[email protected]>

* explicitly add sample data flights and logs

Signed-off-by: Paul Sebastian <[email protected]>

* otel imported

Signed-off-by: Paul Sebastian <[email protected]>

* update otel requests

Signed-off-by: Paul Sebastian <[email protected]>

* reintroduce otel data

Signed-off-by: Paul Sebastian <[email protected]>

* switch browser to electron

Signed-off-by: Paul Sebastian <[email protected]>

* fixed issues with timepicker not being shared properly, separated app and explorer times

Signed-off-by: Paul Sebastian <[email protected]>

* small updates to spec

Signed-off-by: Paul Sebastian <[email protected]>

* fix some of the time logic

Signed-off-by: Paul Sebastian <[email protected]>

* small fixes to spec

Signed-off-by: Paul Sebastian <[email protected]>

* double click to open service flyout, doesn't work otherwise

Signed-off-by: Paul Sebastian <[email protected]>

* click to render chart

Signed-off-by: Paul Sebastian <[email protected]>

* fixed issue where availability lines weren't showing up due to deleted parsing

Signed-off-by: Paul Sebastian <[email protected]>

* extra character for query while autocomplete issue is being worked on

Signed-off-by: Paul Sebastian <[email protected]>

* uncomment everything whoops

Signed-off-by: Paul Sebastian <[email protected]>

* typing bandaid

Signed-off-by: Paul Sebastian <[email protected]>

* intercept loading panel and remove waits

Signed-off-by: Paul Sebastian <[email protected]>

* minimally viable testing

Signed-off-by: Paul Sebastian <[email protected]>

* switch back to electron

Signed-off-by: Paul Sebastian <[email protected]>

* temp load data for ftr run

Signed-off-by: Paul Sebastian <[email protected]>

* change waiting order

Signed-off-by: Paul Sebastian <[email protected]>

* only need one loading panels wait

Signed-off-by: Paul Sebastian <[email protected]>

* update ftr tests

Signed-off-by: Paul Sebastian <[email protected]>

* include two network intercepts

Signed-off-by: Paul Sebastian <[email protected]>

* ftr test update delete success getter

Signed-off-by: Paul Sebastian <[email protected]>

* add intercept after selection

Signed-off-by: Paul Sebastian <[email protected]>

* take into account new name

Signed-off-by: Paul Sebastian <[email protected]>

* manual wait to isolate other issues

Signed-off-by: Paul Sebastian <[email protected]>

* repeat test 10 times to check for flakiness

Signed-off-by: Paul Sebastian <[email protected]>

* revert repeat 10 tests

Signed-off-by: Paul Sebastian <[email protected]>

* updated snapshots

Signed-off-by: Paul Sebastian <[email protected]>

* remove ftr test

Signed-off-by: Paul Sebastian <[email protected]>

* go back to running all integration tests

Signed-off-by: Paul Sebastian <[email protected]>

* reduce usage of within

Signed-off-by: Paul Sebastian <[email protected]>

* run app analytics test 10 times on github runner

Signed-off-by: Paul Sebastian <[email protected]>

* add wait to avoid test avoiding checkbox

Signed-off-by: Paul Sebastian <[email protected]>

* explain why i have a wait

Signed-off-by: Paul Sebastian <[email protected]>

* remove repeating tests

Signed-off-by: Paul Sebastian <[email protected]>

* small changes

Signed-off-by: Paul Sebastian <[email protected]>

* change and remove some jest tests

Signed-off-by: Paul Sebastian <[email protected]>

---------

Signed-off-by: Paul Sebastian <[email protected]>
(cherry picked from commit 3733eeb)
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Copy link

codecov bot commented Feb 6, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (7442e6f) 54.27% compared to head (7442e6f) 54.27%.

❗ Current head 7442e6f differs from pull request most recent head 2ca6b20. Consider uploading reports for the commit 2ca6b20 to get more accurate results

Additional details and impacted files
@@           Coverage Diff           @@
##             2.12    #1417   +/-   ##
=======================================
  Coverage   54.27%   54.27%           
=======================================
  Files         315      315           
  Lines       11167    11167           
  Branches     2991     2991           
=======================================
  Hits         6061     6061           
  Misses       5057     5057           
  Partials       49       49           
Flag Coverage Δ
dashboards-observability 54.27% <0.00%> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@ps48 ps48 merged commit 931bee2 into 2.12 Feb 6, 2024
20 of 32 checks passed
@github-actions github-actions bot deleted the backport/backport-1398-to-2.12 branch February 6, 2024 17:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants