diff options
| -rw-r--r-- | lib/redmine/ciphering.rb | 4 | ||||
| -rw-r--r-- | test/unit/lib/redmine/ciphering_test.rb | 9 |
2 files changed, 11 insertions, 2 deletions
diff --git a/lib/redmine/ciphering.rb b/lib/redmine/ciphering.rb index eea752159..49b76b7e7 100644 --- a/lib/redmine/ciphering.rb +++ b/lib/redmine/ciphering.rb @@ -74,7 +74,7 @@ module Redmine all.each do |object| clear = object.send(attribute) object.send "#{attribute}=", clear - raise(ActiveRecord::Rollback) unless object.save(:validation => false) + raise(ActiveRecord::Rollback) unless object.save(validate: false) end end ? true : false end @@ -84,7 +84,7 @@ module Redmine all.each do |object| clear = object.send(attribute) object.send :write_attribute, attribute, clear - raise(ActiveRecord::Rollback) unless object.save(:validation => false) + raise(ActiveRecord::Rollback) unless object.save(validate: false) end end ? true : false end diff --git a/test/unit/lib/redmine/ciphering_test.rb b/test/unit/lib/redmine/ciphering_test.rb index 95b6fb222..91954b087 100644 --- a/test/unit/lib/redmine/ciphering_test.rb +++ b/test/unit/lib/redmine/ciphering_test.rb @@ -20,6 +20,7 @@ require File.expand_path('../../../../test_helper', __FILE__) class Redmine::CipheringTest < ActiveSupport::TestCase + fixtures :auth_sources def test_password_should_be_encrypted Redmine::Configuration.with 'database_cipher_key' => 'secret' do @@ -106,4 +107,12 @@ class Redmine::CipheringTest < ActiveSupport::TestCase assert_equal 'bar', r.read_attribute(:password) end end + + def test_encrypt_all_and_decrypt_all_should_skip_validation + auth_source = auth_sources(:auth_sources_001) + # validator checks if AuthSource#host is present + auth_source.update_column(:host, nil) + assert AuthSource.encrypt_all(:account_password) + assert AuthSource.decrypt_all(:account_password) + end end |