Skip to content

Commit 1da421a

Browse files
pmaxhoganclaude
andcommitted
fix(ui): prettier-format the new #309 debug-logging tests
PR #314's "ui build + lint + unit" CI job failed in 38s at the prettier --check step (after pnpm install + lint, before test:unit and build): activity-debug-chip.test.ts and settings-pages.test.ts were hand-typed and not run through Prettier before the previous commit. No logic change, whitespace/wrapping only - re-verified lint, format:check, test:unit (793 passed), and build all pass locally in the workflow's exact step order. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_019xKUm9vH4ifb5LHR5szy1v
1 parent fb2686b commit 1da421a

2 files changed

Lines changed: 17 additions & 8 deletions

File tree

ui/src/__tests__/activity-debug-chip.test.ts

Lines changed: 11 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -81,7 +81,14 @@ function stubBackend(debugLoggingEnabled: boolean): void {
8181
case "get_settings":
8282
return makeSettings(debugLoggingEnabled);
8383
case "query_activity":
84-
return { entries: [], total: 0, limit: 100, hasMore: false, nextBeforeTs: null, nextBeforeId: null };
84+
return {
85+
entries: [],
86+
total: 0,
87+
limit: 100,
88+
hasMore: false,
89+
nextBeforeTs: null,
90+
nextBeforeId: null,
91+
};
8592
case "distinct_activity_event_types":
8693
return ["upload_done"];
8794
case "activity_summary":
@@ -127,7 +134,9 @@ describe("Activity debug-data chip (issue #309)", () => {
127134
const chip = wrapper.get(CHIP);
128135
expect(chip.text()).toBe(i18n.global.t("activity.debugDataIncludedChip"));
129136
// Sits next to the export button (same flex row).
130-
expect(chip.element.parentElement?.querySelector('[data-testid="activity-export-bundle"]')).toBeTruthy();
137+
expect(
138+
chip.element.parentElement?.querySelector('[data-testid="activity-export-bundle"]')
139+
).toBeTruthy();
131140

132141
wrapper.unmount();
133142
});

ui/src/__tests__/settings-pages.test.ts

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -1537,9 +1537,9 @@ describe("PrivacyPage", () => {
15371537
});
15381538
// The backend-computed expiry round-trips back into the toggle's checked
15391539
// state (the store replaces the snapshot with the command's response).
1540-
expect((wrapper.get('[data-testid="debug-logging-toggle"]').element as HTMLInputElement).checked).toBe(
1541-
true
1542-
);
1540+
expect(
1541+
(wrapper.get('[data-testid="debug-logging-toggle"]').element as HTMLInputElement).checked
1542+
).toBe(true);
15431543
});
15441544

15451545
it("issue #309: while debug logging is on, the bundle note shows the persisted expiry", async () => {
@@ -1560,9 +1560,9 @@ describe("PrivacyPage", () => {
15601560
const wrapper = mount(PrivacyPage, { global: globalMountOptions });
15611561
await flushPromises();
15621562

1563-
expect((wrapper.get('[data-testid="debug-logging-toggle"]').element as HTMLInputElement).checked).toBe(
1564-
true
1565-
);
1563+
expect(
1564+
(wrapper.get('[data-testid="debug-logging-toggle"]').element as HTMLInputElement).checked
1565+
).toBe(true);
15661566
expect(wrapper.get('[data-testid="debug-logging-setting"]').text()).toContain(
15671567
i18n.global.t("settings.rules.debugLogging.includeInBundle")
15681568
);

0 commit comments

Comments
 (0)