Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

@disable-bundler is broken on 1.0 alpha #526

Closed
doudou opened this issue Jan 18, 2018 · 9 comments
Closed

@disable-bundler is broken on 1.0 alpha #526

doudou opened this issue Jan 18, 2018 · 9 comments
Assignees
Labels

Comments

@doudou
Copy link
Contributor

doudou commented Jan 18, 2018

@disable-bundler is broken (the underlying method has been removed). Not sure whether it is intentional.

The error is

 undefined local variable or method `unset_bundler_env_vars' for #<Object:0x00000003025be0> (NameError)
 ./aruba/lib/aruba/cucumber/hooks.rb:107:in `Before'
 ./aruba/lib/aruba/platforms/local_environment.rb:22:in `call'
 ./aruba/lib/aruba/platforms/unix_platform.rb:78:in `with_environment'
 ./aruba/lib/aruba/api/core.rb:172:in `with_environment'
 ./aruba/lib/aruba/cucumber/hooks.rb:7:in `block in <top (required)>'
 /home/doudou/.rbenv/versions/2.3.1/bin/bundle:23:in `load'
 /home/doudou/.rbenv/versions/2.3.1/bin/bundle:23:in `<main>'
@mvz
Copy link
Contributor

mvz commented Mar 4, 2018

It seems the module containing the underlying method was deprecated in 0.11.0 already, but without any user-facing warnings.

@mvz mvz added the bug label Mar 4, 2018
@mvz mvz self-assigned this Mar 4, 2018
@mvz
Copy link
Contributor

mvz commented Mar 4, 2018

This functionality is very useful and should be restored.

@maxmeyer
Copy link
Member

@mvz
Copy link
Contributor

mvz commented Apr 29, 2018

@maxmeyer so your intention was to remove the @disable-bundler hook altogether and let people use #delete_environment_variable directly?

@maxmeyer
Copy link
Member

maxmeyer commented Apr 29, 2018

I was not aware of this hook at that time. but yes, the idea was to avoid steps/methods which are tied to some tool or language. to make that vision clear I added the "our focus" and "our vision" section to the Readme. I prefer to add examples for such edge cases.

@maxmeyer
Copy link
Member

Unfortunately I had no clear "data" about uses of aruba. I even thought about asking the users what kind of language they use to see if the project has the correct focus. But unfortunately I didn't had the time to set it up properly.

@doudou
Copy link
Contributor Author

doudou commented Apr 30, 2018

I was not aware of this hook at that time. but yes, the idea was to avoid steps/methods which are tied to some tool or language.

The problem with @disable-bundler is that the specificity of the "tool or language" is coming from Cucumber and Aruba themselves. Cucumber and Aruba are installed mostly with bundler, and therefore ends up in a bundler environment by the very nature of cucumber and aruba.

So ... how do you think this could work in my case ?

Basically, the issue I have is that I use aruba and cucumber to bootstrap a robotic framework, which internally uses some Ruby tools. This framework sets up a Bundler environment for itself to run the tools in.

So, I need to cleanup the caller's Bundler environment before I can spawn things in the callee. I guess I could reimplement @disable-bundler inside my environment, but it feels to me that for ruby @disable-bundler sounds like something that would make sense generically.

@stale
Copy link

stale bot commented Jun 29, 2018

This issue has been automatically marked as stale because it has not had recent activity. It will be closed in a week if no further activity occurs.

@stale stale bot added the stale These issues were closed by stalebot and need to be reviewed to see if they're still relevant. label Jun 29, 2018
@mvz
Copy link
Contributor

mvz commented Jun 29, 2018

This has been fixed in master.

@mvz mvz closed this as completed Jun 29, 2018
@mvz mvz removed the stale These issues were closed by stalebot and need to be reviewed to see if they're still relevant. label Jan 27, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants