On 05/27/2011 01:53 PM, Mark McLoughlin wrote:
Hi Jan,
On Fri, 2011-05-27 at 13:37 +0200, jprovazn(a)redhat.com wrote:
> From: Jan Provaznik<jprovazn(a)redhat.com>
>
> task #1622 (
https://www.aeolusproject.org/redmine/issues/1622)
Looks great to me, ACK
> diff --git a/src/features/provider.feature b/src/features/provider.feature
> index 29bf918..8c55748 100644
> --- a/src/features/provider.feature
> +++ b/src/features/provider.feature
> @@ -21,6 +21,20 @@ Feature: Manage Providers
...
> + And each provider should have "name"
> + And each provider should have "url"
> + And each provider should have "provider_type"
...
> diff --git a/src/features/step_definitions/provider_steps.rb
b/src/features/step_definitions/provider_steps.rb
> index f679ba9..57572d7 100644
> --- a/src/features/step_definitions/provider_steps.rb
> +++ b/src/features/step_definitions/provider_steps.rb
> @@ -68,3 +68,19 @@ end
...
> +Then /^each provider should have "([^"]*)"$/ do |arg1|
> + @xml_response.root.xpath("/providers/provider/#{arg1}").text.should_not
be_nil
> +end
Nice :)
Would it be worth extending this to check the contents of the node too?
Cheers,
Mark.
_______________________________________________
aeolus-devel mailing list
aeolus-devel(a)lists.fedorahosted.org
https://fedorahosted.org/mailman/listinfo/aeolus-devel
Hi, I sent version with extended tests and one change again into list,
would be great if you could ACK it, thx.
Jan