mirror of
https://github.com/pry/pry.git
synced 2022-11-09 12:35:05 -05:00
Use load instead of eval [Fixes #880]
This commit is contained in:
parent
deea81d152
commit
ef09e90af1
2 changed files with 11 additions and 1 deletions
|
@ -124,7 +124,7 @@ class Pry
|
||||||
|
|
||||||
if reload?(file_name)
|
if reload?(file_name)
|
||||||
silence_warnings do
|
silence_warnings do
|
||||||
TOPLEVEL_BINDING.eval(File.read(file_name), file_name)
|
load file_name
|
||||||
end
|
end
|
||||||
end
|
end
|
||||||
end
|
end
|
||||||
|
|
|
@ -66,6 +66,16 @@ describe "edit" do
|
||||||
FileUtils.rm(tf_path)
|
FileUtils.rm(tf_path)
|
||||||
end
|
end
|
||||||
|
|
||||||
|
it "should work with require relative" do
|
||||||
|
Pry.config.editor = lambda { |file, line|
|
||||||
|
File.open(file, 'w'){ |f| f << 'require_relative "baz.rb"' }
|
||||||
|
File.open(file.gsub('bar.rb', 'baz.rb'), 'w'){ |f| f << "Pad.required = true; FileUtils.rm(__FILE__)" }
|
||||||
|
nil
|
||||||
|
}
|
||||||
|
pry_eval "edit #@tf_path"
|
||||||
|
Pad.required.should == true
|
||||||
|
end
|
||||||
|
|
||||||
describe do
|
describe do
|
||||||
before do
|
before do
|
||||||
Pad.counter = 0
|
Pad.counter = 0
|
||||||
|
|
Loading…
Reference in a new issue