QP: use MLv2 metadata provider. Eliminate 50% of app DB calls and improve performance by 10%+ (#33221)

* QP use MLv2 metadata provider (34/2)

* (22/2)

* (21 / 0)

* (20 / 4)

* (7 / 2)

* (2/0)

* Cleanup; (3/0)

* Last two test fixes

* (36 / 2)

* (8 / 1)

* Reorder stuff

* Test fixes 

* Test fixes

* Some test fixes

* More test fixes 

* Test fix 

* MongoDB test fix

* B I G  cached metadata provider performance improvements

* Revert breaking change

* Fix Kondo

* Make sure application database metadata provider returns Database :features

* Test fix 

* Parallel tests for QP macroexpansion middleware

* `with-current-user` is fine in parallel tests.

* Add test util remap metadata providers and rework remap middleware to use mock MLv2 data

* Address PR feedback

* Improve unrelated flaky test

* Some test improvements

* Fix lint error

* MLv2-ize `nest-query-test`

* Fix typo in PR feedback changes

* More parallelization and test fixes 

* Convert lots more tests to mock metadata

* Test fix? 

* Fix Card update logic

* Another Card update logic fix 

* Another round of fixes 

* Hopefully no more test fixes 

* Evil test fixes 

* Test fix 1

* Test fix

* Add test

* Update imports

* Add value normalization and save value on blur

* Update e2e/test/scenarios/dashboard-filters/dashboard-filters-sql-number.cy.spec.js

Co-authored-by: Mahatthana (Kelvin) Nomsawadi <me@bboykelvin.dev>

* Update e2e/test/scenarios/dashboard-filters/dashboard-filters-sql-number.cy.spec.js

Co-authored-by: Mahatthana (Kelvin) Nomsawadi <me@bboykelvin.dev>

* Use helper

* Simplify test

---------

Co-authored-by: Mahatthana (Kelvin) Nomsawadi <me@bboykelvin.dev>

Add test to dashboard parameter should occupy whole horizon space when navigating to dashboards inapp in interactive embed (#33337)

* Add a repro for #30645

* Reuse util function

* Extract logic to generate next unsaved card id to another function

* fix specific aria-label for generic sidebar

* hide dashboard subscription button

- when user is non-admin and email & slack are not configured

* forward non-admin user to configured subscription form

* fix: show pulse list after creation for non-admins

- when pulse count is 0 and a new pulse is added by non-admins, it used to auto-redirect to the creation form again b/c the new pulse was not loaded before the forwarding would occur again

* refactor magic strings

* fix error when pulses has no length attribute

* validate forwarding in subscriptions for non-admins

* add remaining specs

* updated tests for new subscriptions forwarding for non-admins

* update tests for subscriptions forwarding for non-admins

* update tests for subscriptions forwarding for non-admins

* fix unit tests

* update dashboard subscriptions button to exclude heading cards

* [WIP] convert E2E tests to unit tests

* convert subscriptions button test to unit test

* restore some E2E tests

* fix unit tests

- new fetch request occurring in child component DashboardHeader was causing these tests to fail on the fetch request because there was no fallback

* Use findByLabelText as we're targeting a checkbox input

* Add reproduction test for #22265

* Break second question after adding it to series

* Format code

* Correct the repro: Use valid question first before turning it invalid

* Speed up E2E test by preparing data using API

---------

Co-authored-by: Mahatthana Nomsawadi <mahatthana.n@gmail.com>

* Split `admin` E2E group and merge it with `auditing`

* Run new `admin-2` group in CI

* remove add new h2 db test

* update setup specs to test mysql and postgres connections

* check for mongodb connection string errors

* Pass dashboardId and dashcardId in QueryDownloadWidget in ViewFooter
* Add missing dashboardId and dashcardId in QueryDownloadWidget.tsx
* spec

---------

Co-authored-by: Robin <e.robin154@gmail.com>

* Add a repro for 16663

* Make test real

* Address review

* Update e2e/test/scenarios/dashboard-filters/reproductions/16663-filters-can-stay-in-url.cy.spec.js

Co-authored-by: Kamil Mielnik <kamil@kamilmielnik.com>

* typo

---------

Co-authored-by: Kamil Mielnik <kamil@kamilmielnik.com>

* Modify an E2E test and unskipped it

* Fix calling wrap when table is undefined

* troubleshooting CI

* Fix failed test on CI

* Adding is-true and is-false operators

* Adding unit tests

* All Operators adjustment

* PR Feedback

* add "is equal to" in spec

* adding e2e test

---------

Co-authored-by: Jesse Devaney <22608765+JesseSDevaney@users.noreply.github.com>

The flake happened when adding models to the dashboard
due to the conflicting intercepted routes that we're waiting for.

This commit removes intercepts and waits altogether.
It relies on UI instead.

Example of a failed run:
https://github.com/metabase/metabase/actions/runs/5882560809/job/15953891019?pr=33245#step:14:502

* Add E2E tests for downloads in embedding questions pt1

This commit adds tests that make sure preventing downloads from the
embedding questions is not possible without a token.

* Add E2E tests for premium embedding downloads

This commit adds tests that make sure the premium token allows one to
disable downloads for embedded questions.

* cancels ongoing cards when saving a dashboard

* refactor: change from stub to spy so that we can also see in the inspector that the request has been cancelled

* test that the data will eventually be displayed

* Consolidate "create new dashboard" workflow

* Consolidate "add new dashboard card" tests

* Remove redundant repro for 25705

* Group and consolidate "title and description" tests

* Move repro 26826 to the main `dashboard` corpus

* Fix `visualization-options` spec name

* Make `dashboardHeader` more specific

* Consolidate "duplicate dashboard" tests

* Clean up and consolidate all dashboard management test

* Fix test failure

https://www.deploysentinel.com/ci/runs/64d9eb30fa6644982d115267

* Fix dashboard auto apply filters test failure

https://www.deploysentinel.com/ci/runs/64d9e91c7b3e6a1ddb9363c8

* Fix custom homepage test

https://www.deploysentinel.com/ci/runs/64d9e9a8f048784ce96aedf2

* Another attempt to fix a failing test

* Bring back a test removed during rebase

* Extract `aceEditorStyle`

* Extract drag handle

* Port more `NativeQueryEditorRoot` class names

* Port editor root and resizable box

* Remove redundant class names

* Remove `.NativeQueryEditor` class

* Use CSS variable colors in `NativeQueryEditor.css`

* Add a comment on `NativeQueryEditor.css`

* Use named import for ErrorView

* Export ErrorView and ErrorViewProps from Visualization

* Define AddSeriesModalErrorView

* Make ErrorView component configurable in Visualization by introducing errorView prop

* Use AddSeriesModalErrorView thanks to new errorView prop

* Fix export type syntax

* Remove the dot for consistency with other error messages

* Rename AddSeriesModalErrorView to ErrorView and extract it to a separate file

* Allow translating the custom error message

* Don't show custom ErrorView when there is only one series (or none)

* Add a test case for a single incomplete chart

* Inline: dashcardData

* Add a reproduction e2e test for #32231

* Add explicit assertion for the error message

* Add a test case for default error message when there is only 1 series

* Remove redundant setup functions

* Wait for series query endpoint

* Rename ErrorView to VisualizationErrorView

* Rename VisualizationErrorView to MultipleSeriesErrorView

* Improve assertions

* Use editDashboard helper

* Use data-testid for AddSeriesModal

* Refactor errorView prop into errorMessageOverride
- Do not export ErrorViewProps and Error out of Visualization

* Search by table description

[Fixes #25792]

* Fix initial values not reloading in the action modal

* Add a repro test for #33084

* Use cy.button helper

* Fix dashboard actions linked with filters don't work

* Add E2E test

* Make assertion clearer

Creating a collection when saving a question/dashboard should return to the question/dashboard save modal with the new collection selected (#32778)

From the Cypress docs
https://docs.cypress.io/guides/references/configuration#Global

The number of tests for which snapshots and command data are kept in memory. Reduce this number if you are experiencing high memory consumption in your browser during a test run.

The default value is 50.
We're dialing it down to 1.

* reproduce #32985

* don't crash fieldValuesWidget when we get null options

* Add reproduction test for #28756

* Reuse DASHBOARD_SLOW_TIMEOUT