[PATCH conductor 1/1] Use mock_model instead of plain mock.
Matt Wagner
matt.wagner at redhat.com
Thu Aug 9 21:02:53 UTC 2012
On Thu, Aug 09, 2012 at 06:17:29PM +0200, jstransk at redhat.com wrote:
> From: Jiri Stransky <jstransk at redhat.com>
>
> ---
> src/spec/controllers/images_controller_spec.rb | 3 +--
> 1 files changed, 1 insertions(+), 2 deletions(-)
>
> diff --git a/src/spec/controllers/images_controller_spec.rb b/src/spec/controllers/images_controller_spec.rb
> index a684a0d..2ace1e8 100644
> --- a/src/spec/controllers/images_controller_spec.rb
> +++ b/src/spec/controllers/images_controller_spec.rb
> @@ -31,8 +31,7 @@ describe ImagesController do
> it "strips whitespace off the image id provided by the user" do
> ProviderAccount.stub(:find).and_return(@provider_account)
> PoolFamily.stub(:find).and_return(@pool_family)
> - @image = mock('image')
> - @image.stub(:id).and_return('456def')
> + @image = mock_model(Aeolus::Image::Factory::Image)
>
> Image.should_receive(:import).
> with { |provider_account, image_id, pool_family, xml| image_id == 'Mock_mock_123abc' }.
> --
> 1.7.7.6
ACK and pushed (to master and 1.1). Thanks for going in and fixing this!
-- Matt
More information about the aeolus-devel
mailing list