jeeves 0.2.2 → 0.2.3

data/lib/jeeves/define_imported_method.rb

@@ -20,7 +20,7 @@ module Jeeves
     def define_lazy
       scope = @scope
       external_name = @external_name
-      @target.class.send(:define_method, @internal_name) do |*args, &block|
+      @target.singleton_class.send(:define_method, @internal_name) do |*args, &block|
         delegator = ResolveDependency.call(scope, external_name)
         delegator.call(*args, &block)
       end
@@ -28,7 +28,7 @@ module Jeeves
 
     def define_non_lazy
       delegator = ResolveDependency.call(@scope, @external_name)
-      @target.class.send(:define_method, @internal_name) do |*args, &block|
+      @target.singleton_class.send(:define_method, @internal_name) do |*args, &block|
         delegator.call(*args, &block)
       end
     end
@@ -37,9 +37,9 @@ module Jeeves
       scope = @scope
       internal_name = @internal_name
       external_name = @external_name
-      @target.class.send(:define_method, @internal_name) do |*args, &block|
+      @target.singleton_class.send(:define_method, @internal_name) do |*args, &block|
         delegator = ResolveDependency.call(scope, external_name)
-        self.class.send(:define_method, internal_name) do |*args, &block|
+        self.singleton_class.send(:define_method, internal_name) do |*args, &block|
           delegator.call(*args, &block)
         end
         delegator.call(*args, &block)

data/lib/jeeves/version.rb

@@ -1,7 +1,7 @@
 module Jeeves
   MAJOR_VERSION = 0
   MINOR_VERSION = 2
-  PATCH_VERSION = 2
+  PATCH_VERSION = 3
   VERSION = [MAJOR_VERSION, MINOR_VERSION, PATCH_VERSION].join('.')
 end
 

data/spec/integration/import_spec.rb

@@ -26,6 +26,10 @@ module JeevesTestApp
       end
     end
 
+    class AClass
+      A_CONSTANT = 4326
+    end
+
     class TestSubject
       extend Jeeves
       import :my_method, :my_callable, :my_constant, from: OtherScope
@@ -33,6 +37,7 @@ module JeevesTestApp
       import :lazy_method, lazy: true
       import :smart_resolver
       import [:marco, :say_polo]
+      import :a_constant, from: AClass, lazy: true
     end
 
     def self.smart_resolver
@@ -94,6 +99,11 @@ describe "import" do
     result.should == "FOO-BAR-BAZ"
   end
 
+  it "defines imported methods in the singleton class" do
+    subject.a_constant.should == 4326
+    # this results in a stack overflow if the methods are defined on Class
+  end
+
   it "raises an error if no importers can find the dependency" do
     Jeeves.stub(:in_test_framework?) { false } # to avoid RSpec integration
     expect do

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: jeeves
 version: !ruby/object:Gem::Version
-  version: 0.2.2
+  version: 0.2.3
   prerelease: 
 platform: ruby
 authors:
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2012-06-20 00:00:00.000000000 Z
+date: 2012-06-26 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rspec