summaryrefslogtreecommitdiff
path: root/lib/custodian/worker.rb
diff options
context:
space:
mode:
authorSteve Kemp <steve@steve.org.uk>2015-03-09 13:19:15 +0000
committerSteve Kemp <steve@steve.org.uk>2015-03-09 13:19:15 +0000
commitaf2ee063abea8235a8cbd1448533e4f4fbc0d0af (patch)
tree969e448e2d5995f7e838a68ef1d25d54df9b8d29 /lib/custodian/worker.rb
parentbb2ec1f07af747c69f2fd1e5b70c41b35fb069e6 (diff)
Prefer single-quotes when you don't need interpolation.
So "foo" is less good than 'foo'.
Diffstat (limited to 'lib/custodian/worker.rb')
-rw-r--r--lib/custodian/worker.rb18
1 files changed, 9 insertions, 9 deletions
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