Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Invalid filters #375

Merged
merged 15 commits into from
Jan 14, 2019
Merged

Invalid filters #375

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
15 commits
Select commit Hold shift + click to select a range
6ba022a
Moved filter validation into a separate function
Ro4052 Jan 4, 2019
0c1c93b
Fixed expected/received value order
Ro4052 Jan 7, 2019
a7dcb9f
Separated data and valid filter checks
Ro4052 Jan 7, 2019
780f769
Viewer now highlights invalid filters
Ro4052 Jan 7, 2019
cfe06cb
Slight re-structure to viewer filter validation
Ro4052 Jan 8, 2019
5b40c00
Slight refactor to perspective filter validation
Ro4052 Jan 8, 2019
de8e9fb
Updated styling for invalid filters
Ro4052 Jan 9, 2019
b3b5220
Re-named filter validation function
Ro4052 Jan 9, 2019
520850b
Added some commenting for newly exposed table method
Ro4052 Jan 9, 2019
5dba849
Added tool tip to invalid filters
Ro4052 Jan 9, 2019
f90cbd1
Can no longer filter by an empty string
Ro4052 Jan 9, 2019
c300a6d
Slight refactor of filter validation function
Ro4052 Jan 9, 2019
67131a5
Added screenshot test for invalid filter highlighting
Ro4052 Jan 10, 2019
e261e5f
Added test that empty string filters are not used
Ro4052 Jan 10, 2019
ecb7d44
Added tests for is_valid_filter function
Ro4052 Jan 10, 2019
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Added screenshot test for invalid filter highlighting
  • Loading branch information
@Ro4052
Ro4052 committed Jan 10, 2019
commit 67131a5a9a01ff38043fc88b890571fffe80dad2
7 changes: 6 additions & 1 deletion packages/perspective-viewer-highcharts/test/results/results.json
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -83,5 +83,10 @@
"render_warning.html/render warning should show above size limit.": "6b6f46fd4075987564eee7df17652d3d",
"render_warning.html/dismissing render warning should trigger render.": "1338b4720cc3374d42fe4c75a9e3dc8b",
"render_warning.html/selecting 'do not show again' should stop render warnings.": "1338b4720cc3374d42fe4c75a9e3dc8b",
"render_warning.html/underlying data updates should not trigger rerender if warning is visible.": "e606e604c4ade6671319f8d049ee5a63"
"render_warning.html/underlying data updates should not trigger rerender if warning is visible.": "e606e604c4ade6671319f8d049ee5a63",
"bar.html/highlights invalid filter.": "55b58961efb8ad7acc4d6613d4a5027e",
"scatter.html/highlights invalid filter.": "07814ed1500d1810d934ca8ac9f07486",
"line.html/highlights invalid filter.": "1dfcefa55fb5bad1804b1fbb708204de",
"treemap.html/highlights invalid filter.": "5b36e9d8a49d330118c2d6079262d268",
"heatmap.html/highlights invalid filter.": "1d1971aee71f389067e7f3c732d26a5f"
}
3 changes: 2 additions & 1 deletion packages/perspective-viewer-hypergrid/test/results/results.json
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -15,5 +15,6 @@
"superstore.html/pivots by a column.": "523c39ab27ca610b5ed179cbe742ec5b",
"superstore.html/collapses to depth smaller than viewport": "bdd25c1c40781478bd040d5816b887a6",
"regressions.html/saving a computed column does not interrupt update rendering": "da13afb4284b9c3da21ed57c6ba69301",
"empty.html/empty grids do not explode": "423ca653bbcbc21a28029c149a37b8ec"
"empty.html/empty grids do not explode": "423ca653bbcbc21a28029c149a37b8ec",
"superstore.html/highlights invalid filter.": "e244cca8fc2278cb2477d0a46ab5331f"
}
6 changes: 6 additions & 0 deletions packages/perspective-viewer/test/js/simple_tests.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -65,6 +65,12 @@ exports.default = function() {
await page.evaluate(element => element.setAttribute("filters", '[["Sales", ">", 500]]'), viewer);
});

test.capture("highlights invalid filter.", async page => {
const viewer = await page.$("perspective-viewer");
await page.shadow_click("perspective-viewer", "#config_button");
await page.evaluate(element => element.setAttribute("filters", '[["Sales", "==", null]]'), viewer);
});

test.capture("sorts by an alpha column.", async page => {
const viewer = await page.$("perspective-viewer");
await page.shadow_click("perspective-viewer", "#config_button");
Expand Down
3 changes: 2 additions & 1 deletion packages/perspective-viewer/test/results/results.json
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -30,5 +30,6 @@
"superstore.html/doesn't leak views when setting filters.": "0516bfcff10dd15dca9c7d2e4923ab10",
"superstore.html/adds computed column via attribute": "96016d19e8c94accc24f0d613bdbbab1",
"superstore.html/saving without parameters should fail as button is disabled.": "a26ce038cea02a8e3e53b7e0b74a204d",
"superstore.html/saving a duplicate column should fail with error message.": "9d74a59806ad8ec4ab1bcef7c3c94b68"
"superstore.html/saving a duplicate column should fail with error message.": "9d74a59806ad8ec4ab1bcef7c3c94b68",
"superstore.html/highlights invalid filter.": "3eadf1175e231c1fbcc2bb9ef10ca6ef"
}