From d209b4c0a2df0e20c65791bfe4dbdaccb76744fc Mon Sep 17 00:00:00 2001 From: eregon Date: Fri, 1 Dec 2017 17:51:16 +0000 Subject: [PATCH] Update to ruby/spec@e2d0d1e git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@60975 b2dd03c8-39d4-4d8f-98ff-823fe69b080e --- spec/ruby/core/tracepoint/disable_spec.rb | 43 +++++++++++++--------- spec/ruby/core/tracepoint/enable_spec.rb | 44 ++++++++++++++--------- 2 files changed, 53 insertions(+), 34 deletions(-) diff --git a/spec/ruby/core/tracepoint/disable_spec.rb b/spec/ruby/core/tracepoint/disable_spec.rb index 7da644621b..c34eb3ac85 100644 --- a/spec/ruby/core/tracepoint/disable_spec.rb +++ b/spec/ruby/core/tracepoint/disable_spec.rb @@ -3,18 +3,18 @@ require File.expand_path('../../../spec_helper', __FILE__) describe 'TracePoint#disable' do def test; end it 'returns true if trace was enabled' do - event_name, method_name = nil + called = false trace = TracePoint.new(:call) do |tp| - event_name = tp.event - method_name = tp.method_id + called = true end trace.enable trace.disable.should be_true - event_name, method_name = nil + + # Check the TracePoint is disabled + called = false test - method_name.equal?(:test).should be_false - event_name.should equal(nil) + called.should == false end it 'returns false if trace was disabled' do @@ -35,18 +35,24 @@ describe 'TracePoint#disable' do enabled = nil trace = TracePoint.new(:line) {} trace.enable - trace.disable { enabled = trace.enabled? } - enabled.should be_false - trace.enabled?.should be_true - trace.disable + begin + trace.disable { enabled = trace.enabled? } + enabled.should be_false + trace.enabled?.should be_true + ensure + trace.disable + end end it 'is disabled within a block & also returns false when its called with a block' do trace = TracePoint.new(:line) {} trace.enable - trace.disable { trace.enabled? }.should == false - trace.enabled?.should equal(true) - trace.disable + begin + trace.disable { trace.enabled? }.should == false + trace.enabled?.should equal(true) + ensure + trace.disable + end end ruby_bug "#14057", "2.0"..."2.5" do @@ -54,11 +60,14 @@ describe 'TracePoint#disable' do event_name = nil trace = TracePoint.new(:line) {} trace.enable - trace.disable do |*args| - args.should == [] + begin + trace.disable do |*args| + args.should == [] + end + trace.enabled?.should be_true + ensure + trace.disable end - trace.enabled?.should be_true - trace.disable end end end diff --git a/spec/ruby/core/tracepoint/enable_spec.rb b/spec/ruby/core/tracepoint/enable_spec.rb index 675f9939c1..8651c41bad 100644 --- a/spec/ruby/core/tracepoint/enable_spec.rb +++ b/spec/ruby/core/tracepoint/enable_spec.rb @@ -5,20 +5,21 @@ describe 'TracePoint#enable' do describe 'without a block' do it 'returns true if trace was enabled' do - event_name, method_name = nil, nil - method_name = [] + event_name = nil trace = TracePoint.new(:call) do |tp| event_name = tp.event - method_name << tp.method_id end test event_name.should == nil trace.enable - test - event_name.should equal(:call) - trace.disable + begin + test + event_name.should equal(:call) + ensure + trace.disable + end end it 'returns false if trace was disabled' do @@ -29,21 +30,27 @@ describe 'TracePoint#enable' do end trace.enable.should be_false - event_name.should equal(:call) - test - method_name.equal?(:test).should be_true + begin + event_name.should equal(:call) + test + method_name.equal?(:test).should be_true + ensure + trace.disable + end - trace.disable event_name, method_name = nil test method_name.equal?(:test).should be_false event_name.should equal(nil) trace.enable.should be_false - event_name.should equal(:call) - test - method_name.equal?(:test).should be_true - trace.disable + begin + event_name.should equal(:call) + test + method_name.equal?(:test).should be_true + ensure + trace.disable + end end end @@ -71,9 +78,12 @@ describe 'TracePoint#enable' do enabled = nil trace = TracePoint.new(:line) {} trace.enable - trace.enable { enabled = trace.enabled? } - enabled.should == true - trace.disable + begin + trace.enable { enabled = trace.enabled? } + enabled.should == true + ensure + trace.disable + end end it 'returns value returned by the block' do