Throw :abort instead of returning false while before_save

Closes: mbleigh#132

Signed-off-by: Jongmin Kim <[email protected]>

spec/spec_helper.rb

@@ -35,7 +35,13 @@ class SeededModel < ActiveRecord::Base
   attr_protected :first_name if self.respond_to?(:protected_attributes)
   attr_accessor :fail_to_save
 
-  before_save { false if fail_to_save }
+  # From Rails 5.1, returning 'false' will not implicitly halt a callback chain.
+  # It was deprecated from Rails 5.0.
+  if ActiveRecord::VERSION::MAJOR < 5
+    before_save { false if fail_to_save }
+  else
+    before_save { throw :abort if fail_to_save }
+  end
 end
 
 class SeededModelNoPrimaryKey < ActiveRecord::Base