Use setter for key instead AR callback

ref: https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/6763

1 files changed, 10 insertions, 0 deletions

diff --git a/spec/models/key_spec.rb b/spec/models/key_spec.rb
index 7fc6ed1dd54..ba9cdf7dddb 100644
--- a/spec/models/key_spec.rb
+++ b/spec/models/key_spec.rb
@@ -82,4 +82,14 @@ describe Key, models: true do
       @key.destroy
     end
   end
+
+  describe '#key=' do
+    let(:valid_key) do
+      "ssh-rsa AAAAB3NzaC1yc2EAAAABJQAAAIEAiPWx6WM4lhHNedGfBpPJNPpZ7yKu+dnn1SJejgt4596k6YjzGGphH2TUxwKzxcKDKKezwkpfnxPkSMkuEspGRt/aZZ9wa++Oi7Qkr8prgHc4soW6NUlfDzpvZK2H5E7eQaSeP3SAwGmQKUFHCddNaP0L+hM7zhFNzjFvpaMgJw0= dummy@gitlab.com"
+    end
+
+    it 'strips white spaces' do
+      expect(described_class.new(key: " #{valid_key} ").key).to eq(valid_key)
+    end
+  end
 end