* allow including weekday to formatted dates

* specs

* remove panning logic from map boundary calculation

* add map viz percy tests

* wait for map load in visual tests

* Install `eslint-plugin-jest`

* Enable `eslint-plugin-jest`

* Run eslint auto fix

* Fix linter issues

* Fix get-column-key

* Fix extra space

* Fix ChartSettingFieldPicker

* Fix eslint issues

* Remove redundant describe

Since the frontend sends a date-style string with various differences from how Java's Date Formatter expects things,
we need to post-process some bits. The 'D' -> 'd' string replacement was missed in the post processing fn, and so in
some cases it was missed altogether, causing this bug.

Existing tests in `metabase.pulse.render.datetime-test` and `metabase.pulse.render.table-test` should already catch
this issue. Prior to the change they would fail, but they now pass after adding this PR's changes.

* oxford comma for parameter values with more than 2 items

* refactor logic to use a better translated string and fix tests

* Add `LoadingState` component

* Add `EmptyState` component

* Use `EmptyState` in `DataPickerView`

* Add loading and empty states to card picker

* Add loading and empty states to raw data picker

* Handle case when there's no tree data

* Add tests

* temp

* Adding cypress test

* removing test

* unskipping test

* Add shared helpers for mocking endpoints

* Add a11y props to simplify testing

* Hide back button when can't go back

* Fix prop type error

* Add data picker tests

* Fix comment

* Rename `server` to `server-mocks`

* Exclude common from coverage

* Fix type error

* Replace `waitFor` with `findByText`

* Fix double aggregation not working on some fields.

This fix adds make sure the table field source is `fields` as this is
done in the previous version before we changed it in PR#25267 which
introduces the bug in issue#27462

* Add a reproduction

* Address review: Improve test readability

* create pivot table viz directory

* extract and type utils and partitions object

* extract RowToggleIcon component

* extract PivotTableCell component

* add unit tests for pivotTable utils

* add PivotTable unit tests

* Remove `titleTemplateChange` from dashboard reducer

* Remove `breadcrumbs` from app reducers

* Always use real reducers in `renderWithProviders`

* Update `ModerationReviewButton` tests

* Update `QueryValidationError` tests

* Update `LastEditInfoLabel` tests

* Update `ProfileLink` tests

* Update table visualization tests

* Update `QuestionInfoSidebar` tests

* Remove `screen.debug` call

* Use `getByText` instead of `queryByText`

* Fix prop errors in `ViewHeader` tests

* Update `DatabaseEditApp` tests

* Remove `currentUser` prop from `renderWithProviders`

* Update `RenderWithProvidersOptions` type

* Fix failing tests

* Extract `setup` helper

* Use more readable test string

* Use `not.toBeInTheDocument` vs `toBeNull`

* Fix missing assertions

* Fix error about `widget` not being an element

* Fix missing `key` prop

* Improve new dashboard modal tests

* Remove not used import

* Fix typo

* Add missing `key` props

* Use `waitFor` instead of `act`

* fetch gtap from table and group

* make tests more repl-friendly

* Add `mockSettings` utility

* Use `mockSettings` in sidebar tests

* Use `mockSettings` in database edit app tests

* Use `mockSettings` in update info form

* Use `mockSettings` in caching utils

* Use `mockSettings` in question cache TTL field

* Use `mockSettings` in new dashboard modal

* Use `mockSettings` in question info sidebar

* Use `mockSettings` in save question modal

* Use `mockSettings` in profile link

* Ensure settings are passed to redux in tests

* Update signature

* Fix test

```
docker run \
       -p 3000:3000 \
       -v $PWD/my_log4j2.xml:/tmp/my_log4j2.xml \
       -e JAVA_OPTS=-Dlog4j.configurationFile=/tmp/my_log4j2.xml \
       metabase/metabase:v0.45.1
```

From https://github.com/metabase/metabase/issues/27497

Note that this specifies `log4j.configurationFile` and not
"log4j2.configurationFile" that we set in our bootstrap.clj.

I'm not sure if this is supposed to work or not but we'll give it a
shot.

the Automatic Configuration section of
https://logging.apache.org/log4j/2.x/manual/configuration.html specifies
that:
1. Log4j will inspect the "log4j2.configurationFile" system property
and, if set, will attempt to load the configuration using the
ConfigurationFactory that matches the file extension. Note that this is
not restricted to a location on the local file system and may contain a
URL.

I'm not sure if that other property will work due to some other
mechanism or log4j2 has an undocumented back-compat mechanism.

* Rethinking the change completely

* Apply suggestions from code review

Co-authored-by: Jeff Bruemmer <jeff.bruemmer@gmail.com>

Co-authored-by: Jeff Bruemmer <jeff.bruemmer@gmail.com>

* Always return fields selected by $project in native MongoDB queries
* Keep _id at the position specified in $project if it's included there or at first position if it's not included

* upgrade clojure cli version

* bump clojure cli in dockerfile

* use expanded cache paths

* list cache contents

* ls repository

* enumerate m2 cache

* remove count of cached dirs

* restore `~` in cache paths

* Revert "remove count of cached dirs"

This reverts commit e768912b31075c7b19b4a3ccdb66c0248ea06c87.

* Revert "Revert "remove count of cached dirs""

This reverts commit 9eeaff2190d78714770d8ec9f64b015c75085876.

* trailing slash seems harmless but unnecessary

* nit alignment

* point to var so changes are picked up

need to refresh this ns after changing the middleware since these add a
bit more helpful doc strings. But capture previous function value so
more annoying at repl

* Ensure MB_API_KEY is set for notify endpoint

Previously when the MB_API_KEY was unset:

```
❯ http POST "localhost:3000/api/notify/db/1?scan=schema"
HTTP/1.1 403 Forbidden
<other headers removed>

Forbidden
```

And now:

```
❯ http POST "localhost:3000/api/notify/db/1?scan=schema"
HTTP/1.1 403 Forbidden
<other headers removed>

MB_API_KEY is not set. See https://www.metabase.com/docs/latest/configuring-metabase/environment-variables#mb_api_key for details
```

An annoying thing in the tests. We set `"-Dmb.api.key=test-api-key"` in
the `:test` alias for CI, but not in regular dev mode. So annoyingly we
either have tests that would fail in the repl in regular dev, or use
`mt/with-temporary-setting-values [api-key "test-api-key"]` when writing
tests that only matter for local stuff. c'est la vie

* Translate error message

Using site locale because this in an `+apikey` context, not an `+auth`
context. The importance of that is that there is no logged in user so no
way to get a user locale, and it just falls back to the server locale
anyways. So lets just use that.

* Fix temp settings when they are env-based

An annoying bit here. `api-key` setting in the notify test is set in the
env in the `:test` alias.

```clojure
notify-test=> (mt/with-temporary-setting-values [api-key nil]
                (mw.auth/api-key))
"test-api-key" ;; where did the nil go?
```

But there are two bugs in `mt/with-temporary-setting-values`:

1. It gets the current env value and then temporarily sets it to the
current value. Note that `do-with-temp-env-var-value` is called with
`env-var-value` which is the current value in the environment. All of
this work for nothing

```
-    (if-let [env-var-value (and (not raw-setting?) (#'setting/env-var-value setting-k))]
-      (do-with-temp-env-var-value setting env-var-value thunk)
+    (if (and (not raw-setting?) (#'setting/env-var-value setting-k))
+      (do-with-temp-env-var-value setting-k value thunk)
```
So cannot possibly do what we want.

2. The second bug is that the pathway through
`mt/with-temporary-setting-values` did not convert `:api-key` to
`:mb-api-key`. So even if it put our new value in above, it would put
`:api-key nil` in the env, but when we look for it it would look for
`:mb-api-key` and we would not find it.

