From af2ee063abea8235a8cbd1448533e4f4fbc0d0af Mon Sep 17 00:00:00 2001 From: Steve Kemp Date: Mon, 9 Mar 2015 13:19:15 +0000 Subject: Prefer single-quotes when you don't need interpolation. So "foo" is less good than 'foo'. --- bin/custodian-dequeue | 26 +++++----- bin/custodian-enqueue | 26 +++++----- bin/custodian-queue | 20 ++++---- lib/custodian/alertfactory.rb | 8 +-- lib/custodian/alerts/file.rb | 4 +- lib/custodian/alerts/graphite.rb | 4 +- lib/custodian/alerts/mauve.rb | 14 +++--- lib/custodian/alerts/redis-state.rb | 42 ++++++++-------- lib/custodian/alerts/smtp.rb | 4 +- lib/custodian/parser.rb | 14 +++--- lib/custodian/protocoltest/dns.rb | 8 +-- lib/custodian/protocoltest/dnsbl.rb | 8 +-- lib/custodian/protocoltest/ftp.rb | 2 +- lib/custodian/protocoltest/http.rb | 18 +++---- lib/custodian/protocoltest/imap.rb | 2 +- lib/custodian/protocoltest/jabber.rb | 2 +- lib/custodian/protocoltest/ldap.rb | 4 +- lib/custodian/protocoltest/mx.rb | 6 +-- lib/custodian/protocoltest/mysql.rb | 2 +- lib/custodian/protocoltest/named.rb | 2 +- lib/custodian/protocoltest/openproxy.rb | 12 ++--- lib/custodian/protocoltest/ping.rb | 6 +-- lib/custodian/protocoltest/pop3.rb | 4 +- lib/custodian/protocoltest/postgresql.rb | 2 +- lib/custodian/protocoltest/redis.rb | 2 +- lib/custodian/protocoltest/rsync.rb | 2 +- lib/custodian/protocoltest/smtp.rb | 2 +- lib/custodian/protocoltest/smtprelay.rb | 20 ++++---- lib/custodian/protocoltest/ssh.rb | 2 +- lib/custodian/protocoltest/ssl.rb | 16 +++--- lib/custodian/protocoltest/tcp.rb | 8 +-- lib/custodian/protocoltest/telnet.rb | 2 +- lib/custodian/queue.rb | 28 +++++------ lib/custodian/settings.rb | 28 +++++------ lib/custodian/testfactory.rb | 6 +-- lib/custodian/util/ping.rb | 6 +-- lib/custodian/util/timespan.rb | 2 +- lib/custodian/worker.rb | 18 +++---- t/test-custodian-alertfactory.rb | 14 +++--- t/test-custodian-parser.rb | 50 +++++++++---------- t/test-custodian-queue.rb | 6 +-- t/test-custodian-settings.rb | 8 +-- t/test-custodian-testfactory.rb | 84 ++++++++++++++++---------------- t/test-custodian-util-bytemark.rb | 10 ++-- t/test-custodian-util-dns.rb | 28 +++++------ t/test-custodian-util-ping.rb | 6 +-- t/test-custodian-util-timespan.rb | 40 +++++++-------- t/test-http-vs-https.rb | 16 +++--- t/test-ldap-probe.rb | 4 +- t/test-suite | 2 +- 50 files changed, 325 insertions(+), 325 deletions(-) diff --git a/bin/custodian-dequeue b/bin/custodian-dequeue index 48dea57..7a4b629 100755 --- a/bin/custodian-dequeue +++ b/bin/custodian-dequeue @@ -69,23 +69,23 @@ if __FILE__ == $PROGRAM_NAME then begin opts = GetoptLong.new( - [ "--help", "-h", GetoptLong::NO_ARGUMENT ], - [ "--manual", "-m", GetoptLong::NO_ARGUMENT ], - [ "--fail", "-f", GetoptLong::NO_ARGUMENT ], - [ "--single", "-s", GetoptLong::NO_ARGUMENT ], - [ "--verbose", "-v", GetoptLong::NO_ARGUMENT ] + [ '--help', '-h', GetoptLong::NO_ARGUMENT ], + [ '--manual', '-m', GetoptLong::NO_ARGUMENT ], + [ '--fail', '-f', GetoptLong::NO_ARGUMENT ], + [ '--single', '-s', GetoptLong::NO_ARGUMENT ], + [ '--verbose', '-v', GetoptLong::NO_ARGUMENT ] ) opts.each do |opt, arg| case opt - when "--verbose" then - ENV["VERBOSE"] = "1" - when "--single" then - ENV["SINGLE"] = "1" - when "--fail" then - ENV["FAIL"] = "1" - when "--help" then + when '--verbose' then + ENV['VERBOSE'] = '1' + when '--single' then + ENV['SINGLE'] = '1' + when '--fail' then + ENV['FAIL'] = '1' + when '--help' then $help = true - when "--manual" then + when '--manual' then $manual = true end end diff --git a/bin/custodian-enqueue b/bin/custodian-enqueue index b1be84d..83e462e 100755 --- a/bin/custodian-enqueue +++ b/bin/custodian-enqueue @@ -70,23 +70,23 @@ if __FILE__ == $PROGRAM_NAME then begin opts = GetoptLong.new( - [ "--dump", "-d", GetoptLong::NO_ARGUMENT ], - [ "--test", GetoptLong::NO_ARGUMENT ], - [ "--file", "-f", GetoptLong::REQUIRED_ARGUMENT ], - [ "--help", "-h", GetoptLong::NO_ARGUMENT ], - [ "--manual","-m", GetoptLong::NO_ARGUMENT ] + [ '--dump', '-d', GetoptLong::NO_ARGUMENT ], + [ '--test', GetoptLong::NO_ARGUMENT ], + [ '--file', '-f', GetoptLong::REQUIRED_ARGUMENT ], + [ '--help', '-h', GetoptLong::NO_ARGUMENT ], + [ '--manual','-m', GetoptLong::NO_ARGUMENT ] ) opts.each do |opt, arg| case opt - when "--dump" then - ENV["DUMP"] = "1" - when "--test" then - ENV["TEST"] = "1" - when "--file" then - ENV["FILE"] = arg - when "--help" then + when '--dump' then + ENV['DUMP'] = '1' + when '--test' then + ENV['TEST'] = '1' + when '--file' then + ENV['FILE'] = arg + when '--help' then $help = true - when "--manual" then + when '--manual' then $manual = true end end diff --git a/bin/custodian-queue b/bin/custodian-queue index 66bf2ba..26f214d 100755 --- a/bin/custodian-queue +++ b/bin/custodian-queue @@ -63,27 +63,27 @@ if __FILE__ == $PROGRAM_NAME then begin opts = GetoptLong.new( - [ "--flush", "-f", GetoptLong::NO_ARGUMENT ], - [ "--help", "-h", GetoptLong::NO_ARGUMENT ], - [ "--manual", "-m", GetoptLong::NO_ARGUMENT ], - [ "--monitor", "-M", GetoptLong::OPTIONAL_ARGUMENT ], - [ "--stats", "-s", GetoptLong::NO_ARGUMENT ] + [ '--flush', '-f', GetoptLong::NO_ARGUMENT ], + [ '--help', '-h', GetoptLong::NO_ARGUMENT ], + [ '--manual', '-m', GetoptLong::NO_ARGUMENT ], + [ '--monitor', '-M', GetoptLong::OPTIONAL_ARGUMENT ], + [ '--stats', '-s', GetoptLong::NO_ARGUMENT ] ) opts.each do |opt, arg| case opt - when "--monitor" then + when '--monitor' then if arg $MONITOR = arg.to_i else $MONITOR = 5000 end - when "--stats" then + when '--stats' then $STATS = true - when "--flush" then + when '--flush' then $FLUSH = true - when "--help" then + when '--help' then $help = true - when "--manual" then + when '--manual' then $manual = true end end diff --git a/lib/custodian/alertfactory.rb b/lib/custodian/alertfactory.rb index 1cb4751..78ad9cf 100644 --- a/lib/custodian/alertfactory.rb +++ b/lib/custodian/alertfactory.rb @@ -46,8 +46,8 @@ module Custodian # def self.create( alert_type, obj ) - raise ArgumentError, "The type of notifier to create cannot be nil" if alert_type.nil? - raise ArgumentError, "The type of notifier to create must be a string" unless alert_type.kind_of? String + raise ArgumentError, 'The type of notifier to create cannot be nil' if alert_type.nil? + raise ArgumentError, 'The type of notifier to create must be a string' unless alert_type.kind_of? String c = @@subclasses[alert_type] if c @@ -112,7 +112,7 @@ module Custodian # Raise an alert. # def raise - puts "NOP" + puts 'NOP' end @@ -121,7 +121,7 @@ module Custodian # Clear an alert. # def clear - puts "NOP" + puts 'NOP' end diff --git a/lib/custodian/alerts/file.rb b/lib/custodian/alerts/file.rb index 5874136..cb04084 100644 --- a/lib/custodian/alerts/file.rb +++ b/lib/custodian/alerts/file.rb @@ -55,7 +55,7 @@ module Custodian # Write the actual message to our target. # def write_message( msg ) - file = File.open(@target, "a") + file = File.open(@target, 'a') file.puts( "#{Time.now} #{msg}" ) file.close @@ -64,7 +64,7 @@ module Custodian - register_alert_type "file" + register_alert_type 'file' diff --git a/lib/custodian/alerts/graphite.rb b/lib/custodian/alerts/graphite.rb index 778b36b..c98f6bf 100644 --- a/lib/custodian/alerts/graphite.rb +++ b/lib/custodian/alerts/graphite.rb @@ -52,7 +52,7 @@ module Custodian # # hostname + test-type # - host = @test.target.gsub(/[\/\\.]/, "_") + host = @test.target.gsub(/[\/\\.]/, '_') test = @test.get_type # @@ -69,7 +69,7 @@ module Custodian end - register_alert_type "graphite" + register_alert_type 'graphite' end diff --git a/lib/custodian/alerts/mauve.rb b/lib/custodian/alerts/mauve.rb index 92c38ee..4c20b08 100644 --- a/lib/custodian/alerts/mauve.rb +++ b/lib/custodian/alerts/mauve.rb @@ -49,7 +49,7 @@ module Custodian require 'mauve/proto' @loaded = true rescue - puts "ERROR Loading mauve libraries!" + puts 'ERROR Loading mauve libraries!' @loaded = false end end @@ -96,14 +96,14 @@ module Custodian # # Lookup the start of the day. # - day_start = @settings.key( "day_start" ).to_i || 10 - day_end = @settings.key( "day_end" ).to_i || 18 + day_start = @settings.key( 'day_start' ).to_i || 10 + day_end = @settings.key( 'day_end' ).to_i || 18 # # In hour suppress # - working_suppress = @settings.key( "working_suppress" ).to_i || 4 - oncall_suppress = @settings.key( "oncall_suppress" ).to_i || 10 + working_suppress = @settings.key( 'working_suppress' ).to_i || 4 + oncall_suppress = @settings.key( 'oncall_suppress' ).to_i || 10 # # If we're Monday-Friday, between the start & end time, then @@ -307,7 +307,7 @@ module Custodian # # Did we get an error? # - return "" unless !resolved.nil? + return '' unless !resolved.nil? # @@ -330,7 +330,7 @@ module Custodian end - register_alert_type "mauve" + register_alert_type 'mauve' diff --git a/lib/custodian/alerts/redis-state.rb b/lib/custodian/alerts/redis-state.rb index f37cef2..27bb2c4 100644 --- a/lib/custodian/alerts/redis-state.rb +++ b/lib/custodian/alerts/redis-state.rb @@ -39,7 +39,7 @@ module Custodian @redis = Redis.new(:host => @target ) rescue - puts "ERROR Loading redis rubygem!" + puts 'ERROR Loading redis rubygem!' end @test = obj @@ -55,16 +55,16 @@ module Custodian return unless( @redis ) tmp = {} - tmp["time"] = Time.now.to_i - tmp["type"] = @test.get_type - tmp["target"] = @test.target - tmp["result"] = "RAISE" - tmp["reason"] = @test.error - tmp["test" ] = @test.to_s - tmp["class" ] = @test.class + tmp['time'] = Time.now.to_i + tmp['type'] = @test.get_type + tmp['target'] = @test.target + tmp['result'] = 'RAISE' + tmp['reason'] = @test.error + tmp['test' ] = @test.to_s + tmp['class' ] = @test.class - @redis.lpush( "recent-tests", tmp.to_json) - @redis.ltrim( "recent-tests", 0, 100 ) + @redis.lpush( 'recent-tests', tmp.to_json) + @redis.ltrim( 'recent-tests', 0, 100 ) end @@ -79,19 +79,19 @@ module Custodian tmp = {} - tmp["time"] = Time.now.to_i - tmp["type"] = @test.get_type - tmp["target"] = @test.target - tmp["result"] = "OK" - tmp["reason"] = "" - tmp["test" ] = @test.to_s - tmp["class" ] = @test.class - - @redis.lpush( "recent-tests", tmp.to_json) - @redis.ltrim( "recent-tests", 0, 100 ) + tmp['time'] = Time.now.to_i + tmp['type'] = @test.get_type + tmp['target'] = @test.target + tmp['result'] = 'OK' + tmp['reason'] = '' + tmp['test' ] = @test.to_s + tmp['class' ] = @test.class + + @redis.lpush( 'recent-tests', tmp.to_json) + @redis.ltrim( 'recent-tests', 0, 100 ) end - register_alert_type "redis" + register_alert_type 'redis' end end diff --git a/lib/custodian/alerts/smtp.rb b/lib/custodian/alerts/smtp.rb index e418031..38ef29e 100644 --- a/lib/custodian/alerts/smtp.rb +++ b/lib/custodian/alerts/smtp.rb @@ -62,14 +62,14 @@ Subject: #{subject} #{body} END_OF_MESSAGE - Net::SMTP.start("127.0.0.1") do |smtp| + Net::SMTP.start('127.0.0.1') do |smtp| smtp.send_message( msg, to, to) end end - register_alert_type "smtp" + register_alert_type 'smtp' diff --git a/lib/custodian/parser.rb b/lib/custodian/parser.rb index c75a050..6942da8 100644 --- a/lib/custodian/parser.rb +++ b/lib/custodian/parser.rb @@ -73,7 +73,7 @@ module Custodian http.open_timeout = 60 http.read_timeout = 60 - if (url.scheme == "https") + if (url.scheme == 'https') http.use_ssl = true http.verify_mode = OpenSSL::SSL::VERIFY_NONE end @@ -89,7 +89,7 @@ module Custodian if ( response.code.to_i != 200 ) puts "Status code of #{uri_str} was #{response.code}" - puts "ABORTING" + puts 'ABORTING' exit( 0 ) end @@ -104,11 +104,11 @@ module Custodian end rescue Errno::EHOSTUNREACH => ex - raise ex, "no route to host" + raise ex, 'no route to host' rescue Timeout::Error => ex - raise ex, "timeout" + raise ex, 'timeout' rescue Errno::ECONNREFUSED => ex - raise ex, "connection refused" + raise ex, 'connection refused' end end @@ -135,7 +135,7 @@ module Custodian # # HTTP-fetch # - uri = $1.dup.chomp(".") + uri = $1.dup.chomp('.') text = get_url_contents(uri) text.split( /[\r\n]/ ).each do |line| @@ -411,7 +411,7 @@ module Custodian # def parse_file( filename ) - raise ArgumentError, "Missing configuration file!" if filename.nil? + raise ArgumentError, 'Missing configuration file!' if filename.nil? raise ArgumentError, "File not found: #{@file}" unless File.exist?( filename) # diff --git a/lib/custodian/protocoltest/dns.rb b/lib/custodian/protocoltest/dns.rb index 8d47e14..7f4b997 100644 --- a/lib/custodian/protocoltest/dns.rb +++ b/lib/custodian/protocoltest/dns.rb @@ -64,9 +64,9 @@ module Custodian # # Ensure we had all the data. # - raise ArgumentError, "Missing host to resolve" unless( @resolve_name ) - raise ArgumentError, "Missing type of record to lookup" unless( @resolve_type ) - raise ArgumentError, "Missing expected results" unless( @resolve_expected ) + raise ArgumentError, 'Missing host to resolve' unless( @resolve_name ) + raise ArgumentError, 'Missing type of record to lookup' unless( @resolve_type ) + raise ArgumentError, 'Missing expected results' unless( @resolve_expected ) raise ArgumentError, "Uknown record type: #{@resolve_type}" unless( @resolve_type =~ /^(A|NS|MX|AAAA)$/ ) # @@ -193,7 +193,7 @@ module Custodian - register_test_type "dns" + register_test_type 'dns' diff --git a/lib/custodian/protocoltest/dnsbl.rb b/lib/custodian/protocoltest/dnsbl.rb index 50a3b7b..d307470 100644 --- a/lib/custodian/protocoltest/dnsbl.rb +++ b/lib/custodian/protocoltest/dnsbl.rb @@ -31,7 +31,7 @@ module Custodian # # Ensure the host is an IP address. # - raise ArgumentError, "The target must be an IP address" unless( @host =~ /^([0-9]+)\.([0-9]+)\.([0-9]+)\.([0-9]+)$/ ) + raise ArgumentError, 'The target must be an IP address' unless( @host =~ /^([0-9]+)\.([0-9]+)\.([0-9]+)\.([0-9]+)$/ ) # # See which blacklist(s) we're testing against. @@ -39,7 +39,7 @@ module Custodian if line =~ /via\s+([^\s]+)\s+/ @zones = $1.dup else - @zones = "zen.spamhaus.org" + @zones = 'zen.spamhaus.org' end # @@ -73,7 +73,7 @@ module Custodian # The error is empty. @error = nil - @zones.split( "," ).each do |zone| + @zones.split( ',' ).each do |zone| # # Convert the IP to be looked up. @@ -110,7 +110,7 @@ module Custodian end # register ourselves with the class-factory - register_test_type "dnsbl" + register_test_type 'dnsbl' end end end diff --git a/lib/custodian/protocoltest/ftp.rb b/lib/custodian/protocoltest/ftp.rb index bbed744..921de62 100644 --- a/lib/custodian/protocoltest/ftp.rb +++ b/lib/custodian/protocoltest/ftp.rb @@ -92,7 +92,7 @@ module Custodian - register_test_type "ftp" + register_test_type 'ftp' diff --git a/lib/custodian/protocoltest/http.rb b/lib/custodian/protocoltest/http.rb index c43aee3..6ed3c5b 100644 --- a/lib/custodian/protocoltest/http.rb +++ b/lib/custodian/protocoltest/http.rb @@ -79,10 +79,10 @@ module Custodian case line when /\s+must\s(not\s+)?run\s+http(\s+|\.|$)/i then - test_type = "http" + test_type = 'http' when /\s+must\s+(not\s+)?run\s+https(\s+|\.|$)/i then - test_type = "https" + test_type = 'https' else raise ArgumentError, "URL has invalid scheme: #{@line}" end @@ -111,7 +111,7 @@ module Custodian if line =~ /with status ([0-9]+)/ @expected_status = $1.dup else - @expected_status = "200" + @expected_status = '200' end if line =~ /with (IPv[46])/i @@ -158,9 +158,9 @@ module Custodian # def get_type if @url =~ /^https:/ - "https" + 'https' elsif @url =~ /^http:/ - "http" + 'http' else raise ArgumentError, "URL isn't http/https: #{@url}" end @@ -255,7 +255,7 @@ module Custodian end unless @host_override.nil? - c.headers["Host"] = @host_override + c.headers['Host'] = @host_override end c.ssl_verify_host = false @@ -265,7 +265,7 @@ module Custodian # # Set a basic protocol message, for use later. # - protocol_msg = (resolve_mode == :ipv4 ? "IPv4" : "IPv6") + protocol_msg = (resolve_mode == :ipv4 ? 'IPv4' : 'IPv6') begin timeout( period ) do @@ -342,8 +342,8 @@ module Custodian end - register_test_type "http" - register_test_type "https" + register_test_type 'http' + register_test_type 'https' end end diff --git a/lib/custodian/protocoltest/imap.rb b/lib/custodian/protocoltest/imap.rb index 3f338b2..fb80305 100644 --- a/lib/custodian/protocoltest/imap.rb +++ b/lib/custodian/protocoltest/imap.rb @@ -92,7 +92,7 @@ module Custodian - register_test_type "imap" + register_test_type 'imap' diff --git a/lib/custodian/protocoltest/jabber.rb b/lib/custodian/protocoltest/jabber.rb index 1431139..916f15f 100644 --- a/lib/custodian/protocoltest/jabber.rb +++ b/lib/custodian/protocoltest/jabber.rb @@ -90,7 +90,7 @@ module Custodian - register_test_type "jabber" + register_test_type 'jabber' diff --git a/lib/custodian/protocoltest/ldap.rb b/lib/custodian/protocoltest/ldap.rb index aee09e0..a5f6605 100644 --- a/lib/custodian/protocoltest/ldap.rb +++ b/lib/custodian/protocoltest/ldap.rb @@ -92,7 +92,7 @@ module Custodian begin require 'ldap' rescue LoadError - @error = "LDAP library not available - test disabled" + @error = 'LDAP library not available - test disabled' return false end @@ -147,7 +147,7 @@ module Custodian - register_test_type "ldap" + register_test_type 'ldap' end diff --git a/lib/custodian/protocoltest/mx.rb b/lib/custodian/protocoltest/mx.rb index 0df1b5d..61e838c 100644 --- a/lib/custodian/protocoltest/mx.rb +++ b/lib/custodian/protocoltest/mx.rb @@ -103,7 +103,7 @@ module Custodian # failed = 0 passed = 0 - error = "" + error = '' mx.each do |backend| @@ -116,7 +116,7 @@ module Custodian # trim to a sane length & strip newlines. if ! read.nil? read = read[0,255] - read.gsub!(/[\n\r]/, "") + read.gsub!(/[\n\r]/, '') end if read =~ /^220/ @@ -161,7 +161,7 @@ module Custodian - register_test_type "mx" + register_test_type 'mx' diff --git a/lib/custodian/protocoltest/mysql.rb b/lib/custodian/protocoltest/mysql.rb index 135dc38..7abc28f 100644 --- a/lib/custodian/protocoltest/mysql.rb +++ b/lib/custodian/protocoltest/mysql.rb @@ -92,7 +92,7 @@ module Custodian - register_test_type "mysql" + register_test_type 'mysql' diff --git a/lib/custodian/protocoltest/named.rb b/lib/custodian/protocoltest/named.rb index c4e5e8a..4c1b134 100644 --- a/lib/custodian/protocoltest/named.rb +++ b/lib/custodian/protocoltest/named.rb @@ -90,7 +90,7 @@ module Custodian - register_test_type "named" + register_test_type 'named' diff --git a/lib/custodian/protocoltest/openproxy.rb b/lib/custodian/protocoltest/openproxy.rb index da432f0..04fe833 100644 --- a/lib/custodian/protocoltest/openproxy.rb +++ b/lib/custodian/protocoltest/openproxy.rb @@ -92,20 +92,20 @@ module Custodian c.ssl_verify_host = false c.proxy_url = @host c.proxy_tunnel = true - c.url = "http://google.com/" + c.url = 'http://google.com/' c.ssl_verify_peer = false c.timeout = period c.perform @status = c.response_code @content = c.body_str rescue Curl::Err::SSLCACertificateError => x - @error = "SSL-Validation error" + @error = 'SSL-Validation error' return false rescue Curl::Err::TimeoutError - @error = "Timed out fetching page." + @error = 'Timed out fetching page.' return false rescue Curl::Err::TooManyRedirectsError - @error = "Too many redirections (more than 10)" + @error = 'Too many redirections (more than 10)' return false rescue => x @error = "Exception: #{x}" @@ -113,7 +113,7 @@ module Custodian end end rescue Timeout::Error => e - @error = "Timed out during fetch." + @error = 'Timed out during fetch.' return false end @@ -141,7 +141,7 @@ module Custodian - register_test_type "openproxy" + register_test_type 'openproxy' diff --git a/lib/custodian/protocoltest/ping.rb b/lib/custodian/protocoltest/ping.rb index 5a9bf0a..7d3d802 100644 --- a/lib/custodian/protocoltest/ping.rb +++ b/lib/custodian/protocoltest/ping.rb @@ -70,8 +70,8 @@ module Custodian # Find the binary we're going to invoke. # binary = nil - binary = "./bin/multi-ping" - binary = "/usr/bin/multi-ping" if File.exist?( "/usr/bin/multi-ping" ) + binary = './bin/multi-ping' + binary = '/usr/bin/multi-ping' if File.exist?( '/usr/bin/multi-ping' ) if binary.nil? @error = "Failed to find '/usr/bin/multi-ping'" @@ -200,7 +200,7 @@ module Custodian - register_test_type "ping" + register_test_type 'ping' diff --git a/lib/custodian/protocoltest/pop3.rb b/lib/custodian/protocoltest/pop3.rb index 48b2d37..3be34b3 100644 --- a/lib/custodian/protocoltest/pop3.rb +++ b/lib/custodian/protocoltest/pop3.rb @@ -92,8 +92,8 @@ module Custodian - register_test_type "pop3" - register_test_type "pop" + register_test_type 'pop3' + register_test_type 'pop' diff --git a/lib/custodian/protocoltest/postgresql.rb b/lib/custodian/protocoltest/postgresql.rb index 7ce4d70..2dad5cd 100644 --- a/lib/custodian/protocoltest/postgresql.rb +++ b/lib/custodian/protocoltest/postgresql.rb @@ -92,7 +92,7 @@ module Custodian - register_test_type "postgresql" + register_test_type 'postgresql' end diff --git a/lib/custodian/protocoltest/redis.rb b/lib/custodian/protocoltest/redis.rb index 1d732a0..d1815aa 100644 --- a/lib/custodian/protocoltest/redis.rb +++ b/lib/custodian/protocoltest/redis.rb @@ -92,7 +92,7 @@ module Custodian - register_test_type "redis" + register_test_type 'redis' diff --git a/lib/custodian/protocoltest/rsync.rb b/lib/custodian/protocoltest/rsync.rb index 75c27ae..0f3c288 100644 --- a/lib/custodian/protocoltest/rsync.rb +++ b/lib/custodian/protocoltest/rsync.rb @@ -94,7 +94,7 @@ module Custodian - register_test_type "rsync" + register_test_type 'rsync' diff --git a/lib/custodian/protocoltest/smtp.rb b/lib/custodian/protocoltest/smtp.rb index 1d98dca..6a8cd65 100644 --- a/lib/custodian/protocoltest/smtp.rb +++ b/lib/custodian/protocoltest/smtp.rb @@ -90,7 +90,7 @@ module Custodian - register_test_type "smtp" + register_test_type 'smtp' diff --git a/lib/custodian/protocoltest/smtprelay.rb b/lib/custodian/protocoltest/smtprelay.rb index 397c5cb..ef07f82 100644 --- a/lib/custodian/protocoltest/smtprelay.rb +++ b/lib/custodian/protocoltest/smtprelay.rb @@ -61,10 +61,10 @@ module Custodian # Read the hostname for usage in the SMTP-transaction. # def get_hostname - hostname = "localhost.localdomain" + hostname = 'localhost.localdomain' - if File.exist?( "/etc/hostname" ) - File.readlines("/etc/hostname" ).each do |line| + if File.exist?( '/etc/hostname' ) + File.readlines('/etc/hostname' ).each do |line| hostname = line if !line.nil? hostname.chomp! end @@ -87,10 +87,10 @@ module Custodian begin Net::SMTP.start(@host,@port, get_hostname ) do |smtp| - sent = smtp.send_message message, "noreply@bytemark.co.uk", "noreply@bytemark.co.uk" + sent = smtp.send_message message, 'noreply@bytemark.co.uk', 'noreply@bytemark.co.uk' @status = sent.status.to_s - if @status === "250" + if @status === '250' @error = "NOT OK: message sent on #{@host} with status #{@status}" else @error = "OK: message not sent on #{@host} with status #{@status}" @@ -101,10 +101,10 @@ module Custodian # and whether or not we're inverting the test. (eg, 'must not') # - return @inverted if @status == "250" and @inverted - return !@inverted if @status == "250" and !@inverted - return @inverted if @status != "250" and !@inverted - return !@inverted if @status != "250" and @inverted + return @inverted if @status == '250' and @inverted + return !@inverted if @status == '250' and !@inverted + return @inverted if @status != '250' and !@inverted + return !@inverted if @status != '250' and @inverted end # Net SMTP @@ -128,7 +128,7 @@ module Custodian end # register ourselves with the class-factory - register_test_type "smtprelay" + register_test_type 'smtprelay' end end end diff --git a/lib/custodian/protocoltest/ssh.rb b/lib/custodian/protocoltest/ssh.rb index 9b5a608..3d5e9fb 100644 --- a/lib/custodian/protocoltest/ssh.rb +++ b/lib/custodian/protocoltest/ssh.rb @@ -93,7 +93,7 @@ module Custodian - register_test_type "ssh" + register_test_type 'ssh' diff --git a/lib/custodian/protocoltest/ssl.rb b/lib/custodian/protocoltest/ssl.rb index 86acb34..39a962d 100644 --- a/lib/custodian/protocoltest/ssl.rb +++ b/lib/custodian/protocoltest/ssl.rb @@ -26,7 +26,7 @@ class SSLCheck # Takes one parameter -- the URL. # def initialize(uri) - raise ArgumentError, "URI must be a string" unless uri.is_a?(String) + raise ArgumentError, 'URI must be a string' unless uri.is_a?(String) @uri = URI.parse(uri) @domain = @uri.host @@ -60,7 +60,7 @@ class SSLCheck # Allows the domain to be set manually. # def domain=(d) - raise ArgumentError, "domain must be a String" unless d.is_a?(String) + raise ArgumentError, 'domain must be a String' unless d.is_a?(String) @domain=d end @@ -76,7 +76,7 @@ class SSLCheck # ones from ALL_TESTS are taken. Anything else is ignored. # def tests=(ts) - raise ArgumentError, "tests must be an Array" unless ts.is_a?(Array) + raise ArgumentError, 'tests must be an Array' unless ts.is_a?(Array) @tests = ts.collect{|t| t.to_sym}.select{|t| ALL_TESTS.include?(t)} @tests @@ -94,7 +94,7 @@ class SSLCheck # Probably not much use here. # def key=(k) - raise ArgumentError, "key must be a String" unless k.is_a?(String) + raise ArgumentError, 'key must be a String' unless k.is_a?(String) if k =~ /-----BEGIN/ @key = OpenSSL::PKey::RSA.new(k) else @@ -127,7 +127,7 @@ class SSLCheck elsif b.is_a?(OpenSSL::X509::Certificate) self.certificate_store.add_cert(b) else - raise ArgumentError, "bundle must be a String, an Array, or an OpenSSL::X509::Certificate" + raise ArgumentError, 'bundle must be a String, an Array, or an OpenSSL::X509::Certificate' end b end @@ -140,7 +140,7 @@ class SSLCheck @certificate_store = OpenSSL::X509::Store.new @certificate_store.set_default_paths - @certificate_store.add_path("/etc/ssl/certs") + @certificate_store.add_path('/etc/ssl/certs') @certificate_store end @@ -412,7 +412,7 @@ module Custodian # Double-check we've got an SSL host # if ! @host =~ /^https:\/\// - puts( "Not an SSL URL" ) + puts( 'Not an SSL URL' ) return true end @@ -442,7 +442,7 @@ module Custodian @error end - register_test_type "https" + register_test_type 'https' end end diff --git a/lib/custodian/protocoltest/tcp.rb b/lib/custodian/protocoltest/tcp.rb index 60c7be7..ca622bb 100644 --- a/lib/custodian/protocoltest/tcp.rb +++ b/lib/custodian/protocoltest/tcp.rb @@ -102,7 +102,7 @@ module Custodian @error = nil if @port.nil? - raise ArgumentError, "Missing port to test against" + raise ArgumentError, 'Missing port to test against' end end @@ -280,7 +280,7 @@ module Custodian # trim to a sane length & strip newlines. if ! read.nil? read = read[0,255] - read.gsub!(/[\n\r]/, "") + read.gsub!(/[\n\r]/, '') end socket.close @@ -317,7 +317,7 @@ module Custodian @error = "TIMEOUT: #{e}" return false end - @error = "Misc failure" + @error = 'Misc failure' false end @@ -334,7 +334,7 @@ module Custodian - register_test_type "tcp" + register_test_type 'tcp' diff --git a/lib/custodian/protocoltest/telnet.rb b/lib/custodian/protocoltest/telnet.rb index b5e0cde..f485191 100644 --- a/lib/custodian/protocoltest/telnet.rb +++ b/lib/custodian/protocoltest/telnet.rb @@ -90,7 +90,7 @@ module Custodian - register_test_type "telnet" + register_test_type 'telnet' diff --git a/lib/custodian/queue.rb b/lib/custodian/queue.rb index b635c1f..12db68d 100644 --- a/lib/custodian/queue.rb +++ b/lib/custodian/queue.rb @@ -6,7 +6,7 @@ begin require library rescue LoadError - ENV["DEBUG"] && puts( "Failed to load the library: #{library}" ) + ENV['DEBUG'] && puts( "Failed to load the library: #{library}" ) end end @@ -25,9 +25,9 @@ module Custodian # def self.create type case type - when "redis" + when 'redis' RedisQueueType.new - when "beanstalk" + when 'beanstalk' BeanstalkQueueType.new else raise "Bad queue-type: #{type}" @@ -39,7 +39,7 @@ module Custodian # Retrieve a job from the queue. # def fetch(_timeout) - raise "Subclasses must implement this method!" + raise 'Subclasses must implement this method!' end @@ -47,7 +47,7 @@ module Custodian # Add a new job to the queue. # def add(_job_string) - raise "Subclasses must implement this method!" + raise 'Subclasses must implement this method!' end @@ -55,7 +55,7 @@ module Custodian # Get the size of the queue # def size? - raise "Subclasses must implement this method!" + raise 'Subclasses must implement this method!' end @@ -63,7 +63,7 @@ module Custodian # Empty the queue # def flush! - raise "Subclasses must implement this method!" + raise 'Subclasses must implement this method!' end end @@ -80,7 +80,7 @@ end # Connect to the server on localhost, unless QUEUE_ADDRESS is set. # def initialize - host = ENV["QUEUE_ADDRESS"] || "127.0.0.1" + host = ENV['QUEUE_ADDRESS'] || '127.0.0.1' @redis = Redis.new( :host => host ) end @@ -96,7 +96,7 @@ end while( true ) - foo, job = @redis.blpop( "queue", :timeout => timeout ) + foo, job = @redis.blpop( 'queue', :timeout => timeout ) if job return job @@ -112,7 +112,7 @@ end # Add a new job to the queue. # def add(job_string) - @redis.rpush( "queue", job_string ) + @redis.rpush( 'queue', job_string ) end @@ -120,7 +120,7 @@ end # How many jobs in the queue? # def size? - @redis.llen( "queue" ) + @redis.llen( 'queue' ) end @@ -128,7 +128,7 @@ end # Empty the queue, discarding all pending jobs. # def flush! - @redis.del( "queue" ) + @redis.del( 'queue' ) end end @@ -144,7 +144,7 @@ end # Connect to the server on localhost, unless QUEUE_ADDRESS is set. # def initialize - host = ENV["QUEUE_ADDRESS"] || "127.0.0.1" + host = ENV['QUEUE_ADDRESS'] || '127.0.0.1' @queue = Beanstalk::Pool.new( ["#{host}:11300" ] ) end @@ -162,7 +162,7 @@ end j.delete return b else - raise "ERRROR" + raise 'ERRROR' end rescue Beanstalk::TimedOut => ex return nil diff --git a/lib/custodian/settings.rb b/lib/custodian/settings.rb index 1739495..c4b11cb 100644 --- a/lib/custodian/settings.rb +++ b/lib/custodian/settings.rb @@ -35,7 +35,7 @@ module Custodian # # Load the configuration file; called only once. # - def _load( file = "/etc/custodian/custodian.cfg" ) + def _load( file = '/etc/custodian/custodian.cfg' ) @parsed = true @settings = {} @@ -48,7 +48,7 @@ module Custodian # # Load and "parse" the key=value content. # - File.open( file, "r" ).each_line do |line| + File.open( file, 'r' ).each_line do |line| next if line.nil? || line =~ /^#/ if line =~ /^(.*)=(.*)$/ key = $1.dup @@ -148,7 +148,7 @@ module Custodian def queue_type _load unless( _loaded? ) - @settings['queue_type'] || "redis" + @settings['queue_type'] || 'redis' end @@ -159,7 +159,7 @@ module Custodian def log_file _load unless( _loaded? ) - @settings['log_file'] || "custodian-dequeue.log" + @settings['log_file'] || 'custodian-dequeue.log' end @@ -169,7 +169,7 @@ module Custodian def alerter _load unless( _loaded? ) - @settings['alerter'] || "file" + @settings['alerter'] || 'file' end @@ -179,7 +179,7 @@ module Custodian def alert_source _load unless( _loaded? ) - @settings['alert_source'] || "Custodian" + @settings['alert_source'] || 'Custodian' end @@ -208,14 +208,14 @@ module Custodian # otherwise per-test defaults. case alert - when "smtp" - "root" - when "mauve" - "alert.bytemark.co.uk" - when "file" - "alerts.log" - when "redis" - "127.0.0.1:6379" + when 'smtp' + 'root' + when 'mauve' + 'alert.bytemark.co.uk' + when 'file' + 'alerts.log' + when 'redis' + '127.0.0.1:6379' else nil end diff --git a/lib/custodian/testfactory.rb b/lib/custodian/testfactory.rb index 748e114..7383ca2 100644 --- a/lib/custodian/testfactory.rb +++ b/lib/custodian/testfactory.rb @@ -31,8 +31,8 @@ module Custodian def self.create( line ) - raise ArgumentError, "The type of test to create cannot be nil" if line.nil? - raise ArgumentError, "The type of test to create must be a string" unless line.kind_of? String + raise ArgumentError, 'The type of test to create cannot be nil' if line.nil? + raise ArgumentError, 'The type of test to create must be a string' unless line.kind_of? String # # The array we return. @@ -46,7 +46,7 @@ module Custodian if line =~ /must\s+(not\s+)?run\s+(\S+)(\s+|\.|$)/ test_type = $2.dup - test_type.chomp!( "." ) + test_type.chomp!( '.' ) if @@subclasses[test_type].nil? raise ArgumentError, "There is no handler registered for the '#{test_type}' test-type" diff --git a/lib/custodian/util/ping.rb b/lib/custodian/util/ping.rb index 771cfd6..cf79754 100644 --- a/lib/custodian/util/ping.rb +++ b/lib/custodian/util/ping.rb @@ -26,8 +26,8 @@ module Custodian # def initialize( hostname ) - raise ArgumentError, "Hostname must not be nil" if hostname.nil? - raise ArgumentError, "Hostname must be a String" unless hostname.kind_of?( String ) + raise ArgumentError, 'Hostname must not be nil' if hostname.nil? + raise ArgumentError, 'Hostname must be a String' unless hostname.kind_of?( String ) @hostname = hostname @resolved = Custodian::Util::DNS.hostname_to_ip( hostname ) @@ -83,7 +83,7 @@ module Custodian return true end else - puts "ERROR: Resolved to neither an IPv6 or IPv4 address." + puts 'ERROR: Resolved to neither an IPv6 or IPv4 address.' end false end diff --git a/lib/custodian/util/timespan.rb b/lib/custodian/util/timespan.rb index 36e8d9e..a033675 100644 --- a/lib/custodian/util/timespan.rb +++ b/lib/custodian/util/timespan.rb @@ -33,7 +33,7 @@ module Custodian # # Ensure within a valid range # - raise ArgumentError, "Integer required for time" unless( desc.kind_of? Integer ) + raise ArgumentError, 'Integer required for time' unless( desc.kind_of? Integer ) raise ArgumentError, "Invalid time: #{desc}" unless( ( desc >= 0 ) && ( desc <= 23 ) ) diff --git a/lib/custodian/worker.rb b/lib/custodian/worker.rb index 9f66e4a..e5df5e5 100644 --- a/lib/custodian/worker.rb +++ b/lib/custodian/worker.rb @@ -93,7 +93,7 @@ module Custodian # Show a message on STDOUT if "--verbose" was specified. # def log_message( msg ) - puts msg if ENV["VERBOSE" ] + puts msg if ENV['VERBOSE' ] end @@ -104,7 +104,7 @@ module Custodian # def run! while( true ) - log_message( "Waiting for job.." ) + log_message( 'Waiting for job..' ) process_single_job end end @@ -125,8 +125,8 @@ module Custodian # # Ensure that the job is sane. # - raise ArgumentError, "Job was empty" if job.nil? - raise ArgumentError, "Job was not a string" unless job.kind_of?(String) + raise ArgumentError, 'Job was empty' if job.nil? + raise ArgumentError, 'Job was not a string' unless job.kind_of?(String) # # Create test-objects from our class-factory, and process them. @@ -173,7 +173,7 @@ module Custodian result = ! result if test.inverted if result - log_message( "Test succeeed - clearing alert" ) + log_message( 'Test succeeed - clearing alert' ) do_clear( test ) end @@ -240,7 +240,7 @@ module Custodian # Raise an alert, with each registered alerter. # def do_raise( test ) - @alerter.split( "," ).each do |alerter| + @alerter.split( ',' ).each do |alerter| log_message( "Creating alerter: #{alerter}" ) alert = Custodian::AlertFactory.create( alerter, test ) @@ -261,7 +261,7 @@ module Custodian # Clear an alert, with each registered alerter. # def do_clear( test ) - @alerter.split( "," ).each do |alerter| + @alerter.split( ',' ).each do |alerter| log_message( "Creating alerter: #{alerter}" ) alert = Custodian::AlertFactory.create( alerter, test ) @@ -280,7 +280,7 @@ module Custodian # Log the test duration with each registered alerter. # def do_duration( test, duration ) - @alerter.split( "," ).each do |alerter| + @alerter.split( ',' ).each do |alerter| log_message( "Creating alerter: #{alerter}" ) alert = Custodian::AlertFactory.create( alerter, test ) @@ -291,7 +291,7 @@ module Custodian # give the alerter a reference to the settings object. alert.set_settings( @settings ) - alert.duration( duration ) if alert.respond_to? "duration" + alert.duration( duration ) if alert.respond_to? 'duration' end end diff --git a/t/test-custodian-alertfactory.rb b/t/test-custodian-alertfactory.rb index 42873ec..eea35ac 100755 --- a/t/test-custodian-alertfactory.rb +++ b/t/test-custodian-alertfactory.rb @@ -31,8 +31,8 @@ class TestAlertFactory < Test::Unit::TestCase # Ensure we can create each of the two alert types we care about # methods = [] - methods.push( "file" ) - methods.push( "smtp" ) + methods.push( 'file' ) + methods.push( 'smtp' ) # # Mauve + Redis are optional @@ -54,8 +54,8 @@ class TestAlertFactory < Test::Unit::TestCase mauve = false end - methods.push( "redis" ) if redis - methods.push( "mauve" ) if mauve + methods.push( 'redis' ) if redis + methods.push( 'mauve' ) if mauve methods.each do |name| @@ -75,8 +75,8 @@ class TestAlertFactory < Test::Unit::TestCase # Ensure that the object implements the raise() + clear() # methods we mandate. # - assert( obj.respond_to? "raise" ) - assert( obj.respond_to? "clear" ) + assert( obj.respond_to? 'raise' ) + assert( obj.respond_to? 'clear' ) end @@ -84,7 +84,7 @@ class TestAlertFactory < Test::Unit::TestCase # Creating an alert we don't know about is an error # assert_raise ArgumentError do - obj = Custodian::AlertFactory.create( "not found", nil ) + obj = Custodian::AlertFactory.create( 'not found', nil ) end # diff --git a/t/test-custodian-parser.rb b/t/test-custodian-parser.rb index 49416b3..bbdb8ad 100755 --- a/t/test-custodian-parser.rb +++ b/t/test-custodian-parser.rb @@ -51,7 +51,7 @@ class TestCustodianParser < Test::Unit::TestCase def test_period parser = Custodian::Parser.new - result = parser.parse_line( "example.vm.bytemark.co.uk must run ping except between 00-23" ) + result = parser.parse_line( 'example.vm.bytemark.co.uk must run ping except between 00-23' ) assert( result.nil? ) end @@ -72,7 +72,7 @@ class TestCustodianParser < Test::Unit::TestCase parser = Custodian::Parser.new # 1.a. Comment lines return nil. - result = parser.parse_line( "# this is a comment" ) + result = parser.parse_line( '# this is a comment' ) assert( result.nil? ) # 1.b. Non-strings are an error @@ -94,7 +94,7 @@ class TestCustodianParser < Test::Unit::TestCase parser = Custodian::Parser.new # 2.a. Comment lines return nil. tmp = [] - tmp.push( "# This is a comment.." ) + tmp.push( '# This is a comment..' ) assert( parser.parse_lines( tmp ).nil? ) # 2.b. Adding a test will return an array of test-objects. @@ -172,8 +172,8 @@ EOF # Input text # text = [] - text.push( "FOO is kvm1.vm.bytemark.co.uk." ) - text.push( "FOO2 is kvm2.vm.bytemark.co.uk." ) + text.push( 'FOO is kvm1.vm.bytemark.co.uk.' ) + text.push( 'FOO2 is kvm2.vm.bytemark.co.uk.' ) # # Test the parser with this text @@ -203,8 +203,8 @@ EOF # Input text to parse. # text = [] - text.push( "FOO is kvm1.vm.bytemark.co.uk." ) - text.push( "FOO is kvm2.vm.bytemark.co.uk." ) + text.push( 'FOO is kvm1.vm.bytemark.co.uk.' ) + text.push( 'FOO is kvm2.vm.bytemark.co.uk.' ) # # Test the parser with this text @@ -241,7 +241,7 @@ EOF # Expand a line - which should result in no change # as the line does not involve a known-macro # - in_txt = "example.bytemark.co.uk must run smtp." + in_txt = 'example.bytemark.co.uk must run smtp.' out_txt = parser.expand_macro( in_txt ) # @@ -255,14 +255,14 @@ EOF # # Now define a macro # - parser.parse_line( "TARGET is example1.bytemark.co.uk and example2.bytemark.co.uk." ) + parser.parse_line( 'TARGET is example1.bytemark.co.uk and example2.bytemark.co.uk.' ) macros = parser.macros assert( !macros.empty? ) # # Now we have a two-host macro, repeat the expansion # - ret = parser.expand_macro( "TARGET must run smtp on 25." ) + ret = parser.expand_macro( 'TARGET must run smtp on 25.' ) # # The result should be an array @@ -284,12 +284,12 @@ EOF # test data # data = { - "http://example must run http." => true, - "http://example must run http with status 200." => true, + 'http://example must run http.' => true, + 'http://example must run http with status 200.' => true, "http://example must run http with content 'bar'." => true, - "http://example must run http following redirects." => true, - "http://example must run http not following redirects." => false, - "http://example must run http not following redirect." => false, + 'http://example must run http following redirects.' => true, + 'http://example must run http not following redirects.' => false, + 'http://example must run http not following redirect.' => false, } data.each do |str,follow| @@ -325,10 +325,10 @@ EOF # test data # data = { - "http://example must run http." => true, - "http://example must run http with status 200." => true, + 'http://example must run http.' => true, + 'http://example must run http with status 200.' => true, "http://example must run http with content 'bar'." => true, - "http://example must run http without cache busting." => false, + 'http://example must run http without cache busting.' => false, } data.each do |str,cb| @@ -365,13 +365,13 @@ EOF # test data # data = { - "foo must run rsync." => nil, - "foo must run redis." => nil, - "foo must not run ping." => nil, - "foo must not run ssh otherwise 'fail'" => "fail", - "foo must not run ssh otherwise 'fail'." => "fail", - "foo must run redis otherwise 'memorystorage service is dead'" => "memorystorage service is dead", - "foo must run ping otherwise 'don't you love me?'" => "don" + 'foo must run rsync.' => nil, + 'foo must run redis.' => nil, + 'foo must not run ping.' => nil, + "foo must not run ssh otherwise 'fail'" => 'fail', + "foo must not run ssh otherwise 'fail'." => 'fail', + "foo must run redis otherwise 'memorystorage service is dead'" => 'memorystorage service is dead', + "foo must run ping otherwise 'don't you love me?'" => 'don' } # diff --git a/t/test-custodian-queue.rb b/t/test-custodian-queue.rb index 0170ef1..bec83e2 100755 --- a/t/test-custodian-queue.rb +++ b/t/test-custodian-queue.rb @@ -40,7 +40,7 @@ class TestCustodianQueue < Test::Unit::TestCase # creation will fail assert_raise RuntimeError do - t = Custodian::QueueType.create( "foo" ) + t = Custodian::QueueType.create( 'foo' ) end end @@ -50,7 +50,7 @@ class TestCustodianQueue < Test::Unit::TestCase def test_redis q = nil assert_nothing_raised do - q = Custodian::QueueType.create( "redis" ) + q = Custodian::QueueType.create( 'redis' ) end # @@ -68,7 +68,7 @@ class TestCustodianQueue < Test::Unit::TestCase def test_beanstalkd q = nil assert_nothing_raised do - q = Custodian::QueueType.create( "redis" ) + q = Custodian::QueueType.create( 'redis' ) end # diff --git a/t/test-custodian-settings.rb b/t/test-custodian-settings.rb index 0347e7c..1923b0f 100755 --- a/t/test-custodian-settings.rb +++ b/t/test-custodian-settings.rb @@ -50,13 +50,13 @@ class TestConfigurationSingleton < Test::Unit::TestCase assert( a.class == Fixnum ) # store a number - settings._store( "retry_delay", 5 ) + settings._store( 'retry_delay', 5 ) a = settings.retry_delay assert( a.class == Fixnum ) assert( a == 5 ) # store a string - settings._store( "retry_delay", "35" ) + settings._store( 'retry_delay', '35' ) a = settings.retry_delay assert( a.class == Fixnum ) assert( a == 35 ) @@ -68,13 +68,13 @@ class TestConfigurationSingleton < Test::Unit::TestCase assert( a.class == Fixnum ) # store a number - settings._store( "timeout", 5 ) + settings._store( 'timeout', 5 ) a = settings.timeout assert( a.class == Fixnum ) assert( a == 5 ) # store a string - settings._store( "timeout", "35" ) + settings._store( 'timeout', '35' ) a = settings.timeout assert( a.class == Fixnum ) assert( a == 35 ) diff --git a/t/test-custodian-testfactory.rb b/t/test-custodian-testfactory.rb index b067ab8..af3e3b3 100755 --- a/t/test-custodian-testfactory.rb +++ b/t/test-custodian-testfactory.rb @@ -45,25 +45,25 @@ class TestTestFactory < Test::Unit::TestCase def test_ftp_uri assert_nothing_raised do - assert( Custodian::TestFactory.create( "ftp.example.com must run ftp." ) ) - assert( Custodian::TestFactory.create( "ftp://ftp.example.com/ must run ftp." ) ) - assert( Custodian::TestFactory.create( "ftp://ftp.example.com/ must run ftp on 21." ) ) + assert( Custodian::TestFactory.create( 'ftp.example.com must run ftp.' ) ) + assert( Custodian::TestFactory.create( 'ftp://ftp.example.com/ must run ftp.' ) ) + assert( Custodian::TestFactory.create( 'ftp://ftp.example.com/ must run ftp on 21.' ) ) assert( Custodian::TestFactory.create( "ftp://ftp.example.com/ must run ftp on 21 otherwise 'xxx'." ) ) end - assert( Custodian::TestFactory.create( "ftp.example.com must run ftp." )[0].target == "ftp.example.com" ) - assert( Custodian::TestFactory.create( "ftp://ftp.example.com/ must run ftp." )[0].target == "ftp.example.com" ) + assert( Custodian::TestFactory.create( 'ftp.example.com must run ftp.' )[0].target == 'ftp.example.com' ) + assert( Custodian::TestFactory.create( 'ftp://ftp.example.com/ must run ftp.' )[0].target == 'ftp.example.com' ) # # Test the port detection # data = { - "foo must run ftp." => "21", - "ftp://ftp.example.com/ must run ftp." => "21", - "foo must run ftp on 1 otherwise 'x'." => "1", - "foo must run ftp on 33 otherwise" => "33", + 'foo must run ftp.' => '21', + 'ftp://ftp.example.com/ must run ftp.' => '21', + "foo must run ftp on 1 otherwise 'x'." => '1', + 'foo must run ftp on 33 otherwise' => '33', } # @@ -76,7 +76,7 @@ class TestTestFactory < Test::Unit::TestCase assert( obj.kind_of? Array ) assert( ! obj.empty? ) - assert_equal( obj[0].get_type, "ftp" ) + assert_equal( obj[0].get_type, 'ftp' ) assert_equal( obj[0].port.to_s, prt ) end @@ -91,11 +91,11 @@ class TestTestFactory < Test::Unit::TestCase # def test_port_detection data = { - "foo must run ftp." => "21", - "foo must run ssh." => "22", - "foo must run mysql otherwise 'alert'" => "3306", - "foo must run redis otherwise 'alert'" => "6379", - "foo must run mysql on 33 otherwise 'alert'" => "33", + 'foo must run ftp.' => '21', + 'foo must run ssh.' => '22', + "foo must run mysql otherwise 'alert'" => '3306', + "foo must run redis otherwise 'alert'" => '6379', + "foo must run mysql on 33 otherwise 'alert'" => '33', } # @@ -124,26 +124,26 @@ class TestTestFactory < Test::Unit::TestCase assert_nothing_raised do - assert( Custodian::TestFactory.create( "example.com must run rsync." ) ) - assert( Custodian::TestFactory.create( "ftp://example.com/ must run rsync." ) ) - assert( Custodian::TestFactory.create( "ftp://example.com/ must run rsync on 333." ) ) + assert( Custodian::TestFactory.create( 'example.com must run rsync.' ) ) + assert( Custodian::TestFactory.create( 'ftp://example.com/ must run rsync.' ) ) + assert( Custodian::TestFactory.create( 'ftp://example.com/ must run rsync on 333.' ) ) assert( Custodian::TestFactory.create( "ftp://example.com/ must run rsync on 3311 otherwise 'xxx'." ) ) end - assert( Custodian::TestFactory.create( "rsync.example.com must run rsync." )[0].target == - "rsync.example.com" ) - assert( Custodian::TestFactory.create( "rsync://rsync.example.com/ must run rsync." )[0].target == - "rsync.example.com" ) + assert( Custodian::TestFactory.create( 'rsync.example.com must run rsync.' )[0].target == + 'rsync.example.com' ) + assert( Custodian::TestFactory.create( 'rsync://rsync.example.com/ must run rsync.' )[0].target == + 'rsync.example.com' ) # # Test the ports # data = { - "foo must run rsync." => "873", - "rsync://foo/ must run rsync." => "873", - "foo must run rsync on 1 otherwise 'x'." => "1", - "foo must run rsync on 33 otherwise" => "33", + 'foo must run rsync.' => '873', + 'rsync://foo/ must run rsync.' => '873', + "foo must run rsync on 1 otherwise 'x'." => '1', + 'foo must run rsync on 33 otherwise' => '33', } # @@ -191,7 +191,7 @@ class TestTestFactory < Test::Unit::TestCase # Missing expected results # assert_raise ArgumentError do - assert( Custodian::TestFactory.create( "a.ns.bytemark.co.uk must run dns for www.bytemark.co.uk resolving NS " ) ) + assert( Custodian::TestFactory.create( 'a.ns.bytemark.co.uk must run dns for www.bytemark.co.uk resolving NS ' ) ) end end @@ -205,9 +205,9 @@ class TestTestFactory < Test::Unit::TestCase assert_nothing_raised do - assert( Custodian::TestFactory.create( "example.com must not run rsync." ) ) - assert( Custodian::TestFactory.create( "ftp://example.com/ must not run rsync." ) ) - assert( Custodian::TestFactory.create( "ftp://example.com/ must not run rsync on 333." ) ) + assert( Custodian::TestFactory.create( 'example.com must not run rsync.' ) ) + assert( Custodian::TestFactory.create( 'ftp://example.com/ must not run rsync.' ) ) + assert( Custodian::TestFactory.create( 'ftp://example.com/ must not run rsync on 333.' ) ) end @@ -215,11 +215,11 @@ class TestTestFactory < Test::Unit::TestCase # Test some inversions # data = { - "foo must run rsync." => false, - "rsync://foo/ must run rsync." => false, - "foo must run ping otherwise" => false, - "foo must not run ping otherwise" => true, - "foo must not run ssh otherwise" => true, + 'foo must run rsync.' => false, + 'rsync://foo/ must run rsync.' => false, + 'foo must run ping otherwise' => false, + 'foo must not run ping otherwise' => true, + 'foo must not run ssh otherwise' => true, } # @@ -292,13 +292,13 @@ class TestTestFactory < Test::Unit::TestCase a = [] - a.push( "test.host.example.com must run ftp.") - a.push( "ftp://test.host.example.com/ must run ftp.") - a.push( "ftp://test.host.example.com/foo must run ftp.") - a.push( "test.host.example.com must run ping.") + a.push( 'test.host.example.com must run ftp.') + a.push( 'ftp://test.host.example.com/ must run ftp.') + a.push( 'ftp://test.host.example.com/foo must run ftp.') + a.push( 'test.host.example.com must run ping.') a.push( "test.host.example.com must run dns for bytemark.co.uk resolving NS as '80.68.80.26;85.17.170.78;80.68.80.27'.") - a.push( "rsync://test.host.example.com must run rsync.") - a.push( "rsync://test.host.example.com must run rsync.") + a.push( 'rsync://test.host.example.com must run rsync.') + a.push( 'rsync://test.host.example.com must run rsync.') a.each do |entry| assert_nothing_raised do @@ -306,7 +306,7 @@ class TestTestFactory < Test::Unit::TestCase assert(obj) assert( obj.kind_of? Array ) assert( ! obj.empty? ) - assert_equal( "test.host.example.com", obj[0].target ) + assert_equal( 'test.host.example.com', obj[0].target ) end end end diff --git a/t/test-custodian-util-bytemark.rb b/t/test-custodian-util-bytemark.rb index 9296dcb..30bb480 100755 --- a/t/test-custodian-util-bytemark.rb +++ b/t/test-custodian-util-bytemark.rb @@ -38,15 +38,15 @@ class TestBytemarkUtil < Test::Unit::TestCase # # Hosts inside the Bytemark network # - "80.68.85.48" => true, - "2001:41c8:125:46::10" => true, + '80.68.85.48' => true, + '2001:41c8:125:46::10' => true, # # Hosts outside the Bytemark network # - "127.0.0.1" => false, - "192.168.1.1" => false, - "2a00:1450:400c:c00::93" => false, + '127.0.0.1' => false, + '192.168.1.1' => false, + '2a00:1450:400c:c00::93' => false, } diff --git a/t/test-custodian-util-dns.rb b/t/test-custodian-util-dns.rb index f226efe..2582d94 100755 --- a/t/test-custodian-util-dns.rb +++ b/t/test-custodian-util-dns.rb @@ -30,26 +30,26 @@ class TestDNSUtil < Test::Unit::TestCase # def test_lookup - details = Custodian::Util::DNS.hostname_to_ip( "bytemark.co.uk" ) - assert( ( details == "2001:41c9:0:1019:81::80" ) || - ( details == "80.68.81.80" ) ) + details = Custodian::Util::DNS.hostname_to_ip( 'bytemark.co.uk' ) + assert( ( details == '2001:41c9:0:1019:81::80' ) || + ( details == '80.68.81.80' ) ) - details = Custodian::Util::DNS.hostname_to_ip( "www.bytemark.co.uk" ) - assert( ( details == "2001:41c9:0:1019:81::80" ) || - ( details == "80.68.81.80" ) ) + details = Custodian::Util::DNS.hostname_to_ip( 'www.bytemark.co.uk' ) + assert( ( details == '2001:41c9:0:1019:81::80' ) || + ( details == '80.68.81.80' ) ) - details = Custodian::Util::DNS.hostname_to_ip( "ipv4.steve.org.uk" ) - assert( details == "80.68.84.103" ) + details = Custodian::Util::DNS.hostname_to_ip( 'ipv4.steve.org.uk' ) + assert( details == '80.68.84.103' ) - details = Custodian::Util::DNS.hostname_to_ip( "ipv6.steve.org.uk" ) - assert( details == "2001:41c8:10b:103::10" ) + details = Custodian::Util::DNS.hostname_to_ip( 'ipv6.steve.org.uk' ) + assert( details == '2001:41c8:10b:103::10' ) # # Failure case # - details = Custodian::Util::DNS.hostname_to_ip( "this.doesnot.exist" ) + details = Custodian::Util::DNS.hostname_to_ip( 'this.doesnot.exist' ) assert( details.nil? ) end @@ -63,19 +63,19 @@ class TestDNSUtil < Test::Unit::TestCase # # IPv6 lookup # - details = Custodian::Util::DNS.ip_to_hostname( "2001:41c9:0:1019:81::80" ) + details = Custodian::Util::DNS.ip_to_hostname( '2001:41c9:0:1019:81::80' ) assert( details =~ /bytemark(-?hosting)?\.(com|eu|co\.uk)$/i ) # # IPv4 lookup # - details = Custodian::Util::DNS.ip_to_hostname( "80.68.81.80" ) + details = Custodian::Util::DNS.ip_to_hostname( '80.68.81.80' ) assert( details =~ /bytemark(-?hosting)?\.(com|eu|co\.uk)$/i ) # # Bogus lookup - should return nil. # - details = Custodian::Util::DNS.ip_to_hostname( "800.683.853.348" ) + details = Custodian::Util::DNS.ip_to_hostname( '800.683.853.348' ) assert( details.nil? ) end diff --git a/t/test-custodian-util-ping.rb b/t/test-custodian-util-ping.rb index bf8ac43..e01ef8d 100755 --- a/t/test-custodian-util-ping.rb +++ b/t/test-custodian-util-ping.rb @@ -34,7 +34,7 @@ class TestPingUtil < Test::Unit::TestCase # Normal construction works. # assert_nothing_raised do - Custodian::Util::Ping.new("foo") + Custodian::Util::Ping.new('foo') end # @@ -64,7 +64,7 @@ class TestPingUtil < Test::Unit::TestCase # def test_lookup_ipv4 - helper = Custodian::Util::Ping.new( "ipv4.steve.org.uk" ) + helper = Custodian::Util::Ping.new( 'ipv4.steve.org.uk' ) assert( helper.is_ipv4? ) assert( ! helper.is_ipv6? ) @@ -76,7 +76,7 @@ class TestPingUtil < Test::Unit::TestCase # def test_lookup_ipv6 - helper = Custodian::Util::Ping.new( "ipv6.steve.org.uk" ) + helper = Custodian::Util::Ping.new( 'ipv6.steve.org.uk' ) assert( helper.is_ipv6? ) assert( ! helper.is_ipv4? ) end diff --git a/t/test-custodian-util-timespan.rb b/t/test-custodian-util-timespan.rb index 29e9b83..210a532 100755 --- a/t/test-custodian-util-timespan.rb +++ b/t/test-custodian-util-timespan.rb @@ -114,7 +114,7 @@ class TestTimeSpanUtil < Test::Unit::TestCase # def test_simple_cases # 8am-5pm - assert(Custodian::Util::TimeSpan.inside?( "8am", "5am", 12 )) + assert(Custodian::Util::TimeSpan.inside?( '8am', '5am', 12 )) assert(Custodian::Util::TimeSpan.inside?( 8, 17, 12 )) end @@ -125,25 +125,25 @@ class TestTimeSpanUtil < Test::Unit::TestCase def test_midnight_cases # 9pm-2am - assert(Custodian::Util::TimeSpan.inside?( "9pm", "2am", 22 )) - assert(Custodian::Util::TimeSpan.inside?( "9pm", "2am", "10pm" )) + assert(Custodian::Util::TimeSpan.inside?( '9pm', '2am', 22 )) + assert(Custodian::Util::TimeSpan.inside?( '9pm', '2am', '10pm' )) assert(Custodian::Util::TimeSpan.inside?( 21, 2, 22 )) - assert(Custodian::Util::TimeSpan.inside?( 21, 2, "10pm" )) + assert(Custodian::Util::TimeSpan.inside?( 21, 2, '10pm' )) # 10pm-3am - assert(Custodian::Util::TimeSpan.inside?( "10pm", "3am", 22 )) + assert(Custodian::Util::TimeSpan.inside?( '10pm', '3am', 22 )) assert(Custodian::Util::TimeSpan.inside?( 22, 3, 22 )) assert(Custodian::Util::TimeSpan.inside?( 22, 3, 22 )) - assert(Custodian::Util::TimeSpan.inside?( 22, 3, "10pm" )) + assert(Custodian::Util::TimeSpan.inside?( 22, 3, '10pm' )) # 11pm-5am - assert(Custodian::Util::TimeSpan.inside?( "11pm", "5am", 23 )) + assert(Custodian::Util::TimeSpan.inside?( '11pm', '5am', 23 )) assert(Custodian::Util::TimeSpan.inside?( 23, 5, 23 )) - assert(Custodian::Util::TimeSpan.inside?( "11pm", "5am", "11pm" )) + assert(Custodian::Util::TimeSpan.inside?( '11pm', '5am', '11pm' )) # midnight-3am - assert( Custodian::Util::TimeSpan.inside?( "0", "3am", 1 )) - assert( Custodian::Util::TimeSpan.inside?( "0", "3am", "1am" )) + assert( Custodian::Util::TimeSpan.inside?( '0', '3am', 1 )) + assert( Custodian::Util::TimeSpan.inside?( '0', '3am', '1am' )) end @@ -154,18 +154,18 @@ class TestTimeSpanUtil < Test::Unit::TestCase # def test_inclusive - open = "4pm" - close = "6pm" + open = '4pm' + close = '6pm' # The hours + the middle should be inside assert( Custodian::Util::TimeSpan.inside?( open, close, 16 ) ) - assert( Custodian::Util::TimeSpan.inside?( open, close, "4pm" ) ) + assert( Custodian::Util::TimeSpan.inside?( open, close, '4pm' ) ) assert( Custodian::Util::TimeSpan.inside?( open, close, 17 ) ) - assert( Custodian::Util::TimeSpan.inside?( open, close, "5pm" ) ) + assert( Custodian::Util::TimeSpan.inside?( open, close, '5pm' ) ) assert( Custodian::Util::TimeSpan.inside?( open, close, 18 ) ) - assert( Custodian::Util::TimeSpan.inside?( open, close, "6pm" ) ) + assert( Custodian::Util::TimeSpan.inside?( open, close, '6pm' ) ) # @@ -177,19 +177,19 @@ class TestTimeSpanUtil < Test::Unit::TestCase # # That is true for the string-versions too # - assert( ! Custodian::Util::TimeSpan.inside?( open, close, "3pm" ) ) - assert( ! Custodian::Util::TimeSpan.inside?( open, close, "7pm" ) ) + assert( ! Custodian::Util::TimeSpan.inside?( open, close, '3pm' ) ) + assert( ! Custodian::Util::TimeSpan.inside?( open, close, '7pm' ) ) # # Random hours should be outside too. # assert( ! Custodian::Util::TimeSpan.inside?( open, close, 3 ) ) - assert( ! Custodian::Util::TimeSpan.inside?( open, close, "3am" ) ) + assert( ! Custodian::Util::TimeSpan.inside?( open, close, '3am' ) ) assert( ! Custodian::Util::TimeSpan.inside?( open, close, 7 ) ) - assert( ! Custodian::Util::TimeSpan.inside?( open, close, "7am" ) ) + assert( ! Custodian::Util::TimeSpan.inside?( open, close, '7am' ) ) assert( ! Custodian::Util::TimeSpan.inside?( open, close, 9 ) ) - assert( ! Custodian::Util::TimeSpan.inside?( open, close, "9am" ) ) + assert( ! Custodian::Util::TimeSpan.inside?( open, close, '9am' ) ) end diff --git a/t/test-http-vs-https.rb b/t/test-http-vs-https.rb index b320777..ab44ffc 100755 --- a/t/test-http-vs-https.rb +++ b/t/test-http-vs-https.rb @@ -42,13 +42,13 @@ class TestTestName < Test::Unit::TestCase test = nil assert_nothing_raised do - test = Custodian::TestFactory.create( "http://example.com/ must run http." ) + test = Custodian::TestFactory.create( 'http://example.com/ must run http.' ) end assert( test ) assert( test.kind_of? Array ) assert( ! test.empty? ) - assert_equal( test[0].get_type, "http" ) + assert_equal( test[0].get_type, 'http' ) end @@ -59,13 +59,13 @@ class TestTestName < Test::Unit::TestCase test = nil assert_nothing_raised do - test = Custodian::TestFactory.create( "https://example.com/ must run https." ) + test = Custodian::TestFactory.create( 'https://example.com/ must run https.' ) end assert( test ) assert( test.kind_of? Array ) assert( ! test.empty? ) - assert_equal( test[0].get_type, "https" ) + assert_equal( test[0].get_type, 'https' ) end @@ -76,19 +76,19 @@ class TestTestName < Test::Unit::TestCase assert_raise ArgumentError do - Custodian::TestFactory.create( "https://example.com/ must run http." ) + Custodian::TestFactory.create( 'https://example.com/ must run http.' ) end assert_raise ArgumentError do - Custodian::TestFactory.create( "http://example.com/ must run https." ) + Custodian::TestFactory.create( 'http://example.com/ must run https.' ) end assert_nothing_raised do - Custodian::TestFactory.create( "http://example.com/ must run http." ) + Custodian::TestFactory.create( 'http://example.com/ must run http.' ) end assert_nothing_raised do - Custodian::TestFactory.create( "https://example.com/ must run https." ) + Custodian::TestFactory.create( 'https://example.com/ must run https.' ) end end diff --git a/t/test-ldap-probe.rb b/t/test-ldap-probe.rb index 5273b31..63e95a3 100755 --- a/t/test-ldap-probe.rb +++ b/t/test-ldap-probe.rb @@ -34,7 +34,7 @@ class TestLDAPProbe < Test::Unit::TestCase assert( test.kind_of? Array ) assert( ! test.empty? ) - assert_equal( test[0].get_type, "ldap" ) + assert_equal( test[0].get_type, 'ldap' ) end @@ -47,7 +47,7 @@ class TestLDAPProbe < Test::Unit::TestCase # test data # data = [ - "foo.example.com must run ldap on 389.", + 'foo.example.com must run ldap on 389.', "foo.example.com must run ldap with username 'test'.", "foo.example.com must run ldap with uername 'test' with password 'x'." ] diff --git a/t/test-suite b/t/test-suite index 02df298..26ddb1f 100755 --- a/t/test-suite +++ b/t/test-suite @@ -27,7 +27,7 @@ dir = File.dirname(__FILE__) # # Load each file in the same directory. # -Dir.glob(File.join(dir,"t*.rb")).each do |test| +Dir.glob(File.join(dir,'t*.rb')).each do |test| require test end -- cgit v1.2.1