Simplification of graphql mutations
What does this MR do?
This MR simplifies few graphql mutations in 2 ways:
- It uses a method
errors
ofServiceResponse
which was introduced in !33501 (merged) - For
JiraImport::Start
it simplifies fetching project and its validation. We don't need to checkproject_path
and thenproject
presence as it is handled inauthorized_find!
(we test this in the related specs, no changes needed there, still passing)
Does this MR meet the acceptance criteria?
Conformity
-
Changelog entry -
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.