Skip to content

test(pd-e2e): PD E2E added applitools eyes to Mix Form in PD#21212

Merged
skowalski08 merged 3 commits intochore_release-pd-8.10.0from
PD-810-add-eyes-mix
Apr 7, 2026
Merged

test(pd-e2e): PD E2E added applitools eyes to Mix Form in PD#21212
skowalski08 merged 3 commits intochore_release-pd-8.10.0from
PD-810-add-eyes-mix

Conversation

@skowalski08
Copy link
Copy Markdown
Contributor

Overview

Addresses https://opentrons.atlassian.net/browse/RQA-5266
Added applitools eyes check to the Mix Step form and associated modals

Changelog

test_pd_mix_step.py

  • removed verifications
  • added applitools eyes
  • removed import_protocol_and_open_editor helper and dismiss_migration_modal functionm in favor of a reusable function imported from utility

utility.py

  • renamed import_protocol_and_open_editor helper to address an old comment from a previous PR regarding the use of a leading _, and updated all affected files

Risk assessment

-Low risk

@skowalski08 skowalski08 requested review from alexjoel42 and y3rsh April 6, 2026 20:46
Copy link
Copy Markdown
Member

@y3rsh y3rsh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice!

@alexjoel42
Copy link
Copy Markdown
Contributor

alexjoel42 commented Apr 7, 2026

Amazing!
@rclarke0 , for visibility RQA-5312 will break this test because it mixes on a Thermocyler

@skowalski08 skowalski08 merged commit 343a29a into chore_release-pd-8.10.0 Apr 7, 2026
13 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants