Do not measure timing if debugging is disabled or no block of code wa…

…s given.

lib/vedeu/logging/timer.rb

@@ -33,24 +33,30 @@ def initialize(message = '')
       end
 
       # Write an entry to the log file stating how long a section of
-      # code took in milliseconds. Useful for debugging performance.
+      # code took in milliseconds when debugging is enabled and a
+      # block was Useful for debugging performance.
       #
       # @param block [Proc]
       # @macro raise_requires_block
-      # @return [void] The return value of the executed block.
+      # @return [void|NilClass] The return value of the executed
+      #   block if given, or nil if debugging is disabled.
       def measure(&block)
-        fail Vedeu::Error::RequiresBlock unless block_given?
+        if block_given?
+          if Vedeu.config.debug?
+            work = yield
 
-        if Vedeu.config.debug?
-          work = yield
+            Vedeu.log(type:    :timer,
+                      message: "#{message} took #{elapsed}ms.")
 
-          Vedeu.log(type:    :timer,
-                    message: "#{message} took #{elapsed}ms.")
+            work
 
-          work
+          else
+            yield
+
+          end
 
         else
-          yield
+          fail Vedeu::Error::RequiresBlock if Vedeu.config.debug?
 
         end
       end
@@ -65,6 +71,8 @@ def measure(&block)
       # @return [String]
       attr_reader :message
 
+      private
+
       # Returns the elapsed time in milliseconds with 3 decimal
       # places.
       #

test/lib/vedeu/logging/timer_test.rb

@@ -51,7 +51,17 @@ module Logging
         context 'when the block is not given' do
           subject { instance.measure }
 
-          it { proc { subject }.must_raise(Vedeu::Error::RequiresBlock) }
+          context 'when debugging is enabled' do
+            before { Vedeu.config.stubs(:debug?).returns(true) }
+
+            it { proc { subject }.must_raise(Vedeu::Error::RequiresBlock) }
+          end
+
+          context 'when debugging is disabled' do
+            before { Vedeu.config.stubs(:debug?).returns(false) }
+
+            it { subject.must_equal(nil) }
+          end
         end
       end