Removing branch name from sonar publishers (#142)

* Removing branch name from sonar publishers Co-authored-by: Alec Sherwood <[email protected]>
constantcontact · Jun 11, 2020 · af67d3f · af67d3f
1 parent dc7162c
commit af67d3f
Show file tree

Hide file tree

Showing 3 changed files with 1 addition and 12 deletions.
diff --git a/lib/jenkins_pipeline_builder/extensions/publishers.rb b/lib/jenkins_pipeline_builder/extensions/publishers.rb
@@ -408,7 +408,6 @@
   xml do |params|
     send('hudson.plugins.sonar.SonarPublisher') do
       jdk params[:jdk] || '(Inherit From Job)'
-      branch params[:branch] || ''
       language
       mavenOpts
       jobAdditionalProperties params[:additional_properties] || ''

diff --git a/lib/jenkins_pipeline_builder/version.rb b/lib/jenkins_pipeline_builder/version.rb
@@ -21,5 +21,5 @@
 #
 
 module JenkinsPipelineBuilder
-  VERSION = '1.9.1'.freeze
+  VERSION = '1.10.0'.freeze
 end
diff --git a/spec/lib/jenkins_pipeline_builder/extensions/publishers_spec.rb b/spec/lib/jenkins_pipeline_builder/extensions/publishers_spec.rb
@@ -48,16 +48,6 @@
       expect(additional_properties_value.first.content).to match ''
     end
 
-    it 'populates branch' do
-      params = { publishers: { sonar_result: { branch: 'test' } } }
-
-      JenkinsPipelineBuilder.registry.traverse_registry_path('job', params, @n_xml)
-
-      sonar_nodes = @n_xml.root.children.first.children
-      branch = sonar_nodes.select { |node| node.name == 'branch' }
-      expect(branch.first.content).to match 'test'
-    end
-
     it 'populates maven installation name' do
       params = { publishers: { sonar_result: { maven_installation_name: 'test' } } }