Skip to content
Snippets Groups Projects
Unverified Commit bbc49e68 authored by Nemanja Glumac's avatar Nemanja Glumac Committed by GitHub
Browse files

Merge `Run Cypress EE tests in isolation` PR from `release-x.37.x` branch (#14339) (#14377)


* Run Cypress EE tests in isolation (#14339)

* Run all tests in isolation for `ad-hoc.cy.spec.js` [ci skip]

* Refactor sandboxes [ci skip] (#14340)

* Fix typo

* Use API for question creation

* Divide test for admins and sanboxed users

* Organize tests better and run them all in isolation

* A detailed explanation of all changes is available in the original PR: #14340

* Use template tag variable in an SQL query

Co-authored-by: default avatarflamber <1447303+flamber@users.noreply.github.com>

* Refactor whitelabel Cypress tests [ci skip] (#14346)

* Run all tests in isolation for `drill_through.cy.spec.js`

Note: This test is skipped. It should probably be deleted.

* Run all tests in isolation for `snippet-permissions.cy.spec.js`

* Use aliased import path [ci skip]

* Refactor Cypress test related to changing the comapny name

* Refactor Cypress test related to the logo customization

* Update tests related to colors

* Run all tests in isolation

* Reorganize tests and run them in isolation

* Extract company name in a `const`

* Supply only `value` to API logo update

* Refactor auditing Cypress tests [ci skip] (#14362)

* Use API to create dashboards
Note: This change alone cuts down total run time by 10s.

* Clean up helper functions

* Extract the setup logic

* Reuse `USERS` object

* Tidy up the test

* Do not export helper functions [ci skip]

* Simplify variables [ci skip]

* Mark skipped test with the issue number [ci skip]

Co-authored-by: default avatarflamber <1447303+flamber@users.noreply.github.com>

* Fix `auditing` test failure on `master` (#14378)

The failure was introduced by this commit: https://github.com/metabase/metabase/commit/992f8475b4c68dd96032e99ee0cd1274204f12b8



### Additional context
- We now have randomized sync times so it's not possible to assert on the string "Every hour" anymore.
- Given that the sync time was not even crucial for this test, I have removed that assertion and added two new ones

Co-authored-by: default avatarflamber <1447303+flamber@users.noreply.github.com>
parent ad1845b6
No related branches found
No related tags found
No related merge requests found
Loading
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment