address pr comment

Opentrons · Jan 10, 2022 · e44e344 · e44e344
1 parent ad54f64
commit e44e344
Show file tree

Hide file tree

Showing 3 changed files with 7 additions and 11 deletions.
diff --git a/app/src/assets/localization/en/labware_position_check.json b/app/src/assets/localization/en/labware_position_check.json
@@ -2,9 +2,8 @@
  "labware_position_check_title": "Labware Position Check",
  "labware_position_check_overview": "Labware Position Check Overview",
  "position_check_description": "<block>Labware Position Check is a guided workflow that checks every labware on the deck for an added degree of precision in your protocol.</block><block>When you check a labware, the OT-2’s pipette nozzle or attached tip will stop at the center of the A1 well. If the pipette nozzle or tip is not centered, you can reveal the OT-2’s jog controls to make an adjustment. This Labware Offset will be applied to the entire labware. Offset data is measured to the nearest 1/10th mm and can be made in the X, Y and/or Z directions.</block>",
- "secondary_pipette_tipracks_section": "Check tip racks with {{secondary_mount}} Pipette and pick up a tip",
+ "secondary_pipette_tipracks_section": "Check tip racks with {{secondary_mount}} Pipette",
  "primary_pipette_tipracks_section": "Check tip racks with {{primary_mount}} Pipette and pick up a tip",
- "primary_pipette_tipracks_section_plural": "Check tip racks with {{primary_mount}} Pipette",
  "check_remaining_labware_with_primary_pipette_section": "Check remaining labware with {{primary_mount}} Pipette and tip",
  "return_tip_section": "Return tip",
  "labware_step_detail_labware": "The tip should be centered above A1 in <italic>{{labware_name}}</italic> and <bold>level</bold> with the top of the labware.",

diff --git a/app/src/organisms/ProtocolSetup/LabwarePositionCheck/SectionList.tsx b/app/src/organisms/ProtocolSetup/LabwarePositionCheck/SectionList.tsx
@@ -99,9 +99,6 @@ export function SectionList(props: Props): JSX.Element {
  {t(`${section.toLowerCase()}_section`, {
  primary_mount: capitalize(primaryPipetteMount),
  secondary_mount: capitalize(secondaryPipetteMount),
- count: sections.includes('SECONDARY_PIPETTE_TIPRACKS')
- ? 2
- : 1,
  })}
  </Text>
  </Box>

diff --git a/app/src/organisms/ProtocolSetup/LabwarePositionCheck/__tests__/SectionList.test.tsx b/app/src/organisms/ProtocolSetup/LabwarePositionCheck/__tests__/SectionList.test.tsx
@@ -53,9 +53,9 @@ describe('SectionList', () => {
  }
  const { getByText } = render(props)
  expect(getByText('1')).toHaveStyle('backgroundColor: C_SELECTED_DARK')
- getByText('Check tip racks with Left Pipette')
+ getByText('Check tip racks with Right Pipette')
  expect(getByText('2')).toHaveStyle('backgroundColor: C_DISABLED')
- getByText('Check tip racks with Right Pipette and pick up a tip')
+ getByText('Check tip racks with Left Pipette and pick up a tip')
  expect(getByText('3')).toHaveStyle('backgroundColor: C_DISABLED')
  getByText('Return tip')
  })
@@ -68,9 +68,9 @@ describe('SectionList', () => {
  }
  const { getByText } = render(props)
  expect(getByText('1')).toHaveStyle('backgroundColor: C_SELECTED_DARK')
- getByText('Check tip racks with Left Pipette')
+ getByText('Check tip racks with Right Pipette')
  expect(getByText('2')).toHaveStyle('backgroundColor: C_DISABLED')
- getByText('Check tip racks with Right Pipette and pick up a tip')
+ getByText('Check tip racks with Left Pipette and pick up a tip')
  expect(getByText('3')).toHaveStyle('backgroundColor: C_DISABLED')
  getByText('Check remaining labware with Left Pipette and tip')
  expect(getByText('4')).toHaveStyle('backgroundColor: C_DISABLED')
@@ -85,9 +85,9 @@ describe('SectionList', () => {
  completedSections: ['PRIMARY_PIPETTE_TIPRACKS'],
  }
  const { getByText } = render(props)
- getByText('Check tip racks with Left Pipette')
+ getByText('Check tip racks with Right Pipette')
  expect(getByText('2')).toHaveStyle('backgroundColor: C_SELECTED_DARK')
- getByText('Check tip racks with Right Pipette and pick up a tip')
+ getByText('Check tip racks with Left Pipette and pick up a tip')
  expect(getByText('3')).toHaveStyle('backgroundColor: C_DISABLED')
  getByText('Return tip')
  })