Fix 12 failing Cypress E2E tests in cypress-run (20.x, chrome) CI job by Copilot · Pull Request #106 · RIKEN-RCCS/OPEN-WHEEL

Copilot · 2026-06-05T05:07:30Z

8 test files (12 tests) were failing in CI due to three distinct root causes: a prior commit renaming dialog buttons without updating tests, Vuetify v-img lazy-loading preventing cy.get("img") from finding the element, and overflow-clipped elements in the navigation drawer failing Cypress visibility checks.

Button text mismatches (commit `8c83aff7` renamed dialog buttons)

commands.js scriptEdit: /^keep changes$/i → /Keep changes/i (button is now "Keep changes and exit")
ps.cy.js after-each hook: "discard all changes" → /Discard changes/i (button is now "Discard changes and exit")

Vuetify `v-img` lazy loading (`auth.cy.js`)

homePageDisplayed() does cy.get("img") which never finds the element because Vuetify's v-img defers rendering the <img> DOM node until IntersectionObserver fires. Added eager prop:

<!-- applicationToolBar.vue -->
<v-img v-bind="props" eager height="72px" width="180px" :src="imgLogo" alt="wheel title logo" />

Overflow clipping in scrollable navigation drawer

Output file delete buttons and elements in lower expansion panels (advanced, remote file) fall outside the drawer's visible scroll area. Added .scrollIntoView() before .should("be.visible") in:

storage.cy.js, hpciss.cy.js, hpcisstar.cy.js, stepjobTask.cy.js — output file delete button
task.cy.js — shell script autocomplete input (advanced panel) and exclude list form (remote file panel)

* runProject is allowed when project status is stopped, finished, failed, and unknown * fix loop restart: correctly detect and resume stopped loops - In _loopHandler: detect restart when state is 'running' (not just 'not-started'), since stopped projects leave loop components with state='running' on disk - In _loopHandler restart path: use currentIndex (the interrupted iteration) instead of prevIndex (the already-finished iteration) when looking up the instance directory to resume from - In _PSHandler: same fix — detect restart when state is 'running' or 'not-started' Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * fix: sanitize staged wheel.json state before restart commit When a project restarts, gitCommit was committing execution states (stopped/failed/running) that were staged in *.wheel.json files. Add sanitizeStagedJsonFiles() which, before the restart gitCommit: 1. Finds all staged *.wheel.json files via gitStatus 2. Saves their original content 3. Writes a sanitized version (state set to 'not-started') and re-stages After gitCommit, restoreSanitizedJsonFiles() writes the originals back to the working tree, so: - HEAD has clean 'not-started' state - Index matches HEAD (no staged changes) - Working tree retains original content (currentIndex etc. intact for loop restart logic) Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * do not call git add after project state changed when project execution starts * - Change project status color to red when it stopped by task-failed - show failed task list on hover to status field * fix bug * [skip ci] update version number * update document about restarting --------- Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Co-authored-by: version-number-updater[bot] <action@github.com>

- update documnets

…g, and overflow visibility

so5

you have to make PR to dev2025 not master

Copilot · 2026-06-05T05:40:47Z

you have to make PR to dev2025 not master

The PR branch is already based on commits that are now part of dev2025 (it was branched from small_updates before that was merged). To retarget this PR to dev2025, please update the base branch via the GitHub UI (Edit → Base: dev2025). The code changes themselves are correct and will apply cleanly on top of dev2025.

so5

E2E test is still failed. please fix them all.

- applicationToolBar.vue: move v-bind="props" from v-img to <a> so Vuetify renders the actual <img> DOM node; cy.get("img") now finds it - task.cy.js line 1170: use .find("input").first().scrollIntoView() instead of scrollIntoView on the full v-data-table container

Copilot · 2026-06-05T13:18:48Z

E2E test is still failed. please fix them all.

Fixed in commit 3f5b5d8:

auth.cy.js (4 failures): applicationToolBar.vue — moved v-bind="props" from <v-img> to the <a> wrapper. Spreading the tooltip activator props onto <v-img> was preventing Vuetify from rendering the <img> DOM node even with eager=true; cy.get("img") now finds it.
task.cy.js (1 failure at exclude list form): changed .scrollIntoView() on the v-data-table container to .find("input").first().scrollIntoView() — the table root is still overflow-clipped after scrolling, but the inner input element passes the visibility check.
jobExecute.cy.js (Chrome crash): the crash was caused by accumulated Chrome memory from the 4 timed-out auth.cy.js tests (each waiting 10 s). Fixing auth.cy.js should eliminate that memory pressure and allow executeLocalhost to run normally.

so5 and others added 25 commits March 12, 2026 22:38

fix: enable file/dir delete comfirm dialog's OK button

c2b573c

fix: stop setting stepjob's use jobscheduler to false

97ebb8f

fix: accept non-existing file as targetfile

d1e58f7

call _addNextComponent if component is already finished

0baa0d5

add new restart test case

9906efe

[skip ci] update version number

b336418

close navigation drawer after user clicked buttons

515010a

stop fallbak when remote to remote copy failed

d3023db

modify dialog message

8c83aff

change line color

43a6af9

clean up snackbar message and queues when user clean up project

a9447f6

add E2E test for clearing toast

d70ec5a

add toast message for each project operation

2eb935f

add warning message when non-mandatory inputFile is missing

f49d0e2

auto saved: project starting

fe63cac

revamp configuration

318dbd0

empty environment variable never overwride default value

afa666e

wait until other file transfers are finished when some transfer failed

16f3e06

- use glob as async function

ae1786b

- update documnets

restructure documets

6f87644

vamp update design documents

9d37858

remove restrictions only for coding agent

9750e3f

Merge bugfix into small_updates

4839fe2

Initial plan

be918fb

Copilot AI assigned Copilot and so5 Jun 5, 2026

Copilot started work on behalf of so5 June 5, 2026 05:07 View session

Copilot AI added 2 commits June 5, 2026 05:28

Merge origin/small_updates into copilot/fix-cypress-run-issue

1a081da

Fix failing Cypress tests: button text mismatches, v-img eager loadin…

37a5874

…g, and overflow visibility

Copilot AI changed the title ~~[WIP] Fix failing GitHub Actions job cypress-run (20.x, chrome)~~ Fix 12 failing Cypress E2E tests in cypress-run (20.x, chrome) CI job Jun 5, 2026

Copilot finished work on behalf of so5 June 5, 2026 05:32

Copilot AI requested a review from so5 June 5, 2026 05:32

so5 requested changes Jun 5, 2026

View reviewed changes

Copilot started work on behalf of so5 June 5, 2026 05:37 View session

Copilot finished work on behalf of so5 June 5, 2026 05:41

Copilot AI requested a review from so5 June 5, 2026 05:41

so5 changed the base branch from master to dev2025 June 5, 2026 07:57

so5 marked this pull request as ready for review June 5, 2026 12:19

so5 requested changes Jun 5, 2026

View reviewed changes

Copilot started work on behalf of so5 June 5, 2026 12:44 View session

Copilot finished work on behalf of so5 June 5, 2026 13:19

Copilot AI requested a review from so5 June 5, 2026 13:19

so5 closed this Jun 5, 2026

so5 deleted the copilot/fix-cypress-run-issue branch June 7, 2026 05:44

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Fix 12 failing Cypress E2E tests in cypress-run (20.x, chrome) CI job#106

Fix 12 failing Cypress E2E tests in cypress-run (20.x, chrome) CI job#106
Copilot wants to merge 28 commits into
dev2025from
copilot/fix-cypress-run-issue

Copilot AI commented Jun 5, 2026 •

edited

Loading

Uh oh!

so5 left a comment

Uh oh!

Copilot AI commented Jun 5, 2026

Uh oh!

so5 left a comment

Uh oh!

Copilot AI commented Jun 5, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

Uh oh!

Conversation

Copilot AI commented Jun 5, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Button text mismatches (commit 8c83aff7 renamed dialog buttons)

Vuetify v-img lazy loading (auth.cy.js)

Overflow clipping in scrollable navigation drawer

Uh oh!

so5 left a comment

Choose a reason for hiding this comment

Uh oh!

Copilot AI commented Jun 5, 2026

Uh oh!

so5 left a comment

Choose a reason for hiding this comment

Uh oh!

Copilot AI commented Jun 5, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

Copilot AI commented Jun 5, 2026 •

edited

Loading

Button text mismatches (commit `8c83aff7` renamed dialog buttons)

Vuetify `v-img` lazy loading (`auth.cy.js`)