Record RuboCop issues for Katello

This changes the invocation to directly call RuboCop instead of relying on a rake task. This is copied from smart-proxy.groovy. Interstingly enough, it doesn't use the katello:rubocop:jenkins task that also exists.
theforeman · Jan 5, 2024 · c84e5eb · c84e5eb
1 parent a68120f
commit c84e5eb
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 2 deletions.
diff --git a/theforeman.org/pipelines/test/foreman/katello.groovy b/theforeman.org/pipelines/test/foreman/katello.groovy
@@ -58,7 +58,12 @@ pipeline {
                     stage('rubocop') {
                         steps {
                             dir('foreman') {
-                                withRVM(['bundle exec rake katello:rubocop TESTOPTS="-v" --trace'], ruby)
+                                withRVM(["bundle exec rubocop --format progress --out ${env.WORKSPACE}/rubocop.log --format progress ${env.WORKSPACE}"], ruby)
+                            }
+                        }
+                        post {
+                            always {
+                                recordIssues tool: ruboCop(pattern: "${env.WORKSPACE}/rubocop.log"), enabledForFailure: true
                             }
                         }
                     }

diff --git a/theforeman.org/pipelines/test/testKatello.groovy b/theforeman.org/pipelines/test/testKatello.groovy
@@ -116,7 +116,12 @@ pipeline {
                     stage('rubocop') {
                         steps {
                             dir('foreman') {
-                                withRVM(['bundle exec rake katello:rubocop TESTOPTS="-v" --trace'], ruby)
+                                withRVM(["bundle exec rubocop --format progress --out ${env.WORKSPACE}/rubocop.log --format progress ${env.WORKSPACE}"], ruby)
+                            }
+                        }
+                        post {
+                            always {
+                                recordIssues tool: ruboCop(pattern: "${env.WORKSPACE}/rubocop.log"), enabledForFailure: true
                             }
                         }
                     }