Thus the use of `setting-env-map-name`. Then clean up the few other
places that did it kinda right, but not necessarily. Seems the "correct"
way to do this is get the setting name, munge it, prepend with mb. The
other call sites were using variants of `(keyword (str "mb-" (name
setting-name)))` which is close but could miss the munging.

* Safely set env/env keys

There are two ways that we put things into the env/env so they appear as
environmentally set variables.

- explicitly with `mt/with-temp-env-var-value`. This simulates a user
setting these values and they are specified in the env way:
mb-email-smtp-port, "MB_DISABLE_SCHEDULER", etc. At the call site you
are aware that they are env related so get the mb prefix
- incidentally with `mt/with-temporary-setting-values`. Settings are
always referred to by their "nice" internal name like `api-key`,
etc. But if a setting is found to be an env variable, we override it in
env/env. But we have to make sure the key is set properly with an mb-
prefix.

Owing to this, we have to conditionally add the mb- prefix if not
already present.

* cleanup the constants a bit

* Fix some site-url tests

The tests themselves are testing useful stuff: site-url doesn't return a
broken value if it fails the verification when set as an env
variable. But the tests themselves were a bit nonsensical in their
setup:

```
@@ -62,21 +62,19 @@
 (deftest site-url-settings-normalize
   (testing "We should normalize `site-url` when set via env var we should still normalize it (#9764)"
     (mt/with-temp-env-var-value [mb-site-url "localhost:3000/"]
-      (mt/with-temporary-setting-values [site-url nil]
-        (is (= "localhost:3000/"
-               (setting/get-value-of-type :string :site-url)))
-        (is (= "http://localhost:3000"
-               (public-settings/site-url)))))))
+      (is (= "localhost:3000/"
+             (setting/get-value-of-type :string :site-url)))
+      (is (= "http://localhost:3000"
+             (public-settings/site-url))))))
```

Why would it set an env-variable version [mb-site-url "localhost:3000/"]
then try to blank it out with `(mt/with-temporary-setting-values
[site-url nil])`.

Fixing the bug in how we set env var values made this nonsensical stuff
break.

Now we just have to do the obvious thing: set the env var to a bad
value, assert that (setting/get-value-of-type :string :site-url) is that
value, but assert that the _setting_ is a well-formatted value or nil if
we can't solve it (the "asd_12w31%$;" case).

Setting the env value then setting a temporary value to nil didn't make
much sense. And the fix made that set the env var to nil.

* fixup last tests

* Don't modify things from `with-temp-env-var-value`

this allows us to set non-mb-prefixed env vars. not sure if necessary
but let's roll with it.

* fix last of the tests

* fix automatic dox generation for malli+defendpoint

- there isn't a concept of not having an error message for a schema \o/
- warn but don't break when the schema is invalid
- polish the descriptions a little "a nullable a vector" -> "a nullable vector"

* gitignore malli types files

* schema -> defendpoint api/setting GET :key

+ docs example

* schema -> defendpoint api/action + dox generation

* quickfix + add smaller trivial tests

* add imported clj kondo rules to git

* add with-api-error-message to fix some tests

* use mu/with-api-error-message

* fix tests

* finish defendpoint migration for api/action ns

* do not stringify errors

* fix devtime annoyance with malli-dox generation

* more polish

- add length to numerics
- respond to review

* unused var linter fix

* fix off by 1 in umd/description

* aligator eats the big number + update action.md

* Disallow axis ranges with the same min/max, which causes divide by zero error

The group-axes private fn is used to try group series axes together if the ranges of the series are 'close enough',
which is calculated by determining the percentage overlap the ranges share.

As the comparison is made, the first series in the list will always be compared with itself, and should have
1.0 (100%) overlap, as the ranges are going to be identical. The divide by zero issue arises when this first series,
and potentially any other series, has a 'range' where the min and max are the same. This happens if the series has the
same value in every row.

With this change, these ranges are considered invalid, and we avoid the divide by zero.

* Unskip e2e repro

