Model specs for DeployKeys

Dmitriy Zaporozhets
1 parent ff346c01
Showing 4 changed files with 50 additions and 26 deletions Show diff stats
spec/factories.rb
spec/models/deploy_key_spec.rb
spec/models/deploy_keys_project_spec.rb
spec/models/key_spec.rb
@@ -158,8 +158,7 @@ FactoryGirl.define do
       "ssh-rsa AAAAB3NzaC1yc2EAAAABJQAAAIEAiPWx6WM4lhHNedGfBpPJNPpZ7yKu+dnn1SJejgt4596k6YjzGGphH2TUxwKzxcKDKKezwkpfnxPkSMkuEspGRt/aZZ9wa++Oi7Qkr8prgHc4soW6NUlfDzpvZK2H5E7eQaSeP3SAwGmQKUFHCddNaP0L+hM7zhFNzjFvpaMgJw0="
     end
  
-    factory :deploy_key do
-      project
+    factory :deploy_key, class: 'DeployKey' do
     end
  
     factory :personal_key do
@@ -222,4 +221,9 @@ FactoryGirl.define do
     url
     service
   end
+
+  factory :deploy_keys_project do
+    deploy_key
+    project
+  end
 end
@@ -0,0 +1,25 @@
+# == Schema Information
+#
+# Table name: keys
+#
+#  id         :integer          not null, primary key
+#  user_id    :integer
+#  created_at :datetime         not null
+#  updated_at :datetime         not null
+#  key        :text
+#  title      :string(255)
+#  identifier :string(255)
+#  project_id :integer
+#
+
+require 'spec_helper'
+
+describe DeployKey do
+  let(:project) { create(:project) }
+  let(:deploy_key) { create(:deploy_key, projects: [project]) }
+
+  describe "Associations" do
+    it { should have_many(:deploy_keys_projects) }
+    it { should have_many(:projects) }
+  end
+end
@@ -0,0 +1,13 @@
+require 'spec_helper'
+
+describe DeployKeysProject do
+  describe "Associations" do
+    it { should belong_to(:deploy_key) }
+    it { should belong_to(:project) }
+  end
+
+  describe "Validation" do
+    it { should validate_presence_of(:project_id) }
+    it { should validate_presence_of(:deploy_key_id) }
+  end
+end
@@ -17,7 +17,6 @@ require &#39;spec_helper&#39;
 describe Key do
   describe "Associations" do
     it { should belong_to(:user) }
-    it { should belong_to(:project) }
   end
  
   describe "Mass assignment" do
@@ -37,32 +36,15 @@ describe Key do
   end
  
   context "validation of uniqueness" do
+    let(:user) { create(:user) }
  
-    context "as a deploy key" do
-      let!(:deploy_key) { create(:deploy_key) }
-
-      it "does not accept the same key twice for a project" do
-        key = build(:key, project: deploy_key.project)
-        key.should_not be_valid
-      end
-
-      it "does not accept the same key for another project" do
-        key = build(:key, project_id: 0)
-        key.should_not be_valid
-      end
+    it "accepts the key once" do
+      build(:key, user: user).should be_valid
     end
  
-    context "as a personal key" do
-      let(:user) { create(:user) }
-
-      it "accepts the key once" do
-        build(:key, user: user).should be_valid
-      end
-
-      it "does not accepts the key twice" do
-        create(:key, user: user)
-        build(:key, user: user).should_not be_valid
-      end
+    it "does not accepts the key twice" do
+      create(:key, user: user)
+      build(:key, user: user).should_not be_valid
     end
   end
...	...	@@ -158,8 +158,7 @@ FactoryGirl.define do
158	158	"ssh-rsa AAAAB3NzaC1yc2EAAAABJQAAAIEAiPWx6WM4lhHNedGfBpPJNPpZ7yKu+dnn1SJejgt4596k6YjzGGphH2TUxwKzxcKDKKezwkpfnxPkSMkuEspGRt/aZZ9wa++Oi7Qkr8prgHc4soW6NUlfDzpvZK2H5E7eQaSeP3SAwGmQKUFHCddNaP0L+hM7zhFNzjFvpaMgJw0="
159	159	end
160	160
161		- factory :deploy_key do
162		- project
	161	+ factory :deploy_key, class: 'DeployKey' do
163	162	end
164	163
165	164	factory :personal_key do
...	...	@@ -222,4 +221,9 @@ FactoryGirl.define do
222	221	url
223	222	service
224	223	end
	224	+
	225	+ factory :deploy_keys_project do
	226	+ deploy_key
	227	+ project
	228	+ end
225	229	end
...	...
...	...	@@ -0,0 +1,25 @@
	1	+# == Schema Information
	2	+#
	3	+# Table name: keys
	4	+#
	5	+# id :integer not null, primary key
	6	+# user_id :integer
	7	+# created_at :datetime not null
	8	+# updated_at :datetime not null
	9	+# key :text
	10	+# title :string(255)
	11	+# identifier :string(255)
	12	+# project_id :integer
	13	+#
	14	+
	15	+require 'spec_helper'
	16	+
	17	+describe DeployKey do
	18	+ let(:project) { create(:project) }
	19	+ let(:deploy_key) { create(:deploy_key, projects: [project]) }
	20	+
	21	+ describe "Associations" do
	22	+ it { should have_many(:deploy_keys_projects) }
	23	+ it { should have_many(:projects) }
	24	+ end
	25	+end
...	...
...	...	@@ -0,0 +1,13 @@
	1	+require 'spec_helper'
	2	+
	3	+describe DeployKeysProject do
	4	+ describe "Associations" do
	5	+ it { should belong_to(:deploy_key) }
	6	+ it { should belong_to(:project) }
	7	+ end
	8	+
	9	+ describe "Validation" do
	10	+ it { should validate_presence_of(:project_id) }
	11	+ it { should validate_presence_of(:deploy_key_id) }
	12	+ end
	13	+end
...	...
...	...	@@ -17,7 +17,6 @@ require 'spec_helper'
17	17	describe Key do
18	18	describe "Associations" do
19	19	it { should belong_to(:user) }
20		- it { should belong_to(:project) }
21	20	end
22	21
23	22	describe "Mass assignment" do
...	...	@@ -37,32 +36,15 @@ describe Key do
37	36	end
38	37
39	38	context "validation of uniqueness" do
	39	+ let(:user) { create(:user) }
40	40
41		- context "as a deploy key" do
42		- let!(:deploy_key) { create(:deploy_key) }
43		-
44		- it "does not accept the same key twice for a project" do
45		- key = build(:key, project: deploy_key.project)
46		- key.should_not be_valid
47		- end
48		-
49		- it "does not accept the same key for another project" do
50		- key = build(:key, project_id: 0)
51		- key.should_not be_valid
52		- end
	41	+ it "accepts the key once" do
	42	+ build(:key, user: user).should be_valid
53	43	end
54	44
55		- context "as a personal key" do
56		- let(:user) { create(:user) }
57		-
58		- it "accepts the key once" do
59		- build(:key, user: user).should be_valid
60		- end
61		-
62		- it "does not accepts the key twice" do
63		- create(:key, user: user)
64		- build(:key, user: user).should_not be_valid
65		- end
	45	+ it "does not accepts the key twice" do
	46	+ create(:key, user: user)
	47	+ build(:key, user: user).should_not be_valid
66	48	end
67	49	end
68	50
...	...