Skip to content

Geo Sites - Jest spec cleanup

Zack Cuddy requested to merge 329569_geo_sites_test_cleanup into master

What does this MR do and why?

Closes #329569 (closed)

This change is a technical debt cleanup of the Jest specs for the Geo Sites UI. This MR does the following things:

  1. Removes all wrapper.destroy calls => &8915
  2. Replaces all extendedWrapper(shallowMount) with shallowMountExtended
  3. Replaces non-explicit selectors (ie. wrapper.find('span')) with wrapper.findByTestId()
  4. Removes a couple redundant tests

Screenshots or screen recordings

N/A

How to set up and validate locally

  1. Run yarn jest ee/spec/frontend/geo_nodes and ensure all tests pass.

MR acceptance checklist

This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.

Merge request reports