jenkins_pipeline_builder 0.10.12 → 0.10.13

data/spec/lib/jenkins_pipeline_builder/extensions/builders_spec.rb

@@ -22,6 +22,8 @@ describe 'builders' do
 
   after :each do |example|
     name = example.description.gsub ' ', '_'
+    require 'fileutils'
+    FileUtils.mkdir_p 'out/xml'
     File.open("./out/xml/builder_#{name}.xml", 'w') { |f| @n_xml.write_xml_to f }
   end
 
@@ -63,13 +65,20 @@ describe 'builders' do
 
     it 'provides job specific config' do
       params = { builders: { multi_job: { phases: { foo: { jobs: [{ name: 'foo', config: {
-        predefined_build_parameters: 'bar'
+        predefined_build_parameters: 'bar',
+        properties_file: { file: 'props', skip_if_missing: true }
       } }] } } } } }
 
       JenkinsPipelineBuilder.registry.traverse_registry_path('job', params, @n_xml)
 
       node = @n_xml.xpath '//hudson.plugins.parameterizedtrigger.PredefinedBuildParameters'
       expect(node.children.first.content).to eq 'bar'
+
+      node = @n_xml.xpath '//hudson.plugins.parameterizedtrigger.FileBuildParameters/propertiesFile'
+      expect(node.children.first.content).to eq 'props'
+
+      node = @n_xml.xpath '//hudson.plugins.parameterizedtrigger.FileBuildParameters/failTriggerOnMissing'
+      expect(node.children.first.content).to eq 'true'
     end
   end
 end

data/spec/lib/jenkins_pipeline_builder/generator_spec.rb

@@ -1,11 +1,7 @@
 require File.expand_path('../spec_helper', __FILE__)
 
 describe JenkinsPipelineBuilder::Generator do
-  after :each do
-    JenkinsPipelineBuilder.registry.clear_versions
-  end
-
-  before(:all) do
+  before :all do
     JenkinsPipelineBuilder.credentials = {
       server_ip: '127.0.0.1',
       server_port: 8080,
@@ -13,11 +9,17 @@ describe JenkinsPipelineBuilder::Generator do
       password: 'password',
       log_location: '/dev/null'
     }
+  end
+
+  after :each do
+    JenkinsPipelineBuilder.registry.clear_versions
+  end
+
+  before :each do
     @generator = JenkinsPipelineBuilder.generator
   end
 
   after(:each) do
-    JenkinsPipelineBuilder.no_debug!
     @generator.job_collection = JenkinsPipelineBuilder::JobCollection.new
   end
 
@@ -109,80 +111,37 @@ describe JenkinsPipelineBuilder::Generator do
 
   describe '#pull_request' do
     before :each do
-      allow(JenkinsPipelineBuilder.client).to receive(:plugin).and_return double(
-        list_installed: { 'description' => '20.0', 'git' => '20.0' })
-      JenkinsPipelineBuilder.debug!
+      allow(JenkinsPipelineBuilder).to receive(:debug).and_return true
+      JenkinsPipelineBuilder.registry.registry[:job][:scm_params].installed_version = '1000.0'
     end
-    let(:jobs) do
-      {
-        '{{name}}-10-SampleJob' => {
-          name: '{{name}}-10-SampleJob',
-          type: :job,
-          value: {
-            name: '{{name}}-10-SampleJob',
-            scm_branch: 'origin/pr/{{pull_request_number}}/head',
-            scm_params: {
-              refspec: 'refs/pull/*:refs/remotes/origin/pr/*'
-            }
-          }
-        }
-      }
+    after :each do
+      JenkinsPipelineBuilder.registry.registry[:job][:scm_params].clear_installed_version
     end
+
     let(:path) { File.expand_path('../fixtures/generator_tests/pullrequest_pipeline', __FILE__) }
     it 'produces no errors while creating pipeline PullRequest' do
-      # Dummy data
-      purge = []
-      create = [
-        {
-          name: 'PullRequest-PR1',
-          type: :project,
-          value: {
-            name: 'PullRequest-PR1',
-            pull_request_number: '1',
-            jobs: [
-              '{{name}}-10-SampleJob'
-            ]
-          }
-        }
-      ]
-      # Run the test
       job_name = 'PullRequest'
-      expect(JenkinsPipelineBuilder::PullRequestGenerator).to receive(:new).once.and_return(
-        double(purge: purge, create: create, jobs: jobs)
-      )
+      allow_any_instance_of(JenkinsPipelineBuilder::PullRequestGenerator).to receive(:check_for_pull).and_return([1])
+      allow_any_instance_of(JenkinsPipelineBuilder::PullRequestGenerator).to receive(:purge_jobs).and_return(true)
       success = @generator.pull_request(path, job_name)
       expect(success).to be_truthy
     end
 
-    it 'correclty creates jobs when there are multiple pulls open' do
-      purge = []
-      create = %w(1 2).map do |n|
-        {
-          name: "PullRequest-PR#{n}",
-          type: :project,
-          value: {
-            pull_request_number: n,
-            name: "PullRequest-PR#{n}",
-            jobs: [
-              '{{name}}-10-SampleJob'
-            ]
-          }
-        }
-      end
+    it 'correctly creates jobs when there are multiple pulls open' do
       job_name = 'PullRequest'
-      expect(JenkinsPipelineBuilder::PullRequestGenerator).to receive(:new).once.and_return(
-        double(purge: purge, create: create, jobs: jobs)
-      )
+      allow_any_instance_of(JenkinsPipelineBuilder::PullRequestGenerator).to receive(:check_for_pull).and_return([1, 2])
       job1 = double name: 'job name'
       job2 = double name: 'job name'
       expect(JenkinsPipelineBuilder::Job).to receive(:new).once.with(
         name: 'PullRequest-PR1-10-SampleJob', scm_branch: 'origin/pr/1/head', scm_params: {
-          refspec: 'refs/pull/*:refs/remotes/origin/pr/*'
+          refspec: 'refs/pull/1/head:refs/remotes/origin/pr/1/head',
+          changelog_to_branch: { remote: 'origin', branch: 'pr/1/head' }
         }
       ).and_return job1
       expect(JenkinsPipelineBuilder::Job).to receive(:new).once.with(
         name: 'PullRequest-PR2-10-SampleJob', scm_branch: 'origin/pr/2/head', scm_params: {
-          refspec: 'refs/pull/*:refs/remotes/origin/pr/*'
+          refspec: 'refs/pull/2/head:refs/remotes/origin/pr/2/head',
+          changelog_to_branch: { remote: 'origin', branch: 'pr/2/head' }
         }
       ).and_return job2
       expect(job1).to receive(:create_or_update).and_return true
@@ -195,7 +154,7 @@ describe JenkinsPipelineBuilder::Generator do
     # Fails to purge old PR jobs from Jenkins
   end
 
-  describe '#load_collection_from_path' do
+  describe '#load_from_path' do
     let(:project_hash) do
       [{ 'defaults' => { 'name' => 'global', 'description' => 'Tests, all the tests' } },
        { 'project' => { 'name' => 'TestProject', 'jobs' => ['{{name}}-part1'] } }]
@@ -206,28 +165,23 @@ describe JenkinsPipelineBuilder::Generator do
       }]
     end
 
-    before :each do
-      expect(@generator.job_collection).to receive(:load_file).once.with(view_hash, false).and_return(true)
-      expect(@generator.job_collection).to receive(:load_file).once.with(project_hash, false).and_return(true)
-    end
-
     it 'loads a yaml collection from a path' do
       path = File.expand_path('../fixtures/generator_tests/test_yaml_files', __FILE__)
-      @generator.job_collection.send(:load_from_path, path)
+      @generator.job_collection.load_from_path path
     end
     it 'loads a json collection from a path' do
       path = File.expand_path('../fixtures/generator_tests/test_json_files', __FILE__)
-      @generator.job_collection.send(:load_from_path, path)
+      @generator.job_collection.load_from_path path
     end
     it 'loads both yaml and json files from a path' do
       path = File.expand_path('../fixtures/generator_tests/test_combo_files', __FILE__)
-      @generator.job_collection.send(:load_from_path, path)
+      @generator.job_collection.load_from_path path
     end
   end
 
   describe '#dump' do
     it "writes a job's config XML to a file" do
-      JenkinsPipelineBuilder.debug!
+      allow(JenkinsPipelineBuilder).to receive(:debug).and_return true
       job_name = 'test_job'
       body = ''
       test_path = File.expand_path('../fixtures/generator_tests', __FILE__)

data/spec/lib/jenkins_pipeline_builder/job_spec.rb

@@ -0,0 +1,142 @@
+require File.expand_path('../spec_helper', __FILE__)
+
+describe JenkinsPipelineBuilder::Job do
+  before :all do
+    JenkinsPipelineBuilder.credentials = {
+      server_ip: '127.0.0.1',
+      server_port: 8080,
+      username: 'username',
+      password: 'password',
+      log_location: '/dev/null'
+    }
+  end
+
+  before :each do
+    allow(JenkinsPipelineBuilder).to receive(:logger).and_return double(
+      debug: true,
+      info: true,
+      warn: true,
+      error: true,
+      fatal: true
+    )
+  end
+
+  context '#create_or_update' do
+    before :each do
+      @job_double = double
+      @client = double job: @job_double
+      allow(JenkinsPipelineBuilder).to receive(:client).and_return @client
+    end
+
+    let(:job) { described_class.new name: 'asdf' }
+
+    it 'fails if to_xml fails' do
+      expect(job).to receive(:to_xml).ordered.and_return [false, 'oops']
+      expect(job.create_or_update).to eq [false, 'oops']
+    end
+
+    it 'does not call the client in debug mode' do
+      expect(job).to receive(:to_xml).and_return [true, '']
+      expect(JenkinsPipelineBuilder).to receive(:debug).and_return true
+      expect(JenkinsPipelineBuilder).to_not receive(:file_mode)
+      expect(JenkinsPipelineBuilder.client.job).to_not receive(:exists?)
+      job.create_or_update
+    end
+
+    it 'does not call the client in file mode' do
+      expect(job).to receive(:to_xml).and_return [true, '']
+      expect(JenkinsPipelineBuilder).to receive(:debug).and_return false
+      expect(JenkinsPipelineBuilder).to receive(:file_mode).and_return true
+      expect(JenkinsPipelineBuilder.client.job).to_not receive(:exists?)
+      job.create_or_update
+    end
+
+    it 'creates if the job does not exist' do
+      expect(JenkinsPipelineBuilder).to receive(:debug).and_return false
+      expect(JenkinsPipelineBuilder).to receive(:file_mode).and_return false
+      expect(job).to receive(:to_xml).and_return [true, '']
+      expect(@job_double).to receive(:exists?).with('asdf').and_return false
+      expect(@job_double).to receive(:create).and_return true
+      job.create_or_update
+    end
+
+    it 'updates if the job exists' do
+      expect(JenkinsPipelineBuilder).to receive(:debug).and_return false
+      expect(JenkinsPipelineBuilder).to receive(:file_mode).and_return false
+      expect(job).to receive(:to_xml).and_return [true, '']
+      expect(@job_double).to receive(:exists?).with('asdf').and_return true
+      expect(@job_double).to receive(:update).and_return true
+      job.create_or_update
+    end
+  end
+
+  context '#to_xml' do
+    it 'fails if the job has no name' do
+      job = described_class.new foo: :bar
+      expect { job.to_xml }.to raise_error 'Job name is not specified'
+    end
+
+    it 'parses inline job dsl' do
+      job = described_class.new job_type: 'job_dsl', name: 'asdf', job_dsl: 'this is my job dsl'
+
+      success, xml = job.to_xml
+      expect(success).to be true
+      expect(xml).to include 'javaposse.jobdsl.plugin.ExecuteDslScripts'
+      expect(xml).to include 'usingScriptText>true'
+      expect(xml).to include 'this is my job dsl'
+    end
+
+    it 'parses job dsl from a file' do
+      job = described_class.new job_type: 'job_dsl', name: 'asdf', job_dsl_targets: 'targets'
+
+      success, xml = job.to_xml
+      expect(success).to be true
+      expect(xml).to include 'javaposse.jobdsl.plugin.ExecuteDslScripts'
+      expect(xml).to include 'targets>targets'
+      expect(xml).to include 'usingScriptText>false'
+    end
+
+    it 'parses multi_job' do
+      job = described_class.new job_type: 'multi_project', name: 'asdf'
+
+      success, xml = job.to_xml
+      expect(success).to be true
+      expect(xml).to include 'com.tikal.jenkins.plugins.multijob.MultiJobProject'
+    end
+
+    it 'parses build flow' do
+      job = described_class.new job_type: 'build_flow', name: 'asdf'
+
+      success, xml = job.to_xml
+      expect(success).to be true
+      expect(xml).to include 'com.cloudbees.plugins.flow.BuildFlow'
+    end
+
+    it 'parses freestyle' do
+      job = described_class.new job_type: 'free_style', name: 'asdf'
+
+      success, xml = job.to_xml
+      expect(success).to be true
+      expect(xml).to_not include 'com.cloudbees.plugins.flow.BuildFlow'
+      expect(xml).to_not include 'javaposse.jobdsl.plugin.ExecuteDslScripts'
+      expect(xml).to_not include 'com.tikal.jenkins.plugins.multijob.MultiJobProject'
+    end
+
+    it 'parses pull request generator' do
+      job = described_class.new job_type: 'pull_request_generator', name: 'asdf'
+
+      success, xml = job.to_xml
+      expect(success).to be true
+      expect(xml).to_not include 'com.cloudbees.plugins.flow.BuildFlow'
+      expect(xml).to_not include 'javaposse.jobdsl.plugin.ExecuteDslScripts'
+      expect(xml).to_not include 'com.tikal.jenkins.plugins.multijob.MultiJobProject'
+    end
+
+    it 'fails on an unknown type' do
+      job = described_class.new job_type: 'unknonw', name: 'asdf'
+
+      success, _ = job.to_xml
+      expect(success).to be false
+    end
+  end
+end

data/spec/lib/jenkins_pipeline_builder/module_registry_spec.rb

@@ -139,6 +139,8 @@ describe JenkinsPipelineBuilder::ModuleRegistry do
     end
 
     it 'pulls the version from the registry if it is not memoized' do
+      subject.clear_versions
+      expect(JenkinsPipelineBuilder).to receive(:debug).and_return false
       expect(JenkinsPipelineBuilder.client).to receive(:plugin).and_return double(list_installed: { 'one' => '0.1' })
       subject.versions
     end

data/spec/lib/jenkins_pipeline_builder/pull_request_generator_spec.rb

@@ -3,10 +3,10 @@ require 'webmock/rspec'
 
 describe JenkinsPipelineBuilder::PullRequestGenerator do
   let(:pull_request_generator) { JenkinsPipelineBuilder::PullRequestGenerator }
-  let(:project) { { name: 'pull_req_test', type: :project, value: { name: 'pull_req_test', jobs: ['{{name}}-00', '{{name}}-10', '{{name}}-11'] } } }
+  let(:project) { { name: 'pull_req_test', type: :project, value: { name: 'pull_req_test', jobs: [{ name: '{{name}}-00', type: :job, name: '{{name}}-00', job_type: 'pull_request_generator', git_url: 'https://www.github.com/', git_repo: 'jenkins_pipeline_builder', git_org: 'constantcontact', jobs: ['{{name}}-10', '{{name}}-11'], builders: [{ shell_command: 'generate -v || gem install jenkins_pipeline_builder\ngenerate pipeline -c config/{{login_config}} pull_request pipeline/ {{name}}\n' }] }, '{{name}}-10', '{{name}}-11'] } } }
   let(:jobs) { { '{{name}}-10' => { name: '{{name}}-10', type: :'job-template', value: { name: '{{name}}-10', description: '{{description}}', publishers: [{ downstream: { project: '{{job@{{name}}-11}}' } }] }  }, '{{name}}-11' => { name: '{{name}}-11', type: :'job-template', value: { name: '{{name}}-11', description: '{{description}}' } } } }
   let(:create_jobs) { [{ name: 'pull_req_test-PR5', type: :project, value: { name: 'pull_req_test-PR5', jobs: ['{{name}}-10', '{{name}}-11'], pull_request_number: '5' } }, { name: 'pull_req_test-PR6', type: :project, value: { name: 'pull_req_test-PR6', jobs: ['{{name}}-10', '{{name}}-11'], pull_request_number: '6' } }] }
-  let(:pull_request) { { name: '{{name}}-00', type: :job, name: '{{name}}-00', job_type: 'pull_request_generator', git_url: 'https://www.github.com/', git_repo: 'jenkins_pipeline_builder', git_org: 'constantcontact', jobs: ['{{name}}-10', '{{name}}-11'], builders: [{ shell_command: 'generate -v || gem install jenkins_pipeline_builder\ngenerate pipeline -c config/{{login_config}} pull_request pipeline/ {{name}}\n' }] } }
+  let(:generator) { JenkinsPipelineBuilder::Generator.new }
   before do
     # Request to get current pull requests from github
     stub_request(:any, 'https://www.github.com/api/v3/repos/constantcontact/jenkins_pipeline_builder/pulls').to_return(body: '[{"number": 5,"state": "open","title": "Update README again" },{"number": 6,"state": "open",  "title": "Update README again2"}]')
@@ -16,13 +16,17 @@ describe JenkinsPipelineBuilder::PullRequestGenerator do
     after(:all) do
       FileUtils.rm_r 'pull_requests.csv'
     end
+
     it 'can work without a csv' do
-      pull = pull_request_generator.new(project, jobs, pull_request)
+      pending 'rework this'
+      pull = pull_request_generator.new(project, generator)
+      generator.job_collection.collection = jobs
       expect(pull.purge.count).to eq(0)
       expect(pull.create).to eq(create_jobs)
     end
     it 'can work with a csv' do
-      pull = pull_request_generator.new(project, jobs, pull_request)
+      pending 'rework this'
+      pull = pull_request_generator.new(project, generator)
       expect(pull.purge.count).to eq(0)
       expect(pull.create).to eq(create_jobs)
     end

data/spec/lib/jenkins_pipeline_builder/pull_request_spec.rb

@@ -2,14 +2,20 @@ require File.expand_path('../spec_helper', __FILE__)
 require 'webmock/rspec'
 
 describe JenkinsPipelineBuilder::PullRequest do
+  before :each do
+    JenkinsPipelineBuilder.registry.registry[:job][:scm_params].installed_version = '1000.0'
+  end
+  after :each do
+    JenkinsPipelineBuilder.registry.registry[:job][:scm_params].clear_installed_version
+  end
   let(:pull_request_class) { JenkinsPipelineBuilder::PullRequest }
   let(:project) { { name: 'pull_req_test', type: :project, value: { name: 'pull_req_test', jobs: ['{{name}}-00', '{{name}}-10', '{{name}}-11'] } } }
