can't modify frozen NilClass when using HighLine::Simulate

[ngw]

can't modify frozen NilClass when using HighLine::Simulate

[abinoam]

Dear @ngw,

Thank you for reporting the issue, we really appreciate it.

You're using version 1.7.8.
Line 45 of lib/highline/simulate.rb is:

 $terminal.instance_variable_set :@input, @input

The message error is RuntimeError: can't modify frozen NilClass

I guess $terminal global variable is not set when instance_variable_set is called (for the context you're giving).
So, if $terminal is nil then calling instance_variable_set on it will trigger this can't modify frozen NilClass error message.

But, the fragment highline.ask(Cobain::Settings::OPTIONS[setting]) may also be problematic as highline should be equal to $terminal for HighLine::Simulate.with to work well.

I was able to set a small snippet to show what I think could be the cause.

require 'highline/simulate'
HighLine::Simulate.with("foo", "bar") { 2.times { puts ask "Say it again" } }
# => RuntimeError: can't modify frozen NilClass

# Now we require 'highline/import' that sets $terminal to a HighLine instance.
# The same line now will work as expected
require 'highline/import'
HighLine::Simulate.with("foo", "bar") { 2.times { puts ask "Say it again" } }
# => Work well

# And it works the same if using `$terminal.ask`
HighLine::Simulate.with("foo", "bar") { 2.times { puts $terminal.ask "Say it again" } }
# => Work well

# But if I create my own HighLine instance, it won't work as expected.
highline = HighLine.new
HighLine::Simulate.with("foo", "bar") { 2.times { puts highline.ask "Say it again" } }
# => It fails

So, my suggestions are:

• Use 'require highline/import' to make sure it correctly sets $terminal global var
• Just use ask instead of highline.ask.

I may agree that this behaviour is weird. We are slowling deprecating this global variable thing. You can se by the docs that we favor using cli = HighLine.new and cli.ask flow for the 2.0.0-pre versions.
But, we haven't refactored the HighLine::Simulate class. I'll put this on my priority list.

Please, give us some feedback if you could make your tests pass with the above informations. And feel free to ask any futher question.

[ngw]

Unfortunately Thor already has an .ask method: https://github.com/erikhuda/thor/blob/master/lib/thor/shell/basic.rb#L72-L81
Tomorrow I'll try your suggestions and report here what I find out, thanks a lot.

[abinoam]

Name collisions like this made us rethink the way we have been messing with global namespace. Unfortunately we haven't made all changes we would like to make.

I think there's a better workaround then.

require 'highline'
require 'highline/simulate'
simulator = HighLine::Simulate.new(["foo", "bar"])
cli = HighLine.new(simulator) # Sets HighLine input to the simulator instance
2.times { puts cli.ask "Asking something to the simulator" }

[ngw]

It actually worked, I only had to mock HighLine.new to return the new simulator instance.
Some docs about testing would really be useful I guess.

[abinoam]

I'll close the issue. Feel free to reopen it if you want to add any information. Thanks for contributing opening the issue and discussing it.