Update tests to properly handle latest content caching

Unit tests cannot expects that models will always be fetched. But to make sure things are still well tested, verify the existence of a configuration, project and repository in the homepage feature.

Update tests to properly handle latest content caching
Unit tests cannot expects that models will always be fetched. But to make sure things are still well tested, verify the existence of a configuration, project and repository in the homepage feature.
Daniel · Diego Araújo
1 parent f893cf3d
Showing 3 changed files with 42 additions and 9 deletions Show diff stats
features/homepage.feature
features/step_definitions/kalibro_configuration_steps.rb
spec/controllers/home_controller_spec.rb
@@ -3,9 +3,11 @@ Feature: Homepage
   As a regular user
   I want to have in one page useful links to manage my account and session
-  @kalibro_processor_restart
+  @kalibro_configurations_restart @kalibro_processor_restart
   Scenario: Before signing in
     Given I have a project named "GCC"
+    And there is a public configuration created named "Test Configuration"
+    And I have a sample repository named "Test Repository"
     Then I am at the homepage
     And I should see "Home"
     And I should see "Projects"
@@ -19,6 +21,8 @@ Feature: Homepage
     And I should see "Latest repositories"
     And I should see "Latest configurations"
     Then I should see "GCC" only "1" times
+    Then I should see "Test Configuration" only "1" times
+    Then I should see "Test Repository" only "1" times
   Scenario: Signed in
     Given I am a regular user
@@ -87,6 +87,12 @@ Given(/^there is a public configuration created$/) do
   FactoryGirl.create(:kalibro_configuration_attributes, kalibro_configuration_id: @public_kc.id)
 end
+Given(/^there is a public configuration created named "(.*?)"$/) do |name|
+  @kalibro_configuration = FactoryGirl.create(:public_kalibro_configuration, name: name)
+  FactoryGirl.create(:kalibro_configuration_attributes, kalibro_configuration_id: @kalibro_configuration.id)
+end
+
+
 Given(/^there is a private configuration created$/) do
   @private_kc = FactoryGirl.create(:another_kalibro_configuration)
   FactoryGirl.create(:kalibro_configuration_attributes, :private, kalibro_configuration_id: @private_kc.id, user: FactoryGirl.create(:another_user, id: nil, email: "private@email.com"))
 require 'rails_helper'
 describe HomeController, :type => :controller do
-  context 'Method' do
-    context '#index' do
-      before :each do
-        Project.expects(:latest).with(5).returns([])
-        Repository.expects(:latest).with(5).returns([])
-        KalibroConfiguration.expects(:latest).with(5).returns([])
-      end
-
+  context 'actions' do
+    context 'index' do
       describe 'Rendering' do
         before :each do
           get :index
@@ -41,4 +35,33 @@ describe HomeController, :type =&gt; :controller do
       end
     end
   end
+
+  context 'helpers' do
+    describe 'latest_repositories' do
+      let(:repositories) { mock }
+
+      it 'should fetch the latest content' do
+        Repository.expects(:latest).with(5).returns(repositories)
+        expect(subject.latest_repositories(5)).to be(repositories)
+      end
+    end
+
+    describe 'latest_projects' do
+      let(:projects) { mock }
+
+      it 'should fetch the latest content' do
+        Project.expects(:latest).with(5).returns(projects)
+        expect(subject.latest_projects(5)).to be(projects)
+      end
+    end
+
+    describe 'latest_configurations' do
+      let(:configurations) { mock }
+
+      it 'should fetch the latest content' do
+        KalibroConfiguration.expects(:latest).with(5).returns(configurations)
+        expect(subject.latest_configurations(5)).to be(configurations)
+      end
+    end
+  end
 end
	@@ -87,6 +87,12 @@ Given(/^there is a public configuration created$/) do		@@ -87,6 +87,12 @@ Given(/^there is a public configuration created$/) do
87	FactoryGirl.create(:kalibro_configuration_attributes, kalibro_configuration_id: @public_kc.id)	87	FactoryGirl.create(:kalibro_configuration_attributes, kalibro_configuration_id: @public_kc.id)
88	end	88	end
89		89
		90	+Given(/^there is a public configuration created named "(.*?)"$/) do \|name\|
		91	+ @kalibro_configuration = FactoryGirl.create(:public_kalibro_configuration, name: name)
		92	+ FactoryGirl.create(:kalibro_configuration_attributes, kalibro_configuration_id: @kalibro_configuration.id)
		93	+end
		94	+
		95	+
90	Given(/^there is a private configuration created$/) do	96	Given(/^there is a private configuration created$/) do
91	@private_kc = FactoryGirl.create(:another_kalibro_configuration)	97	@private_kc = FactoryGirl.create(:another_kalibro_configuration)
92	FactoryGirl.create(:kalibro_configuration_attributes, :private, kalibro_configuration_id: @private_kc.id, user: FactoryGirl.create(:another_user, id: nil, email: "private@email.com"))	98	FactoryGirl.create(:kalibro_configuration_attributes, :private, kalibro_configuration_id: @private_kc.id, user: FactoryGirl.create(:another_user, id: nil, email: "private@email.com"))
1	require 'rails_helper'	1	require 'rails_helper'
2		2
3	describe HomeController, :type => :controller do	3	describe HomeController, :type => :controller do
4	- context 'Method' do
5	- context '#index' do
6	- before :each do
7	- Project.expects(:latest).with(5).returns([])
8	- Repository.expects(:latest).with(5).returns([])
9	- KalibroConfiguration.expects(:latest).with(5).returns([])
10	- end
11	-	4	+ context 'actions' do
		5	+ context 'index' do
12	describe 'Rendering' do	6	describe 'Rendering' do
13	before :each do	7	before :each do
14	get :index	8	get :index
	@@ -41,4 +35,33 @@ describe HomeController, :type => :controller do		@@ -41,4 +35,33 @@ describe HomeController, :type => :controller do
41	end	35	end
42	end	36	end
43	end	37	end
		38	+
		39	+ context 'helpers' do
		40	+ describe 'latest_repositories' do
		41	+ let(:repositories) { mock }
		42	+
		43	+ it 'should fetch the latest content' do
		44	+ Repository.expects(:latest).with(5).returns(repositories)
		45	+ expect(subject.latest_repositories(5)).to be(repositories)
		46	+ end
		47	+ end
		48	+
		49	+ describe 'latest_projects' do
		50	+ let(:projects) { mock }
		51	+
		52	+ it 'should fetch the latest content' do
		53	+ Project.expects(:latest).with(5).returns(projects)
		54	+ expect(subject.latest_projects(5)).to be(projects)
		55	+ end
		56	+ end
		57	+
		58	+ describe 'latest_configurations' do
		59	+ let(:configurations) { mock }
		60	+
		61	+ it 'should fetch the latest content' do
		62	+ KalibroConfiguration.expects(:latest).with(5).returns(configurations)
		63	+ expect(subject.latest_configurations(5)).to be(configurations)
		64	+ end
		65	+ end
		66	+ end
44	end	67	end