diff options
author | Thomas Steur <tsteur@users.noreply.github.com> | 2019-05-25 00:18:03 +0300 |
---|---|---|
committer | diosmosis <diosmosis@users.noreply.github.com> | 2019-05-25 00:18:03 +0300 |
commit | 871c31d0962a144b85ca2601a24b9d10ab707bfc (patch) | |
tree | fd5c680f7f480319ee521b54e604c351f11a7f55 /tests | |
parent | b9df06dadfd8030401bb9ae1b8126e4cf13f33b5 (diff) |
Changed period in row evolution breaks graph/metric association (#14423)
* Changed period in row evolution breaks graph/metric association
fix #14208
worked for me, also with multi row evolution. Only known issue I found is if requesting eg first yearly row evolution which may have the `users` metric hidden, then switching to `day` period, and then the `users` metric is maybe supposed to become available but didn't. That should be quite edge case though.
* add class whether row is hidden
* Fixing couple regressions visible in ui tests.
* fix build + update comment
* tweak
* test tweak
Diffstat (limited to 'tests')
-rw-r--r-- | tests/UI/specs/RowEvolution_spec.js | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/tests/UI/specs/RowEvolution_spec.js b/tests/UI/specs/RowEvolution_spec.js index fab3a148f6..d8cad654e0 100644 --- a/tests/UI/specs/RowEvolution_spec.js +++ b/tests/UI/specs/RowEvolution_spec.js @@ -8,8 +8,6 @@ */ describe("RowEvolution", function () { - this.timeout(0); - const viewDataTableUrl = "?module=Widgetize&action=iframe&moduleToWidgetize=Referrers&idSite=1&period=week&date=2012-02-09&" + "actionToWidgetize=getKeywords&viewDataTable=table&filter_limit=5"; @@ -62,6 +60,7 @@ describe("RowEvolution", function () { await page.waitForNetworkIdle(); const dialog = await page.$('.ui-dialog'); + await page.waitForNetworkIdle(); expect(await dialog.screenshot()).to.matchImage('multirow_evolution'); }); |