fix(ci): unskip visual tests and align node versions

- Unskip `mobile sidebar shares desktop rail behavior` in `tests/visual/overlay-layers.spec.ts`.
- Enable visibility check in `tests/visual/moderation.spec.ts` (removed TODO).
- Update `daily-design-system-audit.yml` and `agent-pr-review.yml` to use Node.js 22.
- Set `fetch-depth: 0` in `ci.yml` build job to ensure `sw-compat` lint check runs.

Co-authored-by: PR0M3TH3AN <[email protected]>
This commit is contained in:
google-labs-jules[bot]andPR0M3TH3AN committed 2026-02-11 22:41:18 +00:00
1 parent 37b1f48f1a
commit 3571b52b13
5 files changed
+6 -6

No files matched your search

+1 -1
View File
@@ -26,7 +26,7 @@ jobs:
- name: Set up Node.js
uses: actions/setup-node@v4
with:
node-version: 20
node-version: 22
cache: 'npm'
- name: Add upstream remote
+2
View File
@@ -22,6 +22,8 @@ jobs:
steps:
- name: Check out repository
uses: actions/checkout@v4
with:
fetch-depth: 0
- name: Set up Node.js ${{ matrix.node-version }}
uses: actions/setup-node@v4
@@ -23,7 +23,7 @@ jobs:
- name: Setup Node.js
uses: actions/setup-node@v4
with:
node-version: '20'
node-version: '22'
cache: 'npm'
- name: Verify gh CLI availability
+1 -3
View File
@@ -211,9 +211,7 @@ test.describe("moderation fixtures", () => {
await expect(muteCard).toHaveAttribute("data-moderation-trusted-mute-count", "1");
await expect(badge).toContainText("Hidden · 1 trusted mute");
await expect(thumbnail).toHaveAttribute("data-thumbnail-state", "blurred");
// TODO: Visibility check fails in headless environment due to layout issues (0x0 size).
// Verifying existence via selector instead of role/visibility.
// await expect(showAnywayButton).toBeVisible();
await expect(showAnywayButton).toBeVisible();
const showAnywayLocator = muteCard.locator('button[data-moderation-action="override"]');
await expect(showAnywayLocator).toHaveCount(1);
await expect(showAnywayLocator).toHaveAttribute("aria-label", "Show anyway");
+1 -1
View File
@@ -10,7 +10,7 @@ declare global {
}
test.describe("overlay layering tokens", () => {
test.skip("mobile sidebar shares desktop rail behavior", async ({ page }) => {
test("mobile sidebar shares desktop rail behavior", async ({ page }) => {
// Increase timeout for this visual test which is prone to flakiness in CI
test.setTimeout(120_000);