[Proposal] Allow multiple mock expectations on a Facade

[karptonite]

Not sure if this is a bug or a proposal.
It seems like it should be possible to add multiple expectations to a Facade.
You can do it now like this:

$mock = View::shouldReceive('make')->andReturn(\Mockery::self())->getMock();
$mock->shouldReceive('with')->andReturn(self::RESPONSE);

But it would be cleaner, I think, if you could do it like this:

View::shouldReceive('make')->andReturn(\Mockery::self());
View::shouldReceive('with')->andReturn(self::RESPONSE);

Currently, that will throw an error "Cannot redeclare Mockery_...".

This could be solved if shouldReceive first checked to see if there is already a mock of this facade, and used that mock if there is one.

public static function shouldReceive()
{
  $name = static::getFacadeAccessor();

  if (isset(static::$resolvedInstance[$name]) && strpos(get_class(static::$resolvedInstance[$name]), 'Mockery') !== false)
  {
    $mock = static::$resolvedInstance[$name];
  }
  else
  {
    static::$resolvedInstance[$name] = $mock = \Mockery::mock(get_class(static::getFacadeRoot()));

    static::$app->instance($name, $mock);           
  }

  return call_user_func_array(array($mock, 'shouldReceive'), func_get_args());
}

There is probably a cleaner way of checking if there is a mock, but something like this should allow multiple expectations with a cleaner syntax.

[niallobrien]

I like this implementation.

[karptonite]

I've replaced this with this pull request:
#539
it seemed more like a bug that a proposal for a new feature anyway.