[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