Slight improvement to failure message for unexpected calls

Author: ryanplusplus

mock.lua

@@ -53,6 +53,9 @@ function ExpectedCall:getReturnValues(...)
   return table.unpack(self._return)
 end
 
+function ExpectedCall:toString()
+  return
+end
 
 
 MockExpectation = {}
@@ -77,7 +80,7 @@ end
 function MockExpectation:when(thunk)
   local function called(m, name, args)
     assert(#self._calls > 0, 'unexpected call')
-    assert(self._calls[1]:functionMatches(m), 'unexpected function "' .. name .. '" called', 2)
+    assert(self._calls[1]:functionMatches(m), 'unexpected function call ' .. name .. '(' .. table.concat(args, ', ') .. ') occurred', 2)
     assert(self._calls[1]:argsMatch(args), 'unexpected arguments provided to function "' .. name .. '"')
 
     return table.remove(self._calls, 1):getReturnValues()

spec/mock_spec.lua

@@ -28,7 +28,7 @@ describe('The mock library', function()
   it('should alert you when a function is not called', function()
     local f = mock:mockFunction('f')
 
-    shouldFail(function()
+    shouldFailWith('not all calls occurred', function()
       mock(f):shouldBeCalled():
       when(function() end)
     end)
@@ -38,7 +38,7 @@ describe('The mock library', function()
     local f1 = mock:mockFunction('f1')
     local f2 = mock:mockFunction('f2')
 
-    shouldFailWith('unexpected function "f2" called', function()
+    shouldFailWith('unexpected function call f2() occurred', function()
       mock(f1):shouldBeCalled():
       when(function() f2() end)
     end)
@@ -47,7 +47,7 @@ describe('The mock library', function()
   it('should alert you when a function is called unexpectedly', function()
     local f = mock:mockFunction('f')
 
-    shouldFailWith('unexpected function "f" called', function()
+    shouldFailWith('unexpected function call f() occurred', function()
       f()
     end)
   end)