Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat(protocol-designer): remove deck config ff and add deck config cypress test #14096

Merged
merged 7 commits into from
Dec 7, 2023

Conversation

jerader
Copy link
Collaborator

@jerader jerader commented Dec 5, 2023

closes RAUT-876 RAUT-842 RAUT-841

Overview

To prepare for the 8.0 PD release, this PR removes the deck configuration ff and adds a cypress migration test with staging area slots and waste chute

Test Plan

Do a quick smoke test, go through creating a protocol with staging area slots and the waste chute. Try editing them in the Additional Items section. See that it all works as expected.

Changelog

  • clean up unused consts
  • remove ff and every instance of it
  • fix tests

Review requests

see test plan

Risk assessment

low

Copy link

codecov bot commented Dec 5, 2023

Codecov Report

Merging #14096 (f5d8d73) into edge (9b22225) will increase coverage by 0.00%.
Report is 1 commits behind head on edge.
The diff coverage is 53.84%.

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff           @@
##             edge   #14096   +/-   ##
=======================================
  Coverage   70.36%   70.36%           
=======================================
  Files        2481     2481           
  Lines       70351    70325   -26     
  Branches     8790     8780   -10     
=======================================
- Hits        49504    49486   -18     
+ Misses      18690    18687    -3     
+ Partials     2157     2152    -5     
Flag Coverage Δ
protocol-designer 44.51% <53.84%> (-0.08%) ⬇️
step-generation 86.90% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Files Coverage Δ
...onents/modals/CreateFileWizard/PipetteTypeTile.tsx 83.87% <100.00%> (-0.98%) ⬇️
...onents/modals/CreateFileWizard/StagingAreaTile.tsx 46.15% <100.00%> (-1.35%) ⬇️
...ponents/modals/FilePipettesModal/PipetteFields.tsx 80.64% <100.00%> (+2.52%) ⬆️
...esigner/src/components/modules/EditModulesCard.tsx 82.60% <100.00%> (-0.38%) ⬇️
protocol-designer/src/constants.ts 97.95% <ø> (-0.09%) ⬇️
protocol-designer/src/feature-flags/reducers.ts 57.14% <ø> (-5.36%) ⬇️
protocol-designer/src/feature-flags/selectors.ts 62.50% <ø> (+4.16%) ⬆️
protocol-designer/src/feature-flags/types.ts 100.00% <ø> (ø)
...ol-designer/src/file-data/selectors/fileCreator.ts 68.29% <ø> (ø)
step-generation/src/constants.ts 100.00% <ø> (ø)
... and 2 more

@jerader jerader force-pushed the pd_remove-deck-config-ff branch from d5f4c7a to 1064944 Compare December 6, 2023 18:23
@jerader jerader marked this pull request as ready for review December 6, 2023 19:30
@jerader jerader requested a review from a team December 6, 2023 19:30
@jerader jerader requested a review from a team as a code owner December 6, 2023 19:30
@jerader jerader requested review from koji and mjhuff and removed request for a team December 6, 2023 19:30
@jerader jerader force-pushed the pd_remove-deck-config-ff branch from 272e280 to 1df5f32 Compare December 7, 2023 15:17
@jerader jerader force-pushed the pd_remove-deck-config-ff branch from 1df5f32 to 83a4e72 Compare December 7, 2023 16:00
@@ -1,6 +1,6 @@
import { OutputSelector, Selector } from 'reselect'
declare module 'reselect' {
// declaring type for createSelector with 15 selectors because the reselect types only support up to 12 selectors
// declaring type for createSelector with 14 selectors because the reselect types only support up to 12 selectors
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

this change is needed since i removed the getAdditionalEquipmentEntities selector from fileCreator

Copy link
Contributor

@koji koji left a comment

Choose a reason for hiding this comment

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

left one comment but otherwise lgtm

@jerader jerader requested a review from koji December 7, 2023 18:13
Copy link
Contributor

@koji koji left a comment

Choose a reason for hiding this comment

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

lgtm

@jerader jerader merged commit 1ce77e5 into edge Dec 7, 2023
16 of 17 checks passed
@jerader jerader deleted the pd_remove-deck-config-ff branch December 7, 2023 18:28
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.

2 participants