Skip to content

Commit 257f603

Browse files
committed
DEV: param-input-test: adds null category_id query param check
This commit includes DRY update: by adding getSearchParam func
1 parent ee13330 commit 257f603

File tree

1 file changed

+11
-10
lines changed

1 file changed

+11
-10
lines changed

test/javascripts/acceptance/param-input-test.js

Lines changed: 11 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -334,15 +334,18 @@ acceptance("Data Explorer Plugin | Param Input", function (needs) {
334334
});
335335
});
336336

337+
function getSearchParam(param) {
338+
const searchParams = new URLSearchParams(currentURL().split("?")[1]);
339+
return JSON.parse(searchParams.get("params"))[param];
340+
}
341+
337342
test("puts params for the query into the url", async function (assert) {
338343
await visit("/admin/plugins/explorer/queries/-6");
339344
const monthsAgoValue = "2";
340345
await fillIn(".query-params input", monthsAgoValue);
341346
await click("form.query-run button");
342347

343-
const searchParams = new URLSearchParams(currentURL().split("?")[1]);
344-
const monthsAgoParam = JSON.parse(searchParams.get("params")).months_ago;
345-
assert.strictEqual(monthsAgoParam, monthsAgoValue);
348+
assert.strictEqual(getSearchParam("months_ago"), monthsAgoValue);
346349
});
347350

348351
test("puts params for the query into the url for group reports", async function (assert) {
@@ -351,9 +354,7 @@ acceptance("Data Explorer Plugin | Param Input", function (needs) {
351354
await fillIn(".query-params input", monthsAgoValue);
352355
await click("form.query-run button");
353356

354-
const searchParams = new URLSearchParams(currentURL().split("?")[1]);
355-
const monthsAgoParam = JSON.parse(searchParams.get("params")).months_ago;
356-
assert.strictEqual(monthsAgoParam, monthsAgoValue);
357+
assert.strictEqual(getSearchParam("months_ago"), monthsAgoValue);
357358
});
358359

359360
test("loads the page if one of the parameter is null", async function (assert) {
@@ -396,14 +397,14 @@ acceptance("Data Explorer Plugin | Param Input", function (needs) {
396397
await visit("/admin/plugins/explorer/queries/4");
397398
assert.strictEqual(catChooser.header().value(), null);
398399

400+
await click("form.query-run button");
401+
assert.strictEqual(getSearchParam("category"), "");
402+
399403
await catChooser.expand();
400404
await catChooser.selectRowByIndex(0);
401405
assert.strictEqual(catChooser.header().label(), selectedCategory.name);
402406

403407
await click("form.query-run button");
404-
405-
const searchParams = new URLSearchParams(currentURL().split("?")[1]);
406-
const categoryIdParam = JSON.parse(searchParams.get("params")).category;
407-
assert.strictEqual(categoryIdParam, selectedCategory.id);
408+
assert.strictEqual(getSearchParam("category"), selectedCategory.id);
408409
});
409410
});

0 commit comments

Comments
 (0)