From 62e215bbd0072b6577643ef270f1687a2ef70c7d Mon Sep 17 00:00:00 2001 From: XiaoSeS <87064762+XiaoSeS@users.noreply.github.com> Date: Tue, 20 Oct 2026 19:06:26 +0800 Subject: [PATCH] fix(ci): prepare Go interop and align review E2E interactions Signed-off-by: XiaoSeS <87064762+XiaoSeS@users.noreply.github.com> --- .github/workflows/pr-tests.yml | 10 ++++++++++ web/e2e/namespace-review-detail-access.spec.ts | 4 ++-- web/e2e/rejected-version-republish.spec.ts | 2 ++ 3 files changed, 14 insertions(+), 2 deletions(-) diff --git a/.github/workflows/pr-tests.yml b/.github/workflows/pr-tests.yml index b1586b072..15e18e5db 100644 --- a/.github/workflows/pr-tests.yml +++ b/.github/workflows/pr-tests.yml @@ -64,6 +64,16 @@ jobs: java-version: 21 cache: maven + - name: Set up Go for integration tests + uses: actions/setup-go@v5 + with: + go-version-file: github-sync-go/go.mod + cache-dependency-path: github-sync-go/go.mod + + - name: Compile Go integration test dependencies + working-directory: github-sync-go + run: go test ./internal/worker -run '^$' + - name: Ensure Maven wrapper is executable run: chmod +x server/mvnw diff --git a/web/e2e/namespace-review-detail-access.spec.ts b/web/e2e/namespace-review-detail-access.spec.ts index 98c6279ba..f66c66029 100644 --- a/web/e2e/namespace-review-detail-access.spec.ts +++ b/web/e2e/namespace-review-detail-access.spec.ts @@ -25,7 +25,7 @@ test.describe('Namespace Review Detail Access (Real API)', () => { await page.getByRole('link', { name: 'Open review' }).first().click() await expect(page).toHaveURL(new RegExp(`/dashboard/namespaces/${seeded.namespace.slug}/reviews/\\d+$`)) - await expect(page.getByRole('heading', { name: 'Review Detail' })).toBeVisible() + await expect(page.getByRole('heading', { level: 1, name: seeded.skill.slug, exact: true })).toBeVisible() await expect(page.getByText(`${seeded.namespace.slug}/${seeded.skill.slug}`).first()).toBeVisible() } finally { await seeded?.cleanup() @@ -59,7 +59,7 @@ test.describe('Namespace Review Detail Access (Real API)', () => { await page.goto(`/dashboard/reviews/${seeded.reviewTaskId}`) await expect(page).toHaveURL(new RegExp(`/dashboard/namespaces/${seeded.namespace.slug}/reviews/${seeded.reviewTaskId}$`)) - await expect(page.getByRole('heading', { name: 'Review Detail' })).toBeVisible() + await expect(page.getByRole('heading', { level: 1, name: seeded.skill.slug, exact: true })).toBeVisible() } finally { await seeded?.cleanup() } diff --git a/web/e2e/rejected-version-republish.spec.ts b/web/e2e/rejected-version-republish.spec.ts index 8f4911e6e..b4037769c 100644 --- a/web/e2e/rejected-version-republish.spec.ts +++ b/web/e2e/rejected-version-republish.spec.ts @@ -219,6 +219,8 @@ test.describe('Rejected version replacement (Real API)', () => { await page.screenshot({ path: testInfo.outputPath('author-review-progress-mobile.png'), fullPage: true }) await adminPage.goto(`/dashboard/reviews/${replacementReviewId}`) + await adminPage.getByRole('tab', { name: 'Review History', exact: true }).click() + await expect(adminPage.getByRole('tab', { name: 'Review History', exact: true })).toHaveAttribute('aria-selected', 'true') await expect(adminPage.getByRole('heading', { name: 'Submission History' })).toBeVisible() await expect(adminPage.getByText('Attempt 2')).toBeVisible() await expect(adminPage.getByText('Attempt 1')).toBeVisible()