Skip to content

Reduce enzyme usage in unit test down to 50% - [2 of 3] #8806

@smilingkylan

Description

@smilingkylan

What is this about?

Convert the list of unit tests to use react-testing-library instead of enzyme. In total there are 53 files to convert. To make PRs easier to review and more manageable, this covers 17 test files.

Scenario

No response

Design

No response

Technical Details

Convert these files:

 'app/components/UI/OnboardingWizard/Step5/index.test.tsx',
  'app/components/UI/OnboardingWizard/Step6/index.test.tsx',
  'app/components/UI/OnboardingWizard/Step4/index.test.tsx',
  'app/components/UI/OnboardingWizard/Step2/index.test.tsx',
  'app/components/UI/OnboardingWizard/Step1/index.test.tsx',
  'app/component-library/components/Tags/TagUrl/TagUrl.test.tsx',
  'app/components/Views/confirmations/components/SignatureRequest/ExpandedMessage/index.test.tsx',
  'app/component-library/components/Avatars/Avatar/foundation/AvatarBase/AvatarBase.test.tsx',
  'app/components/Views/SimpleWebview/index.test.tsx',
  'app/components/Views/OnboardingCarousel/index.test.tsx',
  'app/components/Views/ManualBackupStep3/index.test.tsx',
  'app/components/UI/NavbarTitle/index.test.js',
  'app/components/UI/SkipAccountSecurityModal/index.test.tsx',
  'app/components/UI/EditGasFeeLegacy/index.test.tsx',
  'app/components/UI/EditGasFee1559/index.test.tsx',
  'app/component-library/components/Badges/BadgeWrapper/BadgeWrapper.test.tsx',
  'app/components/UI/HintModal/index.test.tsx',

Threat Modeling Framework

No response

Acceptance Criteria

Unit tests for the updated files should all pass. Snapshots may be updated

Stakeholder review needed before the work gets merged

  • Engineering (needed in most cases)
  • Design
  • Product
  • QA (automation tests are required to pass before merging PRs but not all changes are covered by automation tests - please review if QA is needed beyond automation tests)
  • Security
  • Legal
  • Marketing
  • Management (please specify)
  • Other (please specify)

References

No response

Metadata

Metadata

Assignees

Labels

No QA NeededApply this label when your PR does not need any QA effort.release-7.19.0Issue or pull request that will be included in release 7.19.0team-mobile-platformMobile Platform team

Type

No type
No fields configured for issues without a type.

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions