Commit 27f4cf75 authored by Jaakko Kantojärvi's avatar Jaakko Kantojärvi

Tests to validate that invalid keys are rejected

parent 6fd88b8c
...@@ -148,6 +148,12 @@ FactoryGirl.define do ...@@ -148,6 +148,12 @@ FactoryGirl.define do
"ssh-rsa AAAAB3NzaC1yc2EAAAABJQAAAIEAiPWx6WM4lhHNedGfBpPJNPpZ7yKu+dnn1SJejgt4596k6YjzGGphH2TUxwKzxcKDKKezwkpfnxPkSMkuEspGRt/aZZ9wa ++Oi7Qkr8prgHc4soW6NUlfDzpvZK2H5E7eQaSeP3SAwGmQKUFHCddNaP0L+hM7zhFNzjFvpaMgJw0=" "ssh-rsa AAAAB3NzaC1yc2EAAAABJQAAAIEAiPWx6WM4lhHNedGfBpPJNPpZ7yKu+dnn1SJejgt4596k6YjzGGphH2TUxwKzxcKDKKezwkpfnxPkSMkuEspGRt/aZZ9wa ++Oi7Qkr8prgHc4soW6NUlfDzpvZK2H5E7eQaSeP3SAwGmQKUFHCddNaP0L+hM7zhFNzjFvpaMgJw0="
end end
end end
factory :invalid_key do
key do
"ssh-rsa this_is_invalid_key=="
end
end
end end
factory :milestone do factory :milestone do
......
require 'spec_helper' require 'spec_helper'
INVALID_FACTORIES = [:key_with_a_space_in_the_middle] INVALID_FACTORIES = [
:key_with_a_space_in_the_middle,
:invalid_key,
]
FactoryGirl.factories.map(&:name).each do |factory_name| FactoryGirl.factories.map(&:name).each do |factory_name|
next if INVALID_FACTORIES.include?(factory_name) next if INVALID_FACTORIES.include?(factory_name)
......
...@@ -73,8 +73,12 @@ describe Key do ...@@ -73,8 +73,12 @@ describe Key do
build(:key, user: user).should be_valid build(:key, user: user).should be_valid
end end
it "rejects the unfingerprintable key" do it "rejects the unfingerprintable key (contains space in middle)" do
build(:key_with_a_space_in_the_middle).should_not be_valid build(:key_with_a_space_in_the_middle).should_not be_valid
end end
it "rejects the unfingerprintable key (not a key)" do
build(:invalid_key).should_not be_valid
end
end end
end end
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment