jenkins_job 0.1.0 → 0.1.1

data/README.rdoc

@@ -21,6 +21,12 @@ new project named foo_production testing the production branch.
 Note well: the command must be executed within the directory containing
 Jenkins' jobs, which is <jenkins_home>/jobs.
 
+You can also clone multiple projects at the same time:
+  jenkins_job clone foo_master bar_master --original_branch master --new_branch production
+
+Taking advantage of shell globbing you can even do:
+  jenkins_job clone *_master --original_branch master --new_branch production
+
 == Other features
 jenkins_job can help you in other ways.
 
@@ -44,3 +50,9 @@ Just execute this command:
 Now the bar_master project will have the irc notifications turned on and
 configured like foo_master.
 
+You can also copy a setting to multiple projects at the same time:
+  jenkins_job copy_setting hudson.plugins.ircbot.IrcPublisher foo_master foo_production bar_master
+
+Taking advantage of shell globbing you can even do:
+  jenkins_job copy_setting hudson.plugins.ircbot.IrcPublisher foo_master bar*
+

data/lib/jenkins_job/cli.rb

@@ -60,19 +60,22 @@ module JenkinsJob
       end
     end
 
-    desc "copy_setting SOURCE_PROJECT SETTING_NAME DEST_PROJECT",
-         "Copy SETTING_NAME from SOURCE_PROJECT to DEST_PROJECT"
-    def copy_setting source_project, setting_name, dest_project
+    desc "copy_setting SOURCE_PROJECT SETTING_NAME DEST_PROJECT [DEST_PROJECT, ...]",
+         "Copy SETTING_NAME from SOURCE_PROJECT to the specified destination projects"
+    def copy_setting source_project, setting_name, *dest_project
       source = Project.new(source_project)
-      dest   = Project.new(dest_project)
-     
       setting = source.setting(setting_name)
       if setting.nil?
-        say "#{source_project} doesn't contain the specified setting", :red
-      else
-        dest.update_setting(setting_name, setting)
-        say "#{setting_name} copied", :green
+        say("#{source_project} doesn't contain the specified setting", :red)
+        Kernel::exit(1)
+      end
+
+      dest_project.each do |p|
+       dest = Project.new(p)
+       dest.update_setting(setting_name, setting)
+       say("#{setting_name} copied to #{p}", :green)
       end
     end
+
   end
 end

data/lib/jenkins_job/version.rb

@@ -1,3 +1,3 @@
 module JenkinsJob
-  VERSION = "0.1.0"
+  VERSION = "0.1.1"
 end

data/test/cli_test.rb

@@ -75,6 +75,7 @@ class CliTest < Test::Unit::TestCase
 
       FakeFS.activate!
       FakeFS::FileSystem.clear
+
       FileUtils.mkdir_p 'foo_master'
       File.open('foo_master/config.xml', 'w') do |file|
         file.write @source_xml
@@ -111,7 +112,7 @@ class CliTest < Test::Unit::TestCase
 
       context 'force overwrite enabled' do
         should 'not prompt user when force is set to true' do
-          JenkinsJob::Cli.any_instance.expects(:yes?).never
+          Thor::Shell::Color.any_instance.expects(:yes?).never
           
           capture(:stdout) do
             JenkinsJob::Cli.start(['clone', 'foo_master', '--original_branch',
@@ -131,7 +132,7 @@ class CliTest < Test::Unit::TestCase
       context 'force overwrite is not set' do
         context 'user wants to overwrite' do
           should 'overwrite the old contents and create the new ones' do
-            JenkinsJob::Cli.any_instance.expects(:yes?).once.returns(true)
+            Thor::Shell::Color.any_instance.expects(:yes?).once.returns(true)
             capture(:stdout) do
               JenkinsJob::Cli.start(['clone', 'foo_master', '--original_branch',
                                     'master', '--new_branch', 'production'])
@@ -147,7 +148,7 @@ class CliTest < Test::Unit::TestCase
 
         context "user doesn't want to overwrite" do
           should 'preserve the old contents' do
-            JenkinsJob::Cli.any_instance.expects(:yes?).once.returns(false)
+            Thor::Shell::Color.any_instance.expects(:yes?).once.returns(false)
             capture(:stdout) do
               JenkinsJob::Cli.start(['clone', 'foo_master', '--original_branch',
                                     'master', '--new_branch', 'production'])
@@ -170,33 +171,57 @@ class CliTest < Test::Unit::TestCase
       source = mock()
       source.expects(:setting).with('irc_bot').once.returns(nil)
 
-      dest = mock()
-      dest.expects(:update_setting).never
+      JenkinsJob::Project.expects(:new).once.returns(source)
+      Thor::Shell::Color.any_instance.expects(:say)\
+                        .with(is_a(String), :red).once
+      Kernel.expects(:exit).with(1).raises('exiting')
 
-      JenkinsJob::Project.expects(:new).twice.returns(source).then.returns(dest)
-      JenkinsJob::Cli.any_instance.expects(:say).with(is_a(String), :red).once
+      assert_raises RuntimeError do
+        capture(:stdout) do
+          JenkinsJob::Cli.start(['copy_setting', 'foo_master', 'irc_bot',
+                                'foo_production'])
+        end
+      end
+    end
 
-      capture(:stdout) do
-        JenkinsJob::Cli.start(['copy_setting', 'foo_master', 'irc_bot',
-                              'foo_production'])
+    context 'existing setting' do
+      should 'handle a single destination' do
+        source = mock()
+        source.expects(:setting).with('irc_bot').once.returns('foo')
+
+        dest = mock()
+        dest.expects(:update_setting).with('irc_bot', 'foo').once
+
+        JenkinsJob::Project.expects(:new).twice.returns(source).then.returns(dest)
+        Thor::Shell::Color.any_instance.expects(:say)\
+                          .with(is_a(String), :green) .once
+        capture(:stdout) do
+          JenkinsJob::Cli.start(['copy_setting', 'foo_master', 'irc_bot',
+                                'foo_production'])
+        end
       end
 
-    end
+      should 'handle multiple destination projects' do
+        source = mock()
+        source.expects(:setting).with('irc_bot').once.returns('foo')
 
-    should 'handle an existing setting' do
-      source = mock()
-      source.expects(:setting).with('irc_bot').once.returns('foo')
+        dest_foo_prod = mock()
+        dest_foo_prod.expects(:update_setting).with('irc_bot', 'foo').once
 
-      dest = mock()
-      dest.expects(:update_setting).with('irc_bot', 'foo').once
+        dest_bar_prod = mock()
+        dest_bar_prod.expects(:update_setting).with('irc_bot', 'foo').once
 
-      JenkinsJob::Project.expects(:new).twice.returns(source).then.returns(dest)
-      JenkinsJob::Cli.any_instance.expects(:say).with(is_a(String), :green).once
-      capture(:stdout) do
-        JenkinsJob::Cli.start(['copy_setting', 'foo_master', 'irc_bot',
-                              'foo_production'])
+        JenkinsJob::Project.expects(:new).times(3)\
+                           .returns(source).then\
+                           .returns(dest_foo_prod).then\
+                           .returns(dest_bar_prod)
+        Thor::Shell::Color.any_instance.expects(:say)\
+                          .with(is_a(String), :green).twice
+        capture(:stdout) do
+          JenkinsJob::Cli.start(['copy_setting', 'foo_master', 'irc_bot',
+                                'foo_production', 'bar_production'])
+        end
       end
     end
-
   end
 end

data/test/project_test.rb

@@ -87,26 +87,27 @@ class ProjectTest < Test::Unit::TestCase
         assert_nil actual
       end
 
-      should 'update an existing setting by overwriting it' do
-        node = REXML::Element.new 'disabled'
-        node.text = 'true'
-
-        assert @project.enabled?
-        @project.update_setting 'disabled', node
-        assert @project.disabled?
-      end
-
-      should 'update an existing setting by overwriting it' do
-        parent = REXML::Element.new 'foobar_parent'
-        node = REXML::Element.new 'foobar'
-        node.text = 'hello world'
-        parent << node
-
-
-        @project.update_setting 'foobar_parent', parent
-        actual_node = @project.setting 'foobar_parent/foobar'
-        assert_equal 1, @project.setting('foobar_parent').size
-        assert_equal node.text, actual_node.text
+      context 'update an existing setting by overwriting it' do
+        should 'handle simple nodes' do
+          node = REXML::Element.new 'disabled'
+          node.text = 'true'
+
+          assert @project.enabled?
+          @project.update_setting 'disabled', node
+          assert @project.disabled?
+        end
+
+        should 'handle complex nodes' do
+          parent = REXML::Element.new 'foobar_parent'
+          node = REXML::Element.new 'foobar'
+          node.text = 'hello world'
+          parent << node
+
+          @project.update_setting 'foobar_parent', parent
+          actual_node = @project.setting 'foobar_parent/foobar'
+          assert_equal 1, @project.setting('foobar_parent').size
+          assert_equal node.text, actual_node.text
+        end
       end
 
     end

metadata

@@ -1,13 +1,13 @@
 --- !ruby/object:Gem::Specification 
 name: jenkins_job
 version: !ruby/object:Gem::Version 
-  hash: 27
+  hash: 25
   prerelease: 
   segments: 
   - 0
   - 1
-  - 0
-  version: 0.1.0
+  - 1
+  version: 0.1.1
 platform: ruby
 authors: 
 - Flavio Castelli