* Remove not used `StaticEntitiesProvider`

* Fix typos

* Convert sample database fixture to TypeScript

* Fix `EnhancedState` typing

* Remove global eslint comment

* Fix invalid import

* Fix incorrect type

* Add missing fields to `Settings` type

* Make `MetabaseSettings` use up-to-date setting types

* Fix type errors

* Fix error about redux store fixture

* Always add settings and embedding reducers

* Add `value` to `SettingDefinition` type

* Add `settings` type to admin state branch type

* Add `users` to entities state

* Convert `__support__/ui` to TypeScript

* Fix error when rendering with router mock

* Remove `form` from redux state

* Address feedback

* Fix tests setup

* Add return type

* add the new revamped tooltip to row charts

* more specs

* Fix mongo version checking for now

* Add minor versions for comparison

* Handle nil case

* newline

* Replace _

* Use dbms-version semantic-version instead

* Compare only the major and minor version

* Fix mongo database-supports?

* Fix duplicate require

* Fix typo

* Remove expressions from version check

* Fix various mongo expressions

`trim`, `rtrim`, `ltrim` needed to be wrapped in `{"input" expr}` object
`replace` needed to be wrapped in `{"input" expr "find" ...
"replacement" ...}' object
`substring` needed to fill in the 3rd argument, optional in mbql but
required in mongo. Also to use a 1 based index

* Generalize semantic-version-gte

* Used synced dbms_version for testing feature support

* Expressions are only supported by mongo 4.2+

* Disable some tests

* Fix mongo division

Handle nulls with an upfront condition check.
Handle multiple divisors.

* Remove unused namespace

* Throw exeception if using replace on mongo < 4.4

* Skip test because of #27249

* Move minimum Mongo CI version to 4.2

* Fix sorting by expressions

$sort needs to come before $project otherwise we can only see the
projected fields, however expressions are only added in $project.

So now if a sort includes an expression, we will use $addFields to be
able to sort by that expression.

* Disable tests for expressions inside aggregations

To be addressed by #27275

* Handle aggregation nested in an expression

* Remove :truncation-start workaround

* Enable tests for expressions inside aggregations

* Fix datetime-math-tests

* Make sure dbms_version is included when fetching database for store

* Update doc for replace to indicate it should replace all occurrences

* Add mongo datetime-diff

* Fix database-supports?

* Fix order-by-test

* Handle embedded special aggregations (#27315)

* Handle embedded special aggregations
* Preserve aggregation options for nested aggregations
* Use top-level aggregation name as group name

* Disable nil punning on semantic version check (#27317)

* Remove nil punning from database-supports?

* Formatting

* Refactor based on Tamas' suggestions

* Change datetime-diff from case statement to multimethod

Co-authored-by: Case Nelson <case@metabase.com>
Co-authored-by: Tamás Benkő <tamas@metabase.com>
Co-authored-by: metamben <103100869+metamben@users.noreply.github.com>

* Limit session timeout amount to less than 1 million

* Limit the max timeout to 100 years

* Validate on BE too

* Typo

* Add comment to FE

* Fix BE validation

* Fix BE validation

* Add BE test

* Change FE validation to mirror BE validation

* Add unit test for SessionTimeoutSetting

* Refactor FE unit tests

* Move comment

* Add check for positive amount to BE validation

* Add more BE tests

* Refactor validate function

* Remove cleanup()

* Use ToBeInTheDocument instead

* Use getByText

* Remove conditional expect

* Remove unused import

* Refactor for clarity

* Formatting

* Validate session-timeout in getter too

* Add docstring to check-session-timeout and make private

* Change getter to print warning string instead of throw exception

* Format import

* i18n and use log/warn in getter, and throw 400 in setter

* Reorder require

* Use cam's suggestion

Co-authored-by: Cam Saul <1455846+camsaul@users.noreply.github.com>

Co-authored-by: Cam Saul <1455846+camsaul@users.noreply.github.com>