This project is mirrored from https://github.com/metabase/metabase.
Pull mirroring updated .
- Oct 05, 2020
-
-
Nemanja Glumac authored
* Upgrade Cypress to v5.3.0 * Add Cypress test retries to the config (2 for the "run" mode and 0 for the "open" mode) Notes: - upgrading `@testing-library/cypress` was not possible at this point. It would almost certainly require upgrading webpack, and maybe adjusting babel settings. - `yarn resolutions` will not be needed once we properly upgrade weback and all libraries that have "Cypress" as a dependency
-
Nemanja Glumac authored
Tests are in stored in two separate files: - frontend/test/metabase/scenarios/admin/datamodel/segments.cy.spec.js and - frontend/test/metabase/scenarios/admin/datamodel/metrics.cy.spec.js
-
- Oct 02, 2020
-
-
Robert Roland authored
* Upgrade to log4j 2.13.3 Upgrades us to newer Log4j Resolves #12719 * fixing reflection warnings * reflection warning on test * make tests work took a while to figure out how to configure log4j properly programatically - it's not as easy as log4j 1 add logging adapters for commons-logging, slf4j, liquibase, etc so that all logging routes through log4j2 now * appease the linters * okay cljr-clean-ns makes this line too long, I get it * add log4j2.xml config for Cypress tests * update log4j configuration file documentation update which flag is used by the backend tests [ci all] * update cheshire because clj-http updated * Wire up the metabase-appender to Log4j2 This was brutal to figure out - log4j usually silently fails, so it took finding the right combination of configuration settings and system properties to get it to log enough to find out why the appender was never invoked.
-
-
Nemanja Glumac authored
Add Postgress tests to Cypress test suite This commit is introducing a new feature by adding a possibility to test Postgress related issues. It adds an updated CI configuration that plays nice with [Docker QA database](https://github.com/metabase/metabase-qa) (postgress-12). It also adds a few repros, listed below: * Add repro for #12762 * Add repro for #13263 (#13303) --- * Add an example for how to run metabase-qa postgres in CI * Add e2e test for adding/connecting PostgreSQL database (#13296) Co-authored-by: @dacort
-
Nemanja Glumac authored
* Add repro for #13332 - make the final assertion using "count" - apply different filters to the same question instead of creating 2 separate questions
-
- Sep 28, 2020
-
-
Nemanja Glumac authored
* Add repro for #13000 * Improve test with added log and assertion * Refactor the test to avoid flakiness - it was failing in CI: 1. finding multiple "Product IDs" 2. XHR event "@card" never originated (locally, the card ID is 6, but in CI it seems it is 8, from what I can see in a video) (replacing hard coded ID `6` with a wildcard `*`) - wrote a different flow so I don't have to remove pre-selected "Count" after I add "Sum of ..." - added mid-point assertions on generated titles
-
Nemanja Glumac authored
-
Nemanja Glumac authored
* Add repro for #13306 - test ONLY checks that parenthesis are indeed being removed on blur
-
Nemanja Glumac authored
* Move server and routes before visiting urls More info: https://github.com/cypress-io/cypress/issues/3427#issuecomment-504938130 * Wait for table to finish updating before reordering its fields * @flamber pointed out that the possible cause of this failing test lies in #13024 and this fix might not be enough until that issue is resolved
-
- Sep 25, 2020
-
-
Damon P. Cortesi authored
-
Nemanja Glumac authored
* Fix `snippet` Cypress error - this was very annoying flake responsible for a lot of failed builds => it shouldn't fail again - added "hack" function for typing inside autocomplete fields - added simple math in the second case with altered assertion
-
Nemanja Glumac authored
-
- Sep 22, 2020
-
-
flamber authored
Co-authored-by: Paul Rosenzweig
-
- Sep 21, 2020
-
-
Cam Saul authored
Merge release 0.36.6 into master
-
Nemanja Glumac authored
* Apply new format to the issue references in tests and other "affected" files across the code base * Add previously deleted issue references back to their test
-
Cam Saul authored
-
Nemanja Glumac authored
* Migrate `Button` unit test from enzyme to react-testing-library * Delete 2 obsolete `Button` snapshots
-
- Sep 18, 2020
-
-
Cam Saul authored
-
Reza Lotun authored
* skip db migration test for lowercase email edge-case on mysql and mariadb [ci all] * remove offending test [ci all]
-
Walter Leibbrandt authored
* Add --keep-existing flag to dump-to-h2 command * Explain --keep-existing flag in docstrings * Don't log db password
-
- Sep 17, 2020
-
-
Reza Lotun authored
Changes the app db to lowercase all emails, and queries to default to the lowercased version of emails. This fixes the problem of users logging in with emails where one or more characters could be capitalized. Resolves #3047
-
Nemanja Glumac authored
-
Nemanja Glumac authored
-
- Sep 16, 2020
-
-
Cam Saul authored
[ci postgres] [ci mysql]
-
Nemanja Glumac authored
-
Nemanja Glumac authored
* Add `role` and `aria-label` to icons * Expand `Icon` unit tests * Simplify `NON_STANDARD_VIEWBOX_ICON` * Update snapshots related to a11y improvements to icons * Pass `aria-label` directly as an icon attribute
-
- Sep 15, 2020
-
-
Simon Belak authored
- Sep 14, 2020
-
-
Simon Belak authored
* MBQL: make mbql.u/field-clause->id-or-field work with expressions * Update docs * Fix test
-
- Sep 11, 2020
-
-
Cam Saul authored
Merge 0.36.5.1 back into master
-
Cam Saul authored
-
Robert Roland authored
This test fails when the app db is postgres / mysql / mariadb because the database isn't setup when it runs. For now, skip this test on anything but H2 to fix a broken master build.
-
Cam Saul authored
-
Robert Roland authored
-
Robert Roland authored
* Support IPv6 for LDAP connections Adds support for IPv6 LDAP connections. The underlying clj-ldap library requires a different format of the host parameter to support IPv6. IPv6 addresses must be wrapped in [], like: [2601:600:967f:88d4:b476:56a8:c364:94da] Converted the LDAP expect tests to clojure.test also. Resolves #12879 * Update LDAP IPv6 test to handle CircleCI not supporting IPv6 Co-authored-by:
Cam Saul <github@camsaul.com>
-
Robert Roland authored
Excel will not parse an ISO8601 date, so add a new version of FormatValue specifically made for Excel. The Docjure library used has formatting helpers for a few Java classes, but specifically java.util.Date and not the newer Java 8 Time classes. In this case, it formats any java.util.Date as m/d/yy. Resolves #10803
-
Robert Roland authored
* convert tests to deftest convert more expectations based tests to deftest * Remove orphaned sync jobs from the database If a database is deleted, make sure that any applicable sync jobs are removed upon the next sync run. Resolves #11813 * rename mdb to database for convention
-