depengine 3.0.20 → 3.0.21

data/lib/depengine/dsl/executor.rb

@@ -1,9 +1,9 @@
 module Deployment
   module Methods
-  	module Executor
-	    def local_execute(shell_cmds)
-	      Processor.local_execute(shell_cmds)
-	    end
-  	end
+    module Executor
+      def local_execute(shell_cmds)
+        Processor.local_execute(shell_cmds)
+      end
+    end
   end
 end

data/lib/depengine/dsl/fileops.rb

@@ -1,10 +1,9 @@
 module Deployment
   module Methods
     module Fileops
-
       # Copies a file or directory from a given source to a given target.<br>
       # Both are prefixed with the recipes +:deploy_home+.
-      def copy(source, target, options={})
+      def copy(source, target, options = {})
         Processor.copy(File.join($recipe_config[:deploy_home], source), \
                        File.join($recipe_config[:deploy_home], target), \
                        options)
@@ -23,7 +22,7 @@ module Deployment
       # * +file+ - the file to change mode on
       # * +mode+ - the new file permission mode (e.g. "644")
       # * [+options+] - a hash with additional parameters. This is optional.
-      def chmod(file, mode, options={})
+      def chmod(file, mode, options = {})
         Processor.chmod(File.join($recipe_config[:deploy_home], file), \
                         mode, options)
       end
@@ -52,4 +51,4 @@ module Deployment
       end
     end
   end
-end
+end

data/lib/depengine/dsl/helper.rb

@@ -1,7 +1,6 @@
 module Deployment
   module Methods
     module Helper
-
       # Returns an time-dependant string that can be used in pseudo-unique
       # release versions and the such.
       #
@@ -10,7 +9,7 @@ module Deployment
       #
       # Parameters:
       # * +format+ - the strftime format that is to be used to generate the unique string. Defaults to "%Y-%m-%d_%s".
-      def date_unique(format="%Y-%m-%d_%s")
+      def date_unique(format = '%Y-%m-%d_%s')
         @date_unique ||= Time.now.strftime(format)
       end
 
@@ -23,13 +22,13 @@ module Deployment
       #   * +:from+ - the senders address
       #   * +:to+ - the recipiants address
       def sendmail(options)
-        Helper.validates_presence_of options[:from],    "Mail-FROM not set"
-        Helper.validates_presence_of options[:to],      "Mail-TO not set"
-        Helper.validates_presence_of options[:subject], "No mail subject set"
-        Helper.validates_presence_of options[:body],    "No mail body set"
+        Helper.validates_presence_of options[:from],    'Mail-FROM not set'
+        Helper.validates_presence_of options[:to],      'Mail-TO not set'
+        Helper.validates_presence_of options[:subject], 'No mail subject set'
+        Helper.validates_presence_of options[:body],    'No mail body set'
 
         helper = ::Helper::Mail.new
-        helper.smtp_host = @cdb['smtp_host'] if @cdb and @cdb['smtp_host']
+        helper.smtp_host = @cdb['smtp_host'] if @cdb && @cdb['smtp_host']
         helper.sendmail(options)
       end
 
@@ -42,9 +41,9 @@ module Deployment
       #   * +:check_port+ - the port to connect to
       #   * +:check_uri+ - the recipiants address
       #   * +:check_response_string+ - the string or regexp to check
-      def assert_url_response_of(options={})
+      def assert_url_response_of(options = {})
         asserter = ::Asserter::Url.new
-        asserter.check_protocol  = options[:check_protocol] ||  @cdb['check_protocol'] || "http"
+        asserter.check_protocol  = options[:check_protocol] ||  @cdb['check_protocol'] || 'http'
         asserter.check_host  = options[:check_host] || @cdb['check_host']
         asserter.check_port  = options[:check_port] || @cdb['check_port'] || '80'
         asserter.check_uri  = options[:check_uri] || @cdb['check_uri'] || '/'
@@ -54,11 +53,11 @@ module Deployment
       end
 
       # Reports a deployment to its cdb.
-      def report_to_cdb()
+      def report_to_cdb
         reporter = ::Reporter::Cdb.new
         reporter.version  = @version.to_json
         reporter.worker  = self
-        reporter.set_version()
+        reporter.set_version
       end
 
       # Sends an email to report a deployment to a given recepiant.
@@ -71,16 +70,16 @@ module Deployment
       #   * +:message+ - a additional message to send in the mail, commonly used to specify any non positive status.
       #   * +:deploy_email_from+ - the senders address
       #   * +:deploy_email_to+ - the recipiants address
-      def report_by_mail(options={})
+      def report_by_mail(options = {})
         options = @cdb.merge($recipe_config).merge(options)
         options[:status] ||= 'success'
         options[:message] ||= ''
 
         mail_options = {
-          :from => options[:deploy_email_from] || @cdb['deploy_email_from'],
-          :to => options[:deploy_email_to] || @cdb['deploy_email_to'],
-          :subject => "#{options[:env]} #{options[:job_name]} #{options[:module_name]} #{options[:application_name]} #{options[:version]} #{options[:status]}",
-          :body => "Env: #{options[:env]}\nJob: #{options[:job_name]}\nModule: #{options[:module_name]}\nApplication: #{options[:application_name]}\nVersion: #{options[:version].to_json}\n#{options[:status]}.\n\n#{options[:message]}"
+          from: options[:deploy_email_from] || @cdb['deploy_email_from'],
+          to: options[:deploy_email_to] || @cdb['deploy_email_to'],
+          subject: "#{options[:env]} #{options[:job_name]} #{options[:module_name]} #{options[:application_name]} #{options[:version]} #{options[:status]}",
+          body: "Env: #{options[:env]}\nJob: #{options[:job_name]}\nModule: #{options[:module_name]}\nApplication: #{options[:application_name]}\nVersion: #{options[:version].to_json}\n#{options[:status]}.\n\n#{options[:message]}"
         }
         sendmail(mail_options)
       end
@@ -99,7 +98,7 @@ module Deployment
       #   * +:deploy_email_from+ - the senders address
       #   * +:deploy_email_to+ - the recipiants address
       # Please see `GITREVISIONS(7)` for more info on how to specify a revision range.
-      def report_by_mail_with_git_history(r1='', r2='', options={})
+      def report_by_mail_with_git_history(r1 = '', r2 = '', options = {})
         git = Provider::Git.new
         git.repository_local_dir = options[:git_dir] || '.'
         options[:message] = (options[:message] || '') + git.history(r1, r2).to_s
@@ -121,7 +120,7 @@ module Deployment
       #   * +:logfilename+ - the name of the logfile. Will be automatically prefixed with #{catalina_home}/logs/
       #   * +:check_host+ - an options hash to pass to +#assert_url_response_of+ to check the status of the app
       # All parameters can also be spcified via CDB.
-      def tomcat_deploy(options={})
+      def tomcat_deploy(options = {})
         publisher = ::Publisher::Tomcat.new
         publisher.servers  = options[:servers] || @cdb['servers']
         publisher.runners = options[:runners] || @cdb['runners'] || publisher.servers
@@ -140,33 +139,32 @@ module Deployment
       # Parameters:
       # * +remote_path+ - the path to the remote share (e.g."smb://somehost/someshare")
       # * +:local_path+ - the local path in which the share will be mounted
-      def samba_mount (remote_path, local_path)
+      def samba_mount(remote_path, local_path)
         smb                   = ::Helper::Smb.new
         smb.remote_path       = remote_path
         smb.local_path        = local_path
-        smb.samba_mount()
+        smb.samba_mount
       end
 
       # Unmounts a previously mounted SMB share.
       #
       # Parameters:
       # * +:local_path+ - the local path in which the share is mounted
-      def samba_umount (local_path)
+      def samba_umount(local_path)
         smb                   = ::Helper::Smb.new
         smb.local_path        = local_path
-        smb.samba_umount()
+        smb.samba_umount
       end
 
-      def validates_presence_of(attribute, message=nil)
-        raise ArgumentError, message || "Missing parameter", caller if attribute.nil?
+      def validates_presence_of(attribute, message = nil)
+        fail ArgumentError, message || 'Missing parameter', caller if attribute.nil?
       end
       module_function :validates_presence_of
 
-      def validates_not_empty(attribute, message=nil)
-        raise ArgumentError, message || "Parameter is empty", caller if attribute.nil?
+      def validates_not_empty(attribute, message = nil)
+        fail ArgumentError, message || 'Parameter is empty', caller if attribute.nil?
       end
       module_function :validates_not_empty
-
     end
   end
-end
+end

data/lib/depengine/dsl/iis.rb

@@ -1,20 +1,19 @@
 module Deployment
   module Methods
     module IIS
-
-      def iis_build()
+      def iis_build
         Helper.validates_presence_of @cdb['iis_build_url'], \
-                                     "iis_build_url parameter is missing"
+                                     'iis_build_url parameter is missing'
         Helper.validates_presence_of @cdb['iis_build_protocol'], \
-                                     "iis_build_protocol parameter is missing"
+                                     'iis_build_protocol parameter is missing'
         Helper.validates_presence_of $recipe_config['TAG'], \
-                                     "TAG parameter is missing"
+                                     'TAG parameter is missing'
         Helper.validates_presence_of $recipe_config['ENVIRONMENT'], \
-                                     "ENV parameter is missing"
-        Helper.validates_presence_of  @version['app'], \
-                                     "VERSION parameter is missing"
+                                     'ENV parameter is missing'
+        Helper.validates_presence_of @version['app'], \
+                                     'VERSION parameter is missing'
         Helper.validates_presence_of $recipe_config['VALUE'], \
-                                     "VALUE parameter is missing"
+                                     'VALUE parameter is missing'
 
         iis          = ::Publisher::IIS.new
         iis.protocol = @cdb['iis_build_protocol']
@@ -22,23 +21,23 @@ module Deployment
         if @cdb['http_read_timeout']
           iis.http_read_timeout = @cdb['http_read_timeout']
         else
-           iis.http_read_timeout = 180
+          iis.http_read_timeout = 180
         end
 
         iis.build($recipe_config['TAG'], $recipe_config['ENVIRONMENT'], @version['app'], $recipe_config['VALUE'], $recipe_config[:application_name])
       end
 
-      def iis_deploy(options={})
+      def iis_deploy(options = {})
         Helper.validates_presence_of options[:iis_deploy_url] || @cdb['iis_deploy_url'], \
-                                     "iis_deploy_url parameter is missing"
+                                     'iis_deploy_url parameter is missing'
         Helper.validates_presence_of @cdb['iis_application_name'], \
-                                     "iis_application_name parameter is missing"
+                                     'iis_application_name parameter is missing'
         Helper.validates_presence_of @cdb['iis_deploy_protocol'], \
-                                     "iis_deploy_protocol parameter is missing"
+                                     'iis_deploy_protocol parameter is missing'
         Helper.validates_presence_of $recipe_config['ENVIRONMENT'], \
-                                     "ENV parameter is missing"
-        Helper.validates_presence_of  @version['app'], \
-                                     "VERSION parameter is missing"
+                                     'ENV parameter is missing'
+        Helper.validates_presence_of @version['app'], \
+                                     'VERSION parameter is missing'
 
         iis          = ::Publisher::IIS.new
         iis.protocol = @cdb['iis_deploy_protocol']
@@ -46,18 +45,17 @@ module Deployment
         if @cdb['http_read_timeout']
           iis.http_read_timeout = @cdb['http_read_timeout']
         else
-           iis.http_read_timeout = 180
+          iis.http_read_timeout = 180
         end
 
         iis.deploy(@version['app'], @cdb['iis_application_name'], $recipe_config['ENVIRONMENT'].downcase)
       end
 
-
-      def iis_appcmd()
+      def iis_appcmd
         Helper.validates_presence_of @cdb['iis_appcmd_url'], \
-                                     "iis_appcmd_url parameter is missing"
+                                     'iis_appcmd_url parameter is missing'
         Helper.validates_presence_of $recipe_config['VALUE'], \
-                                     "VALUE parameter is missing"
+                                     'VALUE parameter is missing'
 
         iis          = ::Publisher::IIS.new
         iis.protocol = @cdb['iis_appcmd_protocol']
@@ -65,10 +63,9 @@ module Deployment
         if @cdb['http_read_timeout']
           iis.http_read_timeout = @cdb['http_read_timeout']
         else
-           iis.http_read_timeout = 180
+          iis.http_read_timeout = 180
         end
         iis.appcmd($recipe_config['VALUE'])
-
       end
     end
   end

data/lib/depengine/dsl/patch.rb

@@ -1,10 +1,9 @@
 module Deployment
   module Methods
     module Patch
-
-      def patch_properties (source, target, options={})
+      def patch_properties(source, target, options = {})
         patch_set = options[:patch_set] || 'patch_properties'
-        Helper.validates_presence_of @cdb[patch_set], "Can not find a valid patch_set"
+        Helper.validates_presence_of @cdb[patch_set], 'Can not find a valid patch_set'
 
         assigner = options[:assigner] || '='
 
@@ -13,49 +12,34 @@ module Deployment
         properties.assigner        = assigner
         properties.patch(File.join($recipe_config[:deploy_home], source), \
                          File.join($recipe_config[:deploy_home], target,  \
-                         File.basename(source)))
+                                   File.basename(source)))
       end
 
-      def patch_strings (source, target, options={})
-        patch_set = 'patch_properties'
-        if not options[:patch_set].nil?
-          patch_set = options[:patch_set]
-        end
-        Helper.validates_presence_of @cdb[patch_set],  "Properties not set"
+      def patch_strings(source, target, options = {})
+        patch_set = options[:patch_set].nil? ? 'patch_properties' : options[:patch_set]
 
-        assigner = '='
-        if not options[:assigner].nil?
-          assigner = options[:assigner]
-        end
+        Helper.validates_presence_of @cdb[patch_set],  'Properties not set'
 
         properties                 = Processor::Properties.new
-        properties.assigner        = assigner
+        properties.assigner        = options[:assigner] || '='
         properties.properties_hash = @cdb[patch_set]
         ### add db_endpoint
         properties.properties_hash['db_endpoint_1'] = @cdb['db_endpoint_1']
-        properties.properties_hash['db_endpoint_2'] = @cdb['db_endpoint_2']      
+        properties.properties_hash['db_endpoint_2'] = @cdb['db_endpoint_2']
         properties.substitute(File.join($recipe_config[:deploy_home], source), \
-                         File.join($recipe_config[:deploy_home], target,  \
-                         File.basename(source)))
+                              File.join($recipe_config[:deploy_home], target,  \
+                                        File.basename(source)))
       end
 
-      def patch_strings_r (source, target, options={})
-        patch_set   = 'patch_properties'
-        if not options[:patch_set].nil?
-          patch_set = options[:patch_set]
-        end
-        Helper.validates_presence_of @cdb[patch_set],  "Properties not set"
-
-        assigner = '='
-        if not options[:assigner].nil?
-          assigner = options[:assigner]
-        end
+      def patch_strings_r(source, target, options = {})
+        patch_set = options[:patch_set] || 'patch_properties'
+        Helper.validates_presence_of @cdb[patch_set],  'Properties not set'
 
         source_path = File.join($recipe_config[:deploy_home], source)
         target_path = File.join($recipe_config[:deploy_home], target)
 
         properties                 = Processor::Properties.new
-        properties.assigner        = assigner
+        properties.assigner        = options[:assigner] || '='
         properties.properties_hash = @cdb[patch_set]
         ### add db_endpoint
         properties.properties_hash['db_endpoint_1'] = @cdb['db_endpoint_1']
@@ -63,28 +47,20 @@ module Deployment
         properties.substitute_r(source_path, target_path)
       end
 
-      def add_properties (source, target, options={})
-        patch_set = 'patch_properties'
-        if not options[:patch_set].nil?
-          patch_set = options[:patch_set]
-        end
-        Helper.validates_presence_of @cdb[patch_set],  "Properties not set"
-
-        assigner = '='
-        if not options[:assigner].nil?
-          assigner = options[:assigner]
-        end
+      def add_properties(source, target, options = {})
+        patch_set = options[:patch_set] || 'patch_properties'
+        Helper.validates_presence_of @cdb[patch_set],  'Properties not set'
 
         properties                 = Processor::Properties.new
-        properties.assigner        = assigner
+        properties.assigner        = options[:assigner] || '='
         properties.properties_hash = @cdb[patch_set]
         properties.add(File.join($recipe_config[:deploy_home], source), \
-                         File.join($recipe_config[:deploy_home], target,  \
-                         File.basename(source)))
+                       File.join($recipe_config[:deploy_home], target,  \
+                                 File.basename(source)))
       end
 
-      def sed_strings()
-        Sed.copy($recipe_config[:source],$recipe_config[:target], $recipe_config[:pattern],$recipe_config[:replacement])
+      def sed_strings
+        Sed.copy($recipe_config[:source], $recipe_config[:target], $recipe_config[:pattern], $recipe_config[:replacement])
       end
     end
   end

data/lib/depengine/dsl/publisher.rb

@@ -1,7 +1,7 @@
 module Deployment
   module Methods
     module Publisher
-      def samba_upload(source, target, options={})
+      def samba_upload(source, target, options = {})
         Helper.validates_presence_of source
         Helper.validates_presence_of target
 
@@ -10,15 +10,15 @@ module Deployment
         publisher.worker = self
         publisher.samba_mountpoints      = options[:samba_mountpoints]      || @cdb['samba_mountpoints']
 
-        Helper.validates_presence_of publisher.samba_share, "Sambe share not set"
-        Helper.validates_presence_of publisher.samba_mountpoints, "Sambe mountpoints not set"
+        Helper.validates_presence_of publisher.samba_share, 'Sambe share not set'
+        Helper.validates_presence_of publisher.samba_mountpoints, 'Sambe mountpoints not set'
 
         publisher.copy(File.join($recipe_config[:deploy_home], source), \
-                         target, options)
+                       target, options)
       end
 
-      def sftp_upload(source, target, options={})
-        Helper.validates_presence_of @cdb['ssh_key_file'], "SSH-keyfile not set"
+      def sftp_upload(source, target, options = {})
+        Helper.validates_presence_of @cdb['ssh_key_file'], 'SSH-keyfile not set'
         Helper.validates_presence_of source
         Helper.validates_presence_of target
 
@@ -30,15 +30,15 @@ module Deployment
         publisher.ssh_key_file = File.join(File.dirname($exec_file_path), \
                                            publisher.ssh_key_file)
 
-        Helper.validates_presence_of publisher.remote_host, "Remote host not set"
-        Helper.validates_presence_of publisher.remote_user, "Remote user not set"
+        Helper.validates_presence_of publisher.remote_host, 'Remote host not set'
+        Helper.validates_presence_of publisher.remote_user, 'Remote user not set'
 
         publisher.copy(File.join($recipe_config[:deploy_home], source), \
-                         target, options)
+                       target, options)
       end
 
-      def sftp_chmod(path, permissions, options={})
-        Helper.validates_presence_of @cdb['ssh_key_file'], "SSH-keyfile not set"
+      def sftp_chmod(path, permissions, options = {})
+        Helper.validates_presence_of @cdb['ssh_key_file'], 'SSH-keyfile not set'
         Helper.validates_presence_of path
         Helper.validates_presence_of permissions
 
@@ -50,11 +50,11 @@ module Deployment
         publisher.ssh_key_file = File.join(File.dirname($exec_file_path), \
                                            publisher.ssh_key_file)
 
-        Helper.validates_presence_of publisher.remote_host, "Remote host not set"
-        Helper.validates_presence_of publisher.remote_user, "Remote user not set"
+        Helper.validates_presence_of publisher.remote_host, 'Remote host not set'
+        Helper.validates_presence_of publisher.remote_user, 'Remote user not set'
 
         publisher.chmod(path, permissions, options)
       end
     end
   end
-end
+end

data/lib/depengine/dsl/remote.rb

@@ -1,10 +1,9 @@
 module Deployment
   module Methods
     module Remote
-
-      def remote_execute(command, options={})
-        Helper.validates_presence_of command, "No command to execute!"
-        Helper.validates_presence_of @cdb['ssh_key_file'], "SSH keyfile not set"
+      def remote_execute(command, options = {})
+        Helper.validates_presence_of command, 'No command to execute!'
+        Helper.validates_presence_of @cdb['ssh_key_file'], 'SSH keyfile not set'
 
         publisher = ::Publisher::Ssh.new
         publisher.remote_host  = options[:remote_host]  || @cdb['remote_host']
@@ -13,9 +12,9 @@ module Deployment
         publisher.remote_execute(command)
       end
 
-      def rsync(source, target, options={})
+      def rsync(source, target, options = {})
         if options[:local].nil? or options[:local] == false
-          Helper.validates_presence_of @cdb['ssh_key_file'], "SSH-keyfile not set"
+          Helper.validates_presence_of @cdb['ssh_key_file'], 'SSH-keyfile not set'
         end
         Helper.validates_presence_of source
         Helper.validates_presence_of target
@@ -29,11 +28,11 @@ module Deployment
         #                                    publisher.ssh_key_file)
 
         if options[:local].nil? or options[:local] == false
-          Helper.validates_presence_of publisher.remote_host, "Remote host not set"
-          Helper.validates_presence_of publisher.remote_user, "Remote user not set"
+          Helper.validates_presence_of publisher.remote_host, 'Remote host not set'
+          Helper.validates_presence_of publisher.remote_user, 'Remote user not set'
         end
 
-        if source[0].chr == "/"
+        if source[0].chr == '/'
           source_path = source
         else
           source_path = File.join($recipe_config[:deploy_home], source)
@@ -42,8 +41,8 @@ module Deployment
         publisher.sync(source_path, target, options)
       end
 
-      def remove_old_releases(path, num_of_releases_to_keep=1, release_name_shema=nil, options={})
-        Helper.validates_presence_of path, "No path in which the releases are"
+      def remove_old_releases(path, num_of_releases_to_keep = 1, release_name_shema = nil, options = {})
+        Helper.validates_presence_of path, 'No path in which the releases are'
 
         release_name_shema = /[0-9]{4}\-[0-9]{2}\-[0-9]{2}_[0-9]+.*/ unless release_name_shema # e.g. timestamps like 2014-04-11_1397227004
 
@@ -53,7 +52,6 @@ module Deployment
         publisher.ssh_key_file = options[:ssh_key_file] || @cdb['ssh_key_file']
         publisher.remove_old_releases(path, num_of_releases_to_keep, release_name_shema)
       end
-
     end
   end
-end
+end

data/lib/depengine/dsl/repository.rb

@@ -1,34 +1,33 @@
 module Deployment
   module Methods
     module Repository
-
       def get_dir_from_samba(source, target)
-        Helper.validates_presence_of @cdb['samba_repository'], "Samba repository not set"
+        Helper.validates_presence_of @cdb['samba_repository'], 'Samba repository not set'
 
         samba            = Provider::Repository.new
         samba.repository = @cdb['samba_repository']
         samba.samba_mountpoints = @cdb['samba_mountpoints']
-        samba.method     = "samba"
+        samba.method     = 'samba'
         samba.worker  = self
         samba.get_from_repository(source, File.join($recipe_config[:deploy_home], target))
       end
 
       def get_file_from_maven(source, target)
-        Helper.validates_presence_of @cdb['maven_repository'], "Maven repository not set"
+        Helper.validates_presence_of @cdb['maven_repository'], 'Maven repository not set'
 
         maven            = Provider::Repository.new
         maven.repository = @cdb['maven_repository']
         maven.user       = @cdb['maven_user']
         maven.password   = @cdb['maven_password']
-        maven.method     = "maven"
+        maven.method     = 'maven'
         maven.get_from_repository(source, File.join($recipe_config[:deploy_home], target))
       end
 
-      def get_file_via_scp(source, target, options={})
+      def get_file_via_scp(source, target, options = {})
         Helper.validates_presence_of @cdb['scp_source_host'], \
-        "scp source host not set"
+                                     'scp source host not set'
         Helper.validates_presence_of @cdb['scp_source_user'], \
-                          "scp source user not set"
+                                     'scp source user not set'
 
         scp            = Provider::Repository.new
         scp.host       = @cdb['scp_source_host']
@@ -40,25 +39,25 @@ module Deployment
           scp.sshkey   = File.join(File.dirname($exec_file_path), \
                                    @cdb['ssh_key_file'])
         end
-        scp.method     = "scp"
+        scp.method     = 'scp'
         scp.get_from_repository(source, File.join($recipe_config[:deploy_home], target))
       end
 
-      def get_file_via_svn(source, target, options={})
+      def get_file_via_svn(source, target, options = {})
         Helper.validates_presence_of @cdb['svn_binary'], \
-                          "svn binary not set"
+                                     'svn binary not set'
         Helper.validates_presence_of @cdb['svn_source_base_url'], \
-                          "svn base url  not set"
+                                     'svn base url  not set'
         Helper.validates_presence_of @cdb['svn_source_user'], \
-                          "svn source user not set"
+                                     'svn source user not set'
         Helper.validates_presence_of @cdb['svn_source_command'], \
-                          "svn command not set"
+                                     'svn command not set'
 
         svn            = Provider::Repository.new
         svn.svnbinary  = @cdb['svn_binary']
         svn.user       = @cdb['svn_source_user']
         svn.password   = @cdb['svn_source_password'] if @cdb['svn_source_password']
-        svn.method     = "svn"
+        svn.method     = 'svn'
 
         if options[:svn_command]
           svn.svncmd     = options[:svn_command]
@@ -76,34 +75,33 @@ module Deployment
                                 File.join($recipe_config[:deploy_home], target))
       end
 
-      def git_fetch(options={})
+      def git_fetch(options = {})
         publisher = ::Provider::Git.new
         publisher.repository_url = options[:git_repository_url] || @cdb['git_repository_url']
         publisher.repository_local_dir = options[:git_repository_local] || @cdb['git_repository_local']
-        publisher.fetch()
+        publisher.fetch
       end
 
-      def git_checkout(branch_name, options={})
+      def git_checkout(branch_name, options = {})
         publisher = ::Provider::Git.new
         publisher.repository_url = options[:git_repository_url] || @cdb['git_repository_url']
         publisher.repository_local_dir = options[:git_repository_local] || @cdb['git_repository_local']
         publisher.checkout(branch_name, options)
       end
 
-      def git_submodule(submodule_option, options={})
+      def git_submodule(submodule_option, options = {})
         publisher = ::Provider::Git.new
         publisher.repository_url = options[:git_repository_url] || @cdb['git_repository_url']
         publisher.repository_local_dir = options[:git_repository_local] || @cdb['git_repository_local']
         publisher.submodule(submodule_option, options)
       end
 
-      def git_tag(tag_name, options={})
+      def git_tag(tag_name, options = {})
         publisher = ::Provider::Git.new
         publisher.repository_url = options[:git_repository_url] || @cdb['git_repository_url']
         publisher.repository_local_dir = options[:git_repository_local] || @cdb['git_repository_local']
         publisher.tag(tag_name, options)
       end
-
     end
   end
 end