Skip to content

Commit 82c84a9

Browse files
disnetclaude
andcommitted
e2e: update for Everything→Feeds rename and /home, /feeds, /saved routes
The home-view PR renamed the sidebar's 'Everything' nav to 'Feeds' and moved the app off query-param routing (/?saved=true) onto real routes (/home, /feeds, /saved). Update the navigation, channels, and custom-fields specs to match. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
1 parent 7083deb commit 82c84a9

3 files changed

Lines changed: 14 additions & 14 deletions

File tree

e2e/channels.spec.ts

Lines changed: 10 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -1,14 +1,14 @@
11
import { test, expect } from './fixtures';
22
import type { Page } from '@playwright/test';
33

4-
/** Create a source channel via the Everything row's + button and modal form. */
4+
/** Create a source channel via the Feeds row's + button and modal form. */
55
async function createChannel(page: Page, name: string) {
6-
// Channels live under the Everything nav row. Click the + add button on that row.
7-
const everythingRow = page.locator('.nav-row', {
8-
has: page.locator('.nav-label', { hasText: 'Everything' }),
6+
// Channels live under the Feeds nav row. Click the + add button on that row.
7+
const feedsRow = page.locator('.nav-row', {
8+
has: page.locator('.nav-label', { hasText: 'Feeds' }),
99
});
10-
await expect(everythingRow).toBeVisible({ timeout: 15_000 });
11-
await everythingRow.locator('.row-add-btn').click({ force: true });
10+
await expect(feedsRow).toBeVisible({ timeout: 15_000 });
11+
await feedsRow.locator('.row-add-btn').click({ force: true });
1212

1313
// Fill the channel name in the modal
1414
const nameInput = page.locator('#view-name');
@@ -24,8 +24,8 @@ async function createChannel(page: Page, name: string) {
2424
}
2525

2626
test.describe('Channels', () => {
27-
test('sidebar shows Everything section', async ({ authedPage }) => {
28-
await expect(authedPage.locator('.nav-label', { hasText: 'Everything' })).toBeVisible({
27+
test('sidebar shows Feeds section', async ({ authedPage }) => {
28+
await expect(authedPage.locator('.nav-label', { hasText: 'Feeds' })).toBeVisible({
2929
timeout: 15_000,
3030
});
3131
});
@@ -102,8 +102,8 @@ test.describe('Channels', () => {
102102
});
103103

104104
// Navigate away first
105-
await authedPage.locator('.nav-label', { hasText: 'Everything' }).click();
106-
await expect(authedPage).toHaveURL(/\/$/);
105+
await authedPage.locator('.nav-label', { hasText: 'Feeds' }).click();
106+
await expect(authedPage).toHaveURL(/\/feeds/);
107107

108108
// Click the channel to navigate to it
109109
await viewItem.click();

e2e/custom-fields.spec.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -60,9 +60,9 @@ test.describe('Custom Fields', () => {
6060

6161
// Reload and verify persistence (round-trips through D1 → backend → IndexedDB).
6262
// Reload lands on /sources which doesn't initialize the store — bounce through
63-
// '/' to trigger appManager.initialize() and re-sync subscriptions from backend.
63+
// /feeds to trigger appManager.initialize() and re-sync subscriptions from backend.
6464
await authedPage.reload();
65-
await authedPage.locator('.nav-label', { hasText: 'Everything' }).click();
65+
await authedPage.locator('.nav-label', { hasText: 'Feeds' }).click();
6666
await authedPage.locator('.nav-label', { hasText: 'Manage Sources' }).click();
6767
await expect(sourceRowByTitle(authedPage, customTitle)).toBeVisible({
6868
timeout: 15_000,

e2e/navigation.spec.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@ import { test, expect } from './fixtures';
22

33
test.describe('Navigation', () => {
44
test('sidebar shows all nav items', async ({ authedPage }) => {
5-
await expect(authedPage.locator('.nav-label', { hasText: 'Everything' })).toBeVisible({
5+
await expect(authedPage.locator('.nav-label', { hasText: 'Feeds' })).toBeVisible({
66
timeout: 15_000,
77
});
88
await expect(authedPage.locator('.nav-label', { hasText: 'Saved' })).toBeVisible();
@@ -12,7 +12,7 @@ test.describe('Navigation', () => {
1212

1313
test('clicking Saved sets filter', async ({ authedPage }) => {
1414
await authedPage.locator('.nav-label', { hasText: 'Saved' }).click();
15-
await expect(authedPage).toHaveURL(/saved=true/);
15+
await expect(authedPage).toHaveURL(/\/saved/);
1616
});
1717

1818
test('Settings link navigates to /settings', async ({ authedPage }) => {

0 commit comments

Comments
 (0)