From 73dcfef9e5ff116449961bb20c0dffe08dff036d Mon Sep 17 00:00:00 2001 From: Andy Nicholson Date: Mon, 23 May 2016 14:10:22 +1000 Subject: [PATCH] NullStrategy.cleaning should LocalJumpError without a block, like the other strategies --- lib/database_cleaner/null_strategy.rb | 4 ++-- spec/database_cleaner/null_strategy_spec.rb | 4 ++-- 2 files changed, 4 insertions(+), 4 deletions(-) diff --git a/lib/database_cleaner/null_strategy.rb b/lib/database_cleaner/null_strategy.rb index c232138..f654572 100644 --- a/lib/database_cleaner/null_strategy.rb +++ b/lib/database_cleaner/null_strategy.rb @@ -12,9 +12,9 @@ module DatabaseCleaner # no-op end - def self.cleaning + def self.cleaning(&block) # no-op - yield if block_given? + yield end end end diff --git a/spec/database_cleaner/null_strategy_spec.rb b/spec/database_cleaner/null_strategy_spec.rb index 80a3a7e..3fb1d14 100644 --- a/spec/database_cleaner/null_strategy_spec.rb +++ b/spec/database_cleaner/null_strategy_spec.rb @@ -11,8 +11,8 @@ module DatabaseCleaner end describe '.cleaning' do - it 'no-ops without a block' do - expect { NullStrategy.cleaning }.not_to raise_error(LocalJumpError) + it 'fails without a block' do + expect { NullStrategy.cleaning }.to raise_error(LocalJumpError) end it 'no-ops with a block' do