grasshopper 1.0.2 → 1.0.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 5035f3e632e16b9ba58e9a5e96abed5328070b6e
-  data.tar.gz: d3d4ed98cffa450c981d598145e866e020e93d8b
+  metadata.gz: d2f0e558f28c51b97c4458d0443572bd0232fc70
+  data.tar.gz: 18fe3c7c0729018c8963cc253100cc36c93907e5
 SHA512:
-  metadata.gz: 5ad589cd74f3c3333b6f2096118a6b0e0afd53c108f271ea9f453fa19c33b1825828192bc862ba32a1db1332688be9ef64f08ebf72d39bf09331790747a8b899
-  data.tar.gz: 13af0e1a2278196183e06edb6e2ac1d817cd387745363a345e31e44d43d00c9202c6b978e3208e40dd9b7ec6dc57c9d7e6368cff2fe4f9aac3fbe995378c1be2
+  metadata.gz: 081cd246aa43654bf16c1dd466eac5f8354919bd25aae5de58ddf2b2cbaca1e405b8e30de8798b2e802dbdc861ffb6f9d713892b653f85fb44cec06359510f08
+  data.tar.gz: 01e85437d1520afe44a96a3c3d9da1568020aa390b00e9d0ee8ffd2a6036fab0fa9729ee5843a60fdef14cd38d29b7734e161f31db7e5404d93eb155228507eb

data/lib/grasshopper/mock.rb

@@ -19,19 +19,31 @@ module Grasshopper
           index = @messages.index { |heard| request == heard }
         end
         if index.nil?
-          raise "Should have seen an invocation of #{message}(#{args.join(", ")})\n#{@messages.inspect}"
+          raise not_seen_exception(request)
         end
         @messages.delete_at(index || @messages.length)
       else
-        @messages << request
+        record_request(request)
       end
       nil
     end
 
-    MessageHeard = Struct.new(:message, :args)
+    def not_seen_exception(request)
+      NotSeen.new("Should have seen an invocation of #{request})\n\nMessages Seen:\n#{@messages.map(&:to_s).join("\n")}")
+    end
+
+    def record_request(request)
+      @messages << request
+    end
+
+    MessageHeard = Struct.new(:message, :args) do
+      def to_s
+        "#{self.message}(#{self.args.join(", ")})"
+      end
+    end
 
     def self.verify mock
-      raise "Not a #{self.class}" unless mock.is_a? Grasshopper::Mock
+      raise "Tried to verify a #{mock.class}" unless mock.is_a? Grasshopper::Mock
       mock.verify_next
       mock
     end
@@ -43,4 +55,7 @@ module Grasshopper
     class AnyParams
     end
   end
+
+  class NotSeen < ::Exception
+  end
 end

data/test/mock_test.rb

@@ -5,7 +5,7 @@ class MockTest < MiniTest::Test
   def test_fails_verify_if_no_such_message
     mock = Grasshopper::Mock.new
 
-    assert_raises RuntimeError do
+    assert_raises Grasshopper::NotSeen do
       Grasshopper::Mock.verify(mock).never_sent
     end
   end
@@ -26,7 +26,7 @@ class MockTest < MiniTest::Test
     mock = Grasshopper::Mock.new
     mock.message_sent_once
 
-    assert_raises RuntimeError do
+    assert_raises Grasshopper::NotSeen do
       2.times { Grasshopper::Mock.verify(mock).message_sent_once }
     end
   end
@@ -47,16 +47,25 @@ class MockTest < MiniTest::Test
     mock = Grasshopper::Mock.new
     mock.message_with_param("param")
 
-    assert_raises RuntimeError do
+    e = assert_raises Grasshopper::NotSeen do
       Grasshopper::Mock.verify(mock).message_with_param("wrong param")
     end
+
+    expected_message = <<-TXT.chomp
+Should have seen an invocation of message_with_param(wrong param))
+
+Messages Seen:
+message_with_param(param)
+    TXT
+
+    assert_equal(expected_message, e.message)
   end
 
   def test_validate_wrong_number_params_explodes
     mock = Grasshopper::Mock.new
     mock.message_with_param("param", "two")
 
-    assert_raises RuntimeError do
+    assert_raises Grasshopper::NotSeen do
       Grasshopper::Mock.verify(mock).message_with_param("param")
     end
   end
@@ -64,7 +73,11 @@ class MockTest < MiniTest::Test
   def test_any_param_matcher_can_stand_in_for_any_param
     mock = Grasshopper::Mock.new
     mock.value = "nice value"
+    mock.other_value = "other value"
+    mock.takes_two(1, 2)
     Grasshopper::Mock.verify(mock).value= Grasshopper::Mock.any_params
+    Grasshopper::Mock.verify(mock).other_value= Grasshopper::Mock.any_params
+    Grasshopper::Mock.verify(mock).takes_two Grasshopper::Mock.any_params
   end
 
   def test_always_returns_nil
@@ -74,10 +87,13 @@ class MockTest < MiniTest::Test
     assert_nil mock.anything(["even with array params"])
   end
 
+  OtherObject = Struct.new(:foo)
+
   def test_dont_verify_non_mocks
-    non_mock = Object.new
-    assert_raises RuntimeError do
+    non_mock = OtherObject.new
+    e = assert_raises RuntimeError do
       Grasshopper::Mock.verify(non_mock)
     end
+    assert_equal("Tried to verify a MockTest::OtherObject", e.message)
   end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: grasshopper
 version: !ruby/object:Gem::Version
-  version: 1.0.2
+  version: 1.0.3
 platform: ruby
 authors:
 - Matt J Raibert
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2017-04-09 00:00:00.000000000 Z
+date: 2017-04-10 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: minitest