-  let(:pull_request) { { name: '{{name}}-00', type: :job, name: '{{name}}-00', job_type: 'pull_request_generator', git_url: 'https://www.github.com/', git_repo: 'jenkins_pipeline_builder', git_org: 'constantcontact', jobs: ['{{name}}-10', '{{name}}-11'], builders: [{ shell_command: 'generate -v || gem install jenkins_pipeline_builder\ngenerate pipeline -c config/{{login_config}} pull_request pipeline/ {{name}}\n' }] } }
+  let(:pull_request) { { name: '{{name}}-00', type: :job, name: '{{name}}-00', job_type: 'pull_request_generator', git_url: 'https://www.github.com/', git_repo: 'jenkins_pipeline_builder', git_org: 'constantcontact', value: { jobs: ['{{name}}-10', '{{name}}-11'] }, builders: [{ shell_command: 'generate -v || gem install jenkins_pipeline_builder\ngenerate pipeline -c config/{{login_config}} pull_request pipeline/ {{name}}\n' }] } }
   let(:jobs) { { '{{name}}-10' => { name: '{{name}}-10', type: :'job-template', value: { name: '{{name}}-10', description: '{{description}}', publishers: [{ downstream: { project: '{{job@{{name}}-11}}' } }] }  }, '{{name}}-11' => { name: '{{name}}-11', type: :'job-template', value: { name: '{{name}}-11', description: '{{description}}' } } } }
   describe '#initialize' do
     it 'process pull_request' do
       pull = pull_request_class.new(project, 2, jobs, pull_request)
-      post_jobs = { '{{name}}-10' => { name: '{{name}}-10', type: :'job-template', value: { name: '{{name}}-10', description: '{{description}}', publishers: [{ downstream: { project: '{{job@{{name}}-11}}' } }], scm_branch: 'origin/pr/{{pull_request_number}}/head', scm_params: { refspec: 'refs/pull/*:refs/remotes/origin/pr/*' } } }, '{{name}}-11' => { name: '{{name}}-11', type: :'job-template', value: { name: '{{name}}-11', description: '{{description}}', scm_branch: 'origin/pr/{{pull_request_number}}/head', scm_params: { refspec: 'refs/pull/*:refs/remotes/origin/pr/*' } } } }
+      post_jobs = { '{{name}}-10' => { name: '{{name}}-10', type: :'job-template', value: { name: '{{name}}-10', description: '{{description}}', publishers: [{ downstream: { project: '{{job@{{name}}-11}}' } }], scm_branch: 'origin/pr/{{pull_request_number}}/head', scm_params: { changelog_to_branch: { remote: 'origin', branch: 'pr/{{pull_request_number}}/head' }, refspec: 'refs/pull/{{pull_request_number}}/head:refs/remotes/origin/pr/{{pull_request_number}}/head' } } }, '{{name}}-11' => { name: '{{name}}-11', type: :'job-template', value: { name: '{{name}}-11', description: '{{description}}', scm_branch: 'origin/pr/{{pull_request_number}}/head', scm_params: { changelog_to_branch: { remote: 'origin', branch: 'pr/{{pull_request_number}}/head' },  refspec: 'refs/pull/{{pull_request_number}}/head:refs/remotes/origin/pr/{{pull_request_number}}/head' } } } }
       post_project = { name: 'pull_req_test-PR2', type: :project, value: { name: 'pull_req_test-PR2', jobs: ['{{name}}-00', '{{name}}-10', '{{name}}-11'], pull_request_number: '2' } }
 
       expect(pull.project).to eq(post_project)
@@ -23,7 +29,7 @@ describe JenkinsPipelineBuilder::PullRequest do
     end
     it 'process pull_request' do
       pull = pull_request_class.new(project, 2, jobs, pull_request)
-      post_jobs = { '{{name}}-10' => { name: '{{name}}-10', type: :'job-template', value: { name: '{{name}}-10', description: '{{description}}', publishers: [{ downstream: { project: '{{job@{{name}}-11}}' } }], scm_branch: 'origin/pr/{{pull_request_number}}/head', scm_params: { refspec: 'refs/pull/*:refs/remotes/origin/pr/*' } } }, '{{name}}-11' => { name: '{{name}}-11', type: :'job-template', value: { name: '{{name}}-11', description: '{{description}}', scm_branch: 'origin/pr/{{pull_request_number}}/head', scm_params: { refspec: 'refs/pull/*:refs/remotes/origin/pr/*' } } } }
+      post_jobs = { '{{name}}-10' => { name: '{{name}}-10', type: :'job-template', value: { name: '{{name}}-10', description: '{{description}}', publishers: [{ downstream: { project: '{{job@{{name}}-11}}' } }], scm_branch: 'origin/pr/{{pull_request_number}}/head', scm_params: { refspec: 'refs/pull/{{pull_request_number}}/head:refs/remotes/origin/pr/{{pull_request_number}}/head' } } }, '{{name}}-11' => { name: '{{name}}-11', type: :'job-template', value: { name: '{{name}}-11', description: '{{description}}', scm_branch: 'origin/pr/{{pull_request_number}}/head', scm_params: { refspec: 'refs/pull/{{pull_request_number}}/head:refs/remotes/origin/pr/{{pull_request_number}}/head' } } } }
       expect(pull.jobs).to eq(post_jobs)
     end
   end
@@ -34,7 +40,7 @@ describe JenkinsPipelineBuilder::PullRequest do
     end
     it 'process pull_request' do
       pull = pull_request_class.new(project, 2, jobs, pull_request)
-      post_jobs = { '{{name}}-10' => { name: '{{name}}-10', type: :'job-template', value: { name: '{{name}}-10', description: '{{description}}', publishers: [{ downstream: { project: '{{job@{{name}}-11}}' } }], scm_branch: 'origin/pr/{{pull_request_number}}/head', scm_params: { refspec: 'refs/pull/*:refs/remotes/origin/pr/*', changelog_to_branch: { remote: 'origin', branch: 'pr/{{pull_request_number}}/head' } } } }, '{{name}}-11' => { name: '{{name}}-11', type: :'job-template', value: { name: '{{name}}-11', description: '{{description}}', scm_branch: 'origin/pr/{{pull_request_number}}/head', scm_params: { refspec: 'refs/pull/*:refs/remotes/origin/pr/*', changelog_to_branch: { remote: 'origin', branch: 'pr/{{pull_request_number}}/head' } } } } }
+      post_jobs = { '{{name}}-10' => { name: '{{name}}-10', type: :'job-template', value: { name: '{{name}}-10', description: '{{description}}', publishers: [{ downstream: { project: '{{job@{{name}}-11}}' } }], scm_branch: 'origin/pr/{{pull_request_number}}/head', scm_params: { refspec: 'refs/pull/{{pull_request_number}}/head:refs/remotes/origin/pr/{{pull_request_number}}/head', changelog_to_branch: { remote: 'origin', branch: 'pr/{{pull_request_number}}/head' } } } }, '{{name}}-11' => { name: '{{name}}-11', type: :'job-template', value: { name: '{{name}}-11', description: '{{description}}', scm_branch: 'origin/pr/{{pull_request_number}}/head', scm_params: { refspec: 'refs/pull/{{pull_request_number}}/head:refs/remotes/origin/pr/{{pull_request_number}}/head', changelog_to_branch: { remote: 'origin', branch: 'pr/{{pull_request_number}}/head' } } } } }
       expect(pull.jobs).to eq(post_jobs)
     end
   end

data/spec/lib/jenkins_pipeline_builder/view_spec.rb

@@ -27,6 +27,7 @@ describe JenkinsPipelineBuilder::View do
       stub_request(:any, 'http://username:password@127.0.0.1:8080/api/json').to_return(body: '{"assignedLabels":[{}],"mode":"NORMAL","nodeDescription":"the master Jenkins node","nodeName":"","numExecutors":2,"description":null,"jobs":[{"name":"PurgeTest-PR1","url":"http://localhost:8080/job/PurgeTest-PR1/","color":"notbuilt" },{"name":"PurgeTest-PR3","url":"http://localhost:8080/job/PurgeTest-PR3/","color":"notbuilt" },{"name":"PurgeTest-PR4","url":"http://localhost:8080/job/PurgeTest-PR4/","color":"notbuilt"}],"overallLoad":{},"primaryView":{"name":"All","url":"http://localhost:8080/" },"quietingDown":false,"slaveAgentPort":0,"unlabeledLoad":{},"useCrumbs":false,"useSecurity":true,"views":[{"name":"All","url":"http://localhost:8080/"}, {"name":"duplicate_view"}]}')
       stub_request(:any, 'http://username:password@127.0.0.1:8080/view/ParentView/api/json').to_return(body: '{"assignedLabels":[{}],"mode":"NORMAL","nodeDescription":"the master Jenkins node","nodeName":"","numExecutors":2,"description":null,"jobs":[{"name":"PurgeTest-PR1","url":"http://localhost:8080/job/PurgeTest-PR1/","color":"notbuilt" },{"name":"PurgeTest-PR3","url":"http://localhost:8080/job/PurgeTest-PR3/","color":"notbuilt" },{"name":"PurgeTest-PR4","url":"http://localhost:8080/job/PurgeTest-PR4/","color":"notbuilt"}],"overallLoad":{},"primaryView":{"name":"All","url":"http://localhost:8080/" },"quietingDown":false,"slaveAgentPort":0,"unlabeledLoad":{},"useCrumbs":false,"useSecurity":true,"views":[{"name":"All","url":"http://localhost:8080/"}]}')
       stub_request(:post, /.*/).to_return(status: 200) # Stop actual creating/deleting
+      allow(JenkinsPipelineBuilder).to receive(:debug).and_return false
     end
     context 'parent view is specified' do
       it 'needs to create the parent view and view' do

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: jenkins_pipeline_builder
 version: !ruby/object:Gem::Version
-  version: 0.10.12
+  version: 0.10.13
 platform: ruby
 authors:
 - Igor Moochnick
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2015-03-25 00:00:00.000000000 Z
+date: 2015-04-01 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: nokogiri
@@ -369,7 +369,6 @@ files:
 - lib/jenkins_pipeline_builder/version.rb
 - lib/jenkins_pipeline_builder/view.rb
 - out/.keep
-- out/xml/.keep
 - pipeline/Jenkins-Pipeline-Builder.yaml
 - pipeline/project.yaml
 - spec/lib/jenkins_pipeline_builder/cli/helper_spec.rb
@@ -408,6 +407,7 @@ files:
 - spec/lib/jenkins_pipeline_builder/fixtures/view_test/parent_view.yaml
 - spec/lib/jenkins_pipeline_builder/fixtures/view_test/regular_view.yaml
 - spec/lib/jenkins_pipeline_builder/generator_spec.rb
+- spec/lib/jenkins_pipeline_builder/job_spec.rb
 - spec/lib/jenkins_pipeline_builder/module_registry_spec.rb
 - spec/lib/jenkins_pipeline_builder/pull_request_generator_spec.rb
 - spec/lib/jenkins_pipeline_builder/pull_request_spec.rb
@@ -476,6 +476,7 @@ test_files:
 - spec/lib/jenkins_pipeline_builder/fixtures/view_test/parent_view.yaml
 - spec/lib/jenkins_pipeline_builder/fixtures/view_test/regular_view.yaml
 - spec/lib/jenkins_pipeline_builder/generator_spec.rb
+- spec/lib/jenkins_pipeline_builder/job_spec.rb
 - spec/lib/jenkins_pipeline_builder/module_registry_spec.rb
 - spec/lib/jenkins_pipeline_builder/pull_request_generator_spec.rb
 - spec/lib/jenkins_pipeline_builder/pull_request_spec.rb