Skip to content

Commit caf7c7c

Browse files
kriszypclaude
andcommitted
fix(test): configuration suite — remove unused admin variable, prettier
Removes unused `admin` / `ctx.harper.admin` assignment that triggered the oxlint no-unused-vars error. Also apply prettier formatting. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
1 parent da175a6 commit caf7c7c

1 file changed

Lines changed: 3 additions & 15 deletions

File tree

integrationTests/apiTests/configuration.test.mjs

Lines changed: 3 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -29,12 +29,10 @@ const TEST_PASS = 'TestPassword123!';
2929

3030
suite('Configuration', (ctx) => {
3131
let client;
32-
let admin;
3332

3433
before(async () => {
3534
await startHarper(ctx, { config: {}, env: {} });
3635
client = createApiClient(ctx.harper);
37-
admin = ctx.harper.admin;
3836

3937
// Seed schema and the AttributeDropTest table with a pre-existing attribute
4038
// so the "Drop Attribute" test has something to drop.
@@ -73,11 +71,7 @@ suite('Configuration', (ctx) => {
7371
.req()
7472
.send({ operation: 'create_attribute', schema: SCHEMA, table: ATTR_TEST_TABLE, attribute: 'owner_id' })
7573
.expect((r) =>
76-
assert.equal(
77-
r.body.message,
78-
`attribute '${SCHEMA}.${ATTR_TEST_TABLE}.owner_id' successfully created.`,
79-
r.text
80-
)
74+
assert.equal(r.body.message, `attribute '${SCHEMA}.${ATTR_TEST_TABLE}.owner_id' successfully created.`, r.text)
8175
)
8276
.expect(200);
8377
});
@@ -183,9 +177,7 @@ suite('Configuration', (ctx) => {
183177
table: DROP_ATTR_TABLE,
184178
attribute: 'another_attribute',
185179
})
186-
.expect((r) =>
187-
assert.equal(r.body.message, "successfully deleted attribute 'another_attribute'", r.text)
188-
)
180+
.expect((r) => assert.equal(r.body.message, "successfully deleted attribute 'another_attribute'", r.text))
189181
.expect(200);
190182
});
191183

@@ -258,11 +250,7 @@ suite('Configuration', (ctx) => {
258250
.req()
259251
.send({ operation: 'set_configuration', http_cors: 'spinach' })
260252
.expect((r) =>
261-
assert.equal(
262-
r.body.error,
263-
"Harper config file validation error: 'http.cors' must be a boolean",
264-
r.text
265-
)
253+
assert.equal(r.body.error, "Harper config file validation error: 'http.cors' must be a boolean", r.text)
266254
)
267255
.expect(400);
268256
});

0 commit comments

Comments
 (0)