On Tue, Apr 13, 2010 at 11:35 AM, <[email protected]> wrote:

> From: Tomas Sedovic <[email protected]>
>
> ---
>  src/spec/models/provider_spec.rb |  109
> ++++++++++++++++++++------------------
>  1 files changed, 57 insertions(+), 52 deletions(-)
>
> ACK, with a very minor nit below.


> diff --git a/src/spec/models/provider_spec.rb
> b/src/spec/models/provider_spec.rb
> index 3af6130..cf37ec1 100644
> --- a/src/spec/models/provider_spec.rb
> +++ b/src/spec/models/provider_spec.rb
> @@ -1,73 +1,78 @@
>  require 'spec_helper'
> +
> +    it "should require unique name" do
> +      @provider.save!
> +      provider2 = Factory.build :mock_provider
> +      provider2.stub!(:connect).and_return(@client)
> +      provider2.save!
>

The above save is not needed, test will pass without it, as you are just
checking if this is a valid record by running validate with is_valid.


> +      provider2.should be_valid
>
>
_______________________________________________
deltacloud-devel mailing list
[email protected]
https://fedorahosted.org/mailman/listinfo/deltacloud-devel

Reply via email to