Protected packages: Fix rubocop issues for CreateMavenPackageService
requested to merge gitlab-community/gitlab:323969-gerardo-navarro-protected-packages-maven-create-package-service-return-service-response-follow-up-rubocop-fix into master
What does this MR do and why?
Protected packages: Return ServiceResponse for CreateMavenPackageService
- Returning a successful ServiceResponse as suggested by the best-practice documentation, see https://docs.gitlab.com/ee/development/reusing_abstractions.html#serviceresponse
MR acceptance checklist
Please evaluate this MR against the MR acceptance checklist. It helps you analyze changes to reduce risks in quality, performance, reliability, security, and maintainability.
MR Checklist (@gerardo-navarro)
-
Changelog entry added, if necessary -
Documentation created/updated via this MR -
Documentation reviewed by technical writer or follow-up review issue created -
Tests added for this feature/bug -
Tested in all supported browsers -
Conforms to the code review guidelines -
Conforms to the merge request performance guidelines -
Conforms to the style guides -
Conforms to the javascript style guides -
Conforms to the database guides
Screenshots or screen recordings
Only backend changes.
How to set up and validate locally
- Execute the test
bundle exec spec/services/packages/maven/create_package_service_spec.rb spec/services/packages/maven/find_or_create_package_service_spec.rbRelated to #323969
Edited by Gerardo Navarro