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

E2E tests - Create alert #1729

Merged
merged 22 commits into from
Jan 2, 2025
Merged

E2E tests - Create alert #1729

merged 22 commits into from
Jan 2, 2025

Conversation

harshilvelotio
Copy link
Collaborator

Issue(s)

List links to the issues being addressed.

Description

Describe the changes being introduced.

Checklist

  • if queries to the database are changed: think about query performance and indexes

Screenshots

Attach screenshots of UI/UX changes, if applicable.

How to test

List steps to reproduce the issue(s) and test the change, if required.

@harshilvelotio harshilvelotio requested review from a team as code owners December 17, 2024 14:19
Copy link

vercel bot commented Dec 17, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
thirdeye ✅ Ready (Inspect) Visit Preview 💬 Add feedback Dec 30, 2024 10:16am

@nalin-patidar nalin-patidar merged commit d8a7fae into e2e-setup Jan 2, 2025
6 of 7 checks passed
@nalin-patidar nalin-patidar deleted the harshil/e2e-tests branch January 2, 2025 04:02
nalin-patidar added a commit that referenced this pull request Jan 2, 2025
* playwright setup

* alert list tests

* alert list done

* configuration

* feat(ui): TE-2589 e2e setup and P0 tests

* gh action

* remove hc

* fix action

* test run

* action file fix

* gh action fix

* env fix

* playwright command

* playwright command fix

* run server and wait for it to be up

* install pw browsers

* pw install

* fix tests and slack notification for action

* pwd fix in workflow

* gh action fix

* notification text update

* fix anomaly view test

* fixed flaky test

* update workflows npm ci issue

* E2E tests - Create alert (#1729)

* Create alert basic e2e test

* removed access token

* multi dimensional basic tests

* added comments

* Updated test cases

* undo'ed import

* fix(ui): Updated the test case

* fix(ui): Added Test cases for custom metric and advance options

* fix(ui): Added Test cases for Alert Details and anomaly

* fix(ui): Added Test cases for Onboarding page

* Test Check

* Anomaly test fix

* Anomaly View test fix

* Anomaly api test fix

* Anomaly test fix

* anomaly test updatded

* preventDefault added for Apply filter button

* Revert e2e yaml file change

* fix(ui): Added Test cases for Onboarding page when dataset is Empty

---------

Co-authored-by: harshilvelotio <67183907+harshilvelotio@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
thirdeye-ui Updates to thirdeye-ui project
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants