Actions
Bug #23080
closed
Process Output Params test failing
Status:
Resolved
Priority:
Normal
Assigned To:
Category:
Workbench
Target version:
Story points:
-
Release:
Release relationship:
Auto
Description
In the course of #23063, it was discovered that several e2e tests were being skipped due to a stray `.only()` left in by mistake. Upon removing it, the Process I/O panel tests fail (cypress/e2e/process.cy.js, starting @ L1417). I attempted to find a quick fix and failed to do so, so I commented out the test and moved on. The test has been skipped for over a month, so it's not super critical.
Updated by Brett Smith 7 months ago
- Target version set to Development 2025-08-21
- Assigned To set to Lisa Knox
Updated by Lisa Knox 7 months ago
23080-params-test @ 053351c94fff9ffeb5e6fe5c01648dfd05e391bd
developer-run-tests-services-workbench2: #1576
- ✅ All agreed upon points are implemented / addressed.
- The issue was that tests adjusted in #23063, which was merged when this test was being skipped, changed a cypress createContainerRequest call so that the output values were overwritten. Solved by making a separate createContainerRequest call for that specific test.
- n/a Anything not implemented (discovered or discussed during work) has a follow-up story.
- ✅ Code is tested and passing, both automated and manual, what manual testing was done is described
- ✅ The tested code incorporates recent main branch changes.
- n/a New or changed UX/UX and has gotten feedback from stakeholders.
- n/a Documentation has been updated.
- ✅ Behaves appropriately at the intended scale (describe intended scale).
- ✅ Considered backwards and forwards compatibility issues between client and server.
- ✅ Follows our coding standards and GUI style guidelines.
Updated by
Updated by Lisa Knox 7 months ago
- Status changed from In Progress to Resolved
Applied in changeset arvados|199b1e22ed945d51b6180abb947b9ab784102dae.
Actions