scriptoriumadmin/GitNexus
1
0
Fork 0
mirror of https://github.com/abhigyanpatwari/GitNexus.git synced 2026-09-29 01:41:42 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions 7

fix(e2e): stabilize chromium specs after RepoLanding rework

Browse source
This commit is contained in:
sahal 2026-04-28 11:21:21 -07:00
parent c78668aba8
commit fa115b8203
2 changed files with 29 additions and 18 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

20
gitnexus-web/e2e/onboarding.spec.ts
View file

@ -269,15 +269,18 @@ test.describe('Flow 4: Repo dropdown in exploring view', () => {
}
});
// The project badge is the only header button containing the pulsing
// status-dot span (`bg-node-function` + `animate-pulse`). Selecting by this
// distinguishing child is more robust than `.first()` on `header button:has(svg)`,
// which silently drifts whenever a new icon-bearing button is added to the header.
const PROJECT_BADGE = 'header button:has(span.bg-node-function.animate-pulse)';
test('project badge opens repo dropdown', async ({ page }, testInfo) => {
await enterExploringView(page);
await page.screenshot({ path: testInfo.outputPath('exploring-loaded.png') });
// Click the project badge (has a chevron)
const badge = page
.locator('header button')
.filter({ has: page.locator('svg') })
.first();
const badge = page.locator(PROJECT_BADGE);
await expect(badge).toBeVisible();
await badge.click();
// Repo dropdown should be visible
@ -289,11 +292,8 @@ test.describe('Flow 4: Repo dropdown in exploring view', () => {
test('analyze option opens inline form', async ({ page }, testInfo) => {
await enterExploringView(page);
// Open repo dropdown
const badge = page
.locator('header button')
.filter({ has: page.locator('svg') })
.first();
const badge = page.locator(PROJECT_BADGE);
await expect(badge).toBeVisible();
await badge.click();
// Click "Analyze a new repository..."

27
gitnexus-web/e2e/repo-switching.spec.ts
View file

@ -122,8 +122,12 @@ test.describe('?project= URL persistence', () => {
timeout: READY_TIMEOUT_MS,
});
const url = new URL(page.url());
expect(url.searchParams.get('project')).toBeTruthy();
// Poll the URL — under parallel CI workers, history.replaceState() can
// briefly trail the status-ready render. A single read may observe the
// pre-replaceState URL, even though the state update is in flight.
await expect
.poll(() => new URL(page.url()).searchParams.get('project'), { timeout: 5_000 })
.toBeTruthy();
});
});
@ -157,8 +161,13 @@ test.describe('Windows path normalization', () => {
test('project name uses basename when /api/repo returns a Windows-style repoPath', async ({
page,
}) => {
const repoName = firstRepoName || 'test-repo';
const windowsPath = `C:\\Users\\LENOVO\\.gitnexus\\repos\\${repoName}`;
// Use the last `/`-separated segment so the path's tail is a single segment
// even if the live backend's first repo name happens to contain a `/`. The
// production code splits on both `\` and `/`, so a slashed name would otherwise
// make `expect(project).toBe(repoName)` compare against a multi-segment name
// while the URL only carries the final segment.
const baseName = (firstRepoName || 'test-repo').split('/').pop() || 'test-repo';
const windowsPath = `C:\\Users\\LENOVO\\.gitnexus\\repos\\${baseName}`;
// Mock /api/repo to return a Windows backslash path while keeping name correct
await page.route(/\/api\/repo(?!s)(\?.*)?$/, (route) =>
@ -179,11 +188,13 @@ test.describe('Windows path normalization', () => {
});
// URL ?project= must be the short basename, NOT the full Windows path
const url = new URL(page.url());
const project = url.searchParams.get('project');
expect(project).toBeTruthy();
await expect
.poll(() => new URL(page.url()).searchParams.get('project'), { timeout: 5_000 })
.toBeTruthy();
const project = new URL(page.url()).searchParams.get('project');
expect(project).not.toContain('\\');
expect(project).not.toContain('LENOVO');
expect(project).toBe(repoName);
expect(project).toBe(baseName);
});
});