Skip to content

Commit b57c8bb

Browse files
committed
update test names
1 parent 74ec14f commit b57c8bb

File tree

3 files changed

+9
-17
lines changed

3 files changed

+9
-17
lines changed

plugins/wpgraphql-logging/tests/e2e/specs/basic-usage.spec.js

Lines changed: 4 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -8,12 +8,12 @@ import {
88
} from "../utils";
99
import { GET_POSTS_QUERY } from "../constants";
1010

11-
test.describe("Configure WPGraphQL Logging Plugin and Verify Logging Works", () => {
11+
test.describe("Basic Logging Usage", () => {
1212
test.beforeEach(async ({ admin }) => {
1313
await resetPluginSettings(admin);
1414
});
1515

16-
test("should enable logging and verify GraphQL queries are being logged", async ({
16+
test("enables logging and logs GraphQL queries", async ({
1717
page,
1818
admin,
1919
request,
@@ -70,11 +70,7 @@ test.describe("Configure WPGraphQL Logging Plugin and Verify Logging Works", ()
7070
await expect(page.locator("h1")).toContainText("WPGraphQL Logs");
7171
});
7272

73-
test("should not log queries when logging is disabled", async ({
74-
page,
75-
admin,
76-
request,
77-
}) => {
73+
test("does not log when disabled", async ({ page, admin, request }) => {
7874
// Make sure there are no logs
7975
await goToLogsListPage(admin);
8076
await expect(
@@ -97,7 +93,7 @@ test.describe("Configure WPGraphQL Logging Plugin and Verify Logging Works", ()
9793
).toBeVisible();
9894
});
9995

100-
test("should download the log as CSV and verify the content", async ({
96+
test("downloads log as CSV with correct content", async ({
10197
page,
10298
admin,
10399
request,

plugins/wpgraphql-logging/tests/e2e/specs/data-cleanup.spec.js

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -6,12 +6,12 @@ import {
66
switchToSettingsTab,
77
} from "../utils";
88

9-
test.describe("Data Management - Configure Automatic Cleanup", () => {
9+
test.describe("Data Management", () => {
1010
test.beforeEach(async ({ admin }) => {
1111
await resetPluginSettings(admin);
1212
});
1313

14-
test("should configure data deletion settings and verify they are saved", async ({
14+
test("configures data deletion and verifies cron job", async ({
1515
page,
1616
admin,
1717
}) => {

plugins/wpgraphql-logging/tests/e2e/specs/exclude-and-sanitize.spec.js

Lines changed: 3 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -9,12 +9,12 @@ import {
99
} from "../utils";
1010
import { GET_POSTS_QUERY } from "../constants";
1111

12-
test.describe("Exclude Sensitive Queries from Logging", () => {
12+
test.describe("Query Filtering & Data Privacy", () => {
1313
test.beforeEach(async ({ admin }) => {
1414
await resetPluginSettings(admin);
1515
});
1616

17-
test("should exclude queries from logs when configured", async ({
17+
test("excludes configured queries from logs", async ({
1818
page,
1919
admin,
2020
request,
@@ -45,11 +45,7 @@ test.describe("Exclude Sensitive Queries from Logging", () => {
4545
).toBeVisible();
4646
});
4747

48-
test("should sanitize sensitive data in logs when sanitization is enabled", async ({
49-
page,
50-
admin,
51-
request,
52-
}) => {
48+
test("sanitizes sensitive data in logs", async ({ page, admin, request }) => {
5349
// Set up logging settings and execute a GraphQL query
5450
await goToLoggingSettingsPage(admin);
5551
await expect(page.locator("h1")).toHaveText("WPGraphQL Logging Settings");

0 commit comments

Comments
 (0)