Skip to content

tests: Temporarily skip timerange verification tests #13244

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.

Sign up for GitHub

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

Merged
jdamore-linode merged 2 commits into from
Jan 6, 2026
Merged

tests: Temporarily skip timerange verification tests #13244

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
765ae9a
Temporarily skip timerange verification tests to avoid release disrup…
jdamore-linode Jan 6, 2026
e3a4b7b
Added changeset: Temporarily skip time range verification Cypress tests
jdamore-linode Jan 6, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
5 changes: 5 additions & 0 deletions packages/manager/.changeset/pr-13244-tests-1767717203129.md
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,5 @@
---
"@linode/manager": Tests
---

Temporarily skip time range verification Cypress tests ([#13244](https://github.com/linode/manager/pull/13244))
18 changes: 16 additions & 2 deletions packages/manager/cypress/e2e/core/cloudpulse/timerange-verification.spec.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -187,8 +187,22 @@
.toFormat('yyyy-MM-dd HH:mm');
};

// It is going to be modified
describe('Integration tests for verifying Cloudpulse custom and preset configurations', () => {
/*
* TODO Fix or migrate the tests in `timerange-verification.spec.ts`.

Check warning on line 191 in packages/manager/cypress/e2e/core/cloudpulse/timerange-verification.spec.ts

View workflow job for this annotation

GitHub Actions / ESLint Review (manager) 

[eslint] reported by reviewdog 🐶
Complete the task associated to this "TODO" comment.

Raw Output:
{"ruleId":"sonarjs/todo-tag","severity":1,"message":"Complete the task associated to this \"TODO\" comment.","line":191,"column":4,"nodeType":null,"messageId":"completeTODO","endLine":191,"endColumn":8}
*
* The tests in this spec frequently fail during specific dates and time periods
* throughout the day and year. Because there are so many tests in this spec, the
* timeouts and subsequent failures can delay test runs by several (45+) minutes
* which frequently interferes with unrelated test runs.
*
* Other considerations:
*
* - Would unit tests or component tests be a better fit for this?
*
* - Are these tests adding any value? They fail frequently and the failures do
* not get reviewed. They do not seem to be protecting us from regressions.
*/
describe.skip('Integration tests for verifying Cloudpulse custom and preset configurations', () => {
/*
* - Mocks user preferences for dashboard details (dashboard, engine, resources, and region).
* - Simulates loading test data without real API calls.
Expand Down