refactoring: #logger is now a dual head instance/class method

lib/logg/core.rb

@@ -1,20 +1,19 @@
 module Logg
   class Machine 
-    attr_accessor :message, :namespace
+    attr_reader :message, :namespace
 
-    def self.method_missing(meth, *args, &block)
-      @@loggy ||= ::Logg::Machine.new
-      @@loggy.namespace = meth.to_s
-      @@loggy.message   = (args.first.to_s == 'debug') ? nil : args.first.to_s
-      @@loggy.send :output!
+    def method_missing(meth, *args, &block)
+      @namespace = meth.to_s
+      @message   = (args.first.to_s == 'debug') ? nil : args.first.to_s
+      self.send :output!
     end
 
     private
 
     def output!
       debug  = "#{Time.now} | "
-      debug += "[#{self.namespace}] " unless self.namespace.nil?
-      debug += self.message
+      debug += "[#{@namespace}] " unless @namespace.nil?
+      debug += @message
       puts debug
       return debug
     end
@@ -25,7 +24,16 @@ def self.included(base)
       if !base.respond_to?(:logger)
         base.instance_eval do
           def self.logger
-            ::Logg::Machine
+            @@_logg_er ||= ::Logg::Machine.new
+          end
+        end
+
+        def initialize
+          super
+          class << self
+            self
+          end.send(:define_method, :logger) do
+            @_logg_er ||= ::Logg::Machine.new
           end
         end
       else

spec/logg_spec.rb

@@ -7,6 +7,14 @@ class Foo
 class Bar < Foo
 end
 
+shared_examples_for "a Logg::Er user" do
+  describe "the class" do
+    it "should know about self#logger" do
+      u.should respond_to?(:logger)
+    end
+  end
+end
+
 describe Logg do
   context Logg do
     def subject; Logg; end
@@ -23,19 +31,26 @@ def subject; Logg::Machine; end
     it { should be_a Module }
   end
 
-  context Foo do
+  context Foo, "instance" do
+    subject { Foo.new }
+    #it_should_behave_like "a Logg::Er user" do
+      #let(:u) { subject }
+    #end
     it "should output the logging message" do
-      Foo.logger.debug("test").should =~ /\[debug\] test/
+      subject.logger.debug("test").should =~ /\[debug\] test/
+      subject.class.logger.debug("test").should =~ /\[debug\] test/
     end
 
     it "should allow to define the message using a template" do
       pending
     end
   end
 
-  context Bar do
+  context Bar, "instance" do
+    subject { Bar.new }
     it "should output the logging message" do
-      Bar.logger.debug("test").should =~ /\[debug\] test/
+      subject.logger.debug("test").should =~ /\[debug\] test/
+      subject.class.logger.debug("test").should =~ /\[debug\] test/
     end
   end
 end