Dev2025#111
Merged
Merged
Conversation
…g, and overflow visibility
- 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
…flow visibility - getStringVar() now treats empty string the same as undefined, so config.baseURL="" falls back to "/" instead of passing an empty string to passport's successReturnToOrRedirect (which is falsy and skips the post-login redirect, hitting the 404 handler instead) - task.cy.js exclude text box check: overflow:hidden ancestor prevents scrollIntoView from making the element visible; use exist instead of be.visible since the next test proves the input is interactable Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
…test Strip trailing slash from baseURL before URL construction to prevent protocol-relative URLs (//path) when baseURL is "/". Also relax the autocomplete input visibility assertion to exist-only since the element is clipped by overflow:hidden. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Run each Cypress spec file in its own process so Chrome gets a fresh renderer for every spec. Supports --bail/-b to stop at the first failure. Update CI workflow to use the new sequential runner. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Defaulting to BUILDPLATFORM lets Docker use the host's native architecture. Apple Silicon builds get arm64 images and avoid Rosetta 2 emulation, which was causing wheel_auth to time out during test:e2e:mock:start. CI (linux/amd64 runners) is unaffected. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
The Dockerfile installs @rollup/rollup-linux-x64-gnu which requires x64, so linux/amd64 must stay. On Apple Silicon (Rosetta 2 emulation), wheel_auth's passwordDBTool.js startup takes longer than the original 300s limit. Increase wait-on timeout to 600s to accommodate emulation overhead. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
The wheel_auth container was crashing with "remotehost.json not found" because it had no volume mount for its config directory. CI works around this with docker cp, but the local compose file needs an explicit volume. Mount wheel_config_auth/ the same way wheel_release_test mounts wheel_config/. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
mapfile is bash 4+ only; macOS ships bash 3.2. Replace with a while-read loop which works on both 3.2 and 5.x. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Use BUILDPLATFORM instead of hardcoded linux/amd64 so the image builds natively on arm64 hosts. Detect architecture at build time to install the matching Rollup native binary: x86_64 → @rollup/rollup-linux-x64-gnu aarch64 → @rollup/rollup-linux-arm64-gnu CI (ubuntu-latest, x86_64) is unaffected. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
The folder-expansion assertions (checking for run-a.sh / run-b.sh after clicking task1-run in the treeview) use the default 5s timeout, which is too short on slow CI runners where the getFileList socket call takes longer. Use 15s targeted timeout on those three cy.contains() calls. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
getChildren() closed over the `item` arg, so if getComponentDirRootFiles() replaced this.items with new objects while the socket request was in flight, the callback set children on the orphaned old item, leaving the treeview empty. Fix by re-looking up the item by ID in this.items at callback time. Also change the early-return null guard from a silent promise-hang to resolve(), and add a missing return after reject(). In innerTreeview.vue the folder icon had both v-bind="props" (which binds props.onClick) and an explicit props.onClick(e) call in @click.stop. Vue 3 merges these, causing props.onClick to fire twice, toggling the group open then immediately closed. Remove the duplicate call from @click.stop. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
fix E2E test