Geo: Verification-related refactors
What does this MR do?
Extracts a bunch of refactor commits from !47260 (merged) to improve reviewability. No behavior change.
Most of these commits are only intended to improve readability and understandability.
This MR also removes some verification tests from being run on data types where we are not yet implementing verification. Since those tests will break with the implementation of new verification behavior in !47260 (merged). There is a follow up issue linked in the code to make sure we add back the tests to all the things https://gitlab.com/gitlab-org/gitlab/-/issues/280768
Part of #13839 (closed) and #13981 (closed)
Does this MR meet the acceptance criteria?
Conformity
- [-] Changelog entry => No behavior change
- [-] Documentation (if required)
-
Code review guidelines -
Merge request performance guidelines -
Style guides -
Database guides -
Separation of EE specific content
Availability and Testing
-
Review and add/update tests for this feature/bug. Consider all test levels. See the Test Planning Process.
Edited by Michael Kozono