[PATCH conductor] Fix cucumber tests to work with sessionEntity objects upstream

Matt Wagner matt.wagner at redhat.com
Mon Jul 30 20:26:03 UTC 2012


On Mon, Jul 30, 2012 at 03:30:46PM -0400, Jason Guiditta wrote:
> For some reason, upstream won't work with the entity changes w/o this
> hack.
> ---
>  src/features/support/custom.rb |    8 ++++++++
>  1 files changed, 8 insertions(+), 0 deletions(-)
> 
> diff --git a/src/features/support/custom.rb b/src/features/support/custom.rb
> index 02a60a4..6a2299b 100644
> --- a/src/features/support/custom.rb
> +++ b/src/features/support/custom.rb
> @@ -67,3 +67,11 @@ User.class_eval do
>      alias authenticate_using_ldap authenticate
>    end
>  end
> +
> +# FIXME: For some reason, the session doesn't stick around properly in
> +# cucumber without the after_set_user method, so for now, enable it
> +# in the test suite.
> +#
> +Warden::Manager.after_set_user do |user,auth,opts|
> +    SessionEntity.update_session(auth.request.session_options[:id], user)
> +end
> -- 
> 1.7.7.6

ACK. All cukes pass on system ruby, and the only failure with upstream
gems fails with or without this set of patches.

You do call it a hack and use a FIXME tag, which I second, but it's the
only fix we've found thus far, so for now I'm going to push this along
with Scott's patchset.

-- Matt



More information about the aeolus-devel mailing list