minitest-firemock 0.0.1 → 0.0.2

data/.travis.yml

@@ -0,0 +1,4 @@
+rvm:
+ - 1.9.2
+ - 1.9.3
+ - ruby-head

data/README.md

@@ -28,11 +28,10 @@ class MyOtherClassTest < MiniTest::Unit::TestCase
 end
 ```
 
-The only real difference of using `MiniTest::FireMock` instead of `MiniTest::Mock` is that if `MyClass` is defined, and `my_method` isn't there, it'll raise a `MockExpectationError`.
+The only real difference of using `MiniTest::FireMock` instead of `MiniTest::Mock` is that if `MyClass` is defined, and `my_method` isn't there, it'll raise a `MockExpectationError`. It checks also for the arity of the method, so it'll raise a `MockExpectationError` if the real method have a different arity than the expectation.
 
 TODO
 ----
 
-- Check for the arity of the methods if they are defined.
 - Mock class/module methods too.
 - Make it work with method_missing (as of now it doesn't, even if the #responds_to? is correct)

data/Rakefile

@@ -12,3 +12,5 @@ Rake::TestTask.new do |t|
   t.verbose = true
   t.warning = true
 end
+
+task default: :test

data/lib/minitest/fire_mock.rb

@@ -9,10 +9,15 @@ class MiniTest::FireMock < MiniTest::Mock
   end
 
   def expect(name, retval, args = [])
-    if @constant and not @constant.instance_methods.include? name
+    method = @constant.instance_method(name) rescue nil
+    if @constant and not method
       raise MockExpectationError, "expected #{@constant_name} to define `#{name}`, but it doesn't"
     end
 
+    if method and method.arity != args.size
+      raise MockExpectationError, "`#{name}` expects #{method.arity} arguments, given #{args.size}"
+    end
+
     super(name, retval, args)
   end
 

data/minitest-firemock.gemspec

@@ -1,6 +1,6 @@
 # -*- encoding: utf-8 -*-
 Gem::Specification.new do |gem|
-  gem.version       = '0.0.1'
+  gem.version       = '0.0.2'
 
   gem.authors       = ["Cainã Costa"]
   gem.email         = ["cainan.costa@gmail.com"]

data/test/test_helper.rb

@@ -1,5 +1,4 @@
 require 'minitest/unit'
-require 'minitest/fire_mock'
 
 MiniTest::Unit.autorun
 

data/test/unit/firemock_test.rb

@@ -43,4 +43,11 @@ class FireMockTest < MiniTest::Unit::TestCase
       mock.expect(:not_defined_method, 42)
     end
   end
+
+  def test_valid_mock_with_different_arity
+    mock = MiniTest::FireMock.new('DefinedConstant')
+    assert_raises MockExpectationError, "`defined_method` expects 0 arguments, given 3" do
+      mock.expect(:defined_method, 42, [1,2,3])
+    end
+  end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: minitest-firemock
 version: !ruby/object:Gem::Version
-  version: 0.0.1
+  version: 0.0.2
   prerelease: 
 platform: ruby
 authors:
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2011-10-08 00:00:00.000000000Z
+date: 2011-10-10 00:00:00.000000000Z
 dependencies: []
 description: Makes your MiniTest mocks more resilient.
 email:
@@ -18,6 +18,7 @@ executables: []
 extensions: []
 extra_rdoc_files: []
 files:
+- .travis.yml
 - README.md
 - Rakefile
 - lib/minitest/fire_mock.rb
@@ -44,7 +45,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 1.8.10
+rubygems_version: 1.8.7
 signing_key: 
 specification_version: 3
 summary: Makes your MiniTest mocks more resilient.