hatchet 0.0.12 → 0.0.13

data/lib/hatchet/configuration.rb

@@ -90,6 +90,16 @@ module Hatchet
       end
     end
 
+    # Internal: Removes the caching Hash of every appender so that they will all
+    # be re-initialized.
+    #
+    # Used when a change to logging levels is made so that the caches will not
+    # contain stale values.
+    #
+    def clear_levels_cache!
+      appenders.each(&:clear_levels_cache!)
+    end
+
   end
 
 end

data/lib/hatchet/level_manager.rb

@@ -7,7 +7,7 @@ module Hatchet
   #
   module LevelManager
 
-    # Private: All the possible levels of log filter in order of severity.
+    # Internal: All the possible levels of log filter in order of severity.
     #
     LEVELS = [:debug, :info, :warn, :error, :fatal, :off]
 
@@ -24,6 +24,7 @@ module Hatchet
     # Returns nothing.
     def levels=(levels)
       @levels = levels
+      clear_levels_cache!
     end
 
     # Public: Set the lowest level of message to log for the given context.
@@ -31,13 +32,23 @@ module Hatchet
     # level   - The lowest level of message to log for the given context.
     # context - The context that level applies to (default: nil).
     #
-    # Setting a level for nil set the default level for all contexts that have
+    # Setting a level for nil sets the default level for all contexts that have
     # not been specified.
     #
     # Returns nothing.
     def level(level, context = nil)
       context = context.to_s unless context.nil?
       self.levels[context] = level
+      clear_levels_cache!
+    end
+
+    # Public: Set the lowest level of message to log by default.
+    #
+    # level - The lowest level of message to log by default.
+    #
+    # Returns nothing.
+    def level=(level)
+      self.level(level)
     end
 
     # Internal: Returns true if the appender is configured to log messages of
@@ -50,17 +61,34 @@ module Hatchet
     # level within the given context, otherwise returns false.
     #
     def enabled?(level, context)
-      unless self.levels.key? context
-        lvl = self.levels[nil]
+      unless self.levels_cache.key? context
+        lvl = self.levels_cache[nil]
         root = []
         context.to_s.split('::').each do |part|
           root << part
           path = root.join '::'
-          lvl = self.levels[path] if self.levels.key? path
+          lvl = self.levels_cache[path] if self.levels_cache.key? path
         end
-        self.levels[context] = lvl
+        self.levels_cache[context] = lvl
       end
-      LEVELS.index(level) >= LEVELS.index(self.levels[context])
+      LEVELS.index(level) >= LEVELS.index(self.levels_cache[context])
+    end
+
+    # Internal: Returns a lazily duplicated Hash from the levels Hash which is
+    # used to store the calculated logging level for specific contexts to make
+    # subsequent lookups more efficient.
+    #
+    def levels_cache
+      @_levels_cache ||= self.levels.dup
+    end
+
+    # Internal: Removes the caching Hash so that it will be re-initialized.
+    #
+    # Used when a change to logging levels is made so that the cache will not
+    # contain stale values.
+    #
+    def clear_levels_cache!
+      @_levels_cache = nil
     end
 
   end

data/lib/hatchet/version.rb

@@ -4,6 +4,6 @@ module Hatchet
 
   # Public: The version of Hatchet.
   #
-  VERSION = '0.0.12'
+  VERSION = '0.0.13'
 
 end

data/spec/configuration_spec.rb

@@ -91,6 +91,19 @@ describe 'configuration' do
     it 'set to info' do
       assert appender.levels[nil] == :info
     end
+
+    describe "when changed the appender's reflect that change" do
+      before do
+        assert appender.enabled?(:info, 'Foo')
+        Hatchet.configure do |config|
+          config.level = :fatal
+        end
+      end
+
+      it 'should no longer have info enabled' do
+        refute appender.enabled?(:info, 'Foo')
+      end
+    end
   end
 
   describe 'everything wires up' do

data/spec/level_manager_spec.rb

@@ -60,6 +60,14 @@ describe LevelManager do
         assert @manager.enabled?(:info, 'Foo')
       end
     end
+
+    describe 'altering the default level' do
+      it 'alters the enabled level for subsequent calls' do
+        assert @manager.enabled?(:info, 'Foo::Bar')
+        @manager.level = :fatal
+        refute @manager.enabled?(:info, 'Foo::Bar')
+      end
+    end
   end
 end
 

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: hatchet
 version: !ruby/object:Gem::Version
-  version: 0.0.12
+  version: 0.0.13
   prerelease: 
 platform: ruby
 authors:
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2012-07-30 00:00:00.000000000 Z
+date: 2012-08-05 00:00:00.000000000 Z
 dependencies: []
 description: Logging library that provides the ability to add class/module specific
   filters
@@ -65,7 +65,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 1.8.17
+rubygems_version: 1.8.24
 signing_key: 
 specification_version: 3
 summary: Logging library that provides the ability to add class/module specific filters