Skip to content
This repository was archived by the owner on Jul 14, 2025. It is now read-only.

Commit 7f922c9

Browse files
committed
DEV: acceptance tests cleanup: removes unused lines of code
1 parent f12e3ba commit 7f922c9

File tree

2 files changed

+6
-147
lines changed

2 files changed

+6
-147
lines changed

test/javascripts/acceptance/list-queries-test.js

Lines changed: 1 addition & 72 deletions
Original file line numberDiff line numberDiff line change
@@ -9,78 +9,7 @@ acceptance("Data Explorer Plugin | List Queries", function (needs) {
99

1010
needs.pretender((server, helper) => {
1111
server.get("/admin/plugins/explorer/groups.json", () => {
12-
return helper.response([
13-
{
14-
id: 1,
15-
name: "admins",
16-
},
17-
{
18-
id: 2,
19-
name: "moderators",
20-
},
21-
{
22-
id: 3,
23-
name: "staff",
24-
},
25-
{
26-
id: 0,
27-
name: "everyone",
28-
},
29-
{
30-
id: 10,
31-
name: "trust_level_0",
32-
},
33-
{
34-
id: 11,
35-
name: "trust_level_1",
36-
},
37-
{
38-
id: 12,
39-
name: "trust_level_2",
40-
},
41-
{
42-
id: 13,
43-
name: "trust_level_3",
44-
},
45-
{
46-
id: 14,
47-
name: "trust_level_4",
48-
},
49-
]);
50-
});
51-
52-
server.get("/admin/plugins/explorer/schema.json", () => {
53-
return helper.response({
54-
anonymous_users: [
55-
{
56-
column_name: "id",
57-
data_type: "serial",
58-
primary: true,
59-
},
60-
{
61-
column_name: "user_id",
62-
data_type: "integer",
63-
fkey_info: "users",
64-
},
65-
{
66-
column_name: "master_user_id",
67-
data_type: "integer",
68-
fkey_info: "users",
69-
},
70-
{
71-
column_name: "active",
72-
data_type: "boolean",
73-
},
74-
{
75-
column_name: "created_at",
76-
data_type: "timestamp",
77-
},
78-
{
79-
column_name: "updated_at",
80-
data_type: "timestamp",
81-
},
82-
],
83-
});
12+
return helper.response([]);
8413
});
8514

8615
server.get("/admin/plugins/explorer/queries", () => {

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

Lines changed: 5 additions & 75 deletions
Original file line numberDiff line numberDiff line change
@@ -8,82 +8,11 @@ acceptance("Data Explorer Plugin | Param Input", function (needs) {
88

99
needs.pretender((server, helper) => {
1010
server.get("/admin/plugins/explorer/groups.json", () => {
11-
return helper.response([
12-
{
13-
id: 1,
14-
name: "admins",
15-
},
16-
{
17-
id: 2,
18-
name: "moderators",
19-
},
20-
{
21-
id: 3,
22-
name: "staff",
23-
},
24-
{
25-
id: 0,
26-
name: "everyone",
27-
},
28-
{
29-
id: 10,
30-
name: "trust_level_0",
31-
},
32-
{
33-
id: 11,
34-
name: "trust_level_1",
35-
},
36-
{
37-
id: 12,
38-
name: "trust_level_2",
39-
},
40-
{
41-
id: 13,
42-
name: "trust_level_3",
43-
},
44-
{
45-
id: 14,
46-
name: "trust_level_4",
47-
},
48-
{
49-
id: 41,
50-
name: "discourse",
51-
},
52-
]);
11+
return helper.response([]);
5312
});
5413

5514
server.get("/admin/plugins/explorer/schema.json", () => {
56-
return helper.response({
57-
anonymous_users: [
58-
{
59-
column_name: "id",
60-
data_type: "serial",
61-
primary: true,
62-
},
63-
{
64-
column_name: "user_id",
65-
data_type: "integer",
66-
fkey_info: "users",
67-
},
68-
{
69-
column_name: "master_user_id",
70-
data_type: "integer",
71-
fkey_info: "users",
72-
},
73-
{
74-
column_name: "active",
75-
data_type: "boolean",
76-
},
77-
{
78-
column_name: "created_at",
79-
data_type: "timestamp",
80-
},
81-
{
82-
column_name: "updated_at",
83-
data_type: "timestamp",
84-
},
85-
],
86-
});
15+
return helper.response({});
8716
});
8817

8918
server.get("/admin/plugins/explorer/queries", () => {
@@ -353,7 +282,8 @@ acceptance("Data Explorer Plugin | Param Input", function (needs) {
353282
return helper.response({
354283
query: {
355284
id: 3,
356-
sql: "-- [params]\n-- int :months_ago = 1\n\nSELECT 1",
285+
// sql: "-- [params]\n-- int :months_ago = 1\n\nSELECT 1",
286+
sql: "-- [params]\n-- null category_id :cat_id\n-- int :months_ago = 1\n\nSELECT 1",
357287
name: "Params test",
358288
description: "test for params.",
359289
param_info: [
@@ -450,11 +380,11 @@ acceptance("Data Explorer Plugin | Param Input", function (needs) {
450380
await visit("/admin/plugins/explorer/queries/3");
451381
assert.dom(".query-params input").doesNotExist();
452382
await click(".query-edit .btn-edit-query");
453-
await click(".query-editor .ace_text-input");
454383
await fillIn(
455384
".query-editor .ace_text-input",
456385
"-- [params]\n-- int :months_ago = 1\n\nSELECT 1"
457386
);
387+
await click(".query-editor .ace_text-input"); // enables `Save Changes` button
458388
await click(".query-edit .btn-save-query");
459389
assert.dom(".query-params input").exists();
460390
});

0 commit comments

Comments
 (0)