pugin 0.6.4 → 0.7.0

data/spec/lib/pugin/feature/bandiera_spec.rb

@@ -0,0 +1,65 @@
+require 'spec_helper'
+require 'pugin/feature/bandiera'
+
+describe 'The bandiera client' do
+
+    context 'when checking if Parliament is in dissolution' do 
+        context 'when the feature exists in Bandiera' do
+
+            before :each do
+                Pugin::Feature::Bandiera.instance_variable_set(:@features, {'show-dissolution' => true})
+            end
+
+            it "returns the value of show-dissolution in the Bandiera database if it exists"  do 
+                expect(Pugin::Feature::Bandiera.dissolution?).to equal(true)
+            end
+        end
+
+        context 'when the feature does not exist in Bandiera' do 
+
+            before :each do
+                Pugin::Feature::Bandiera.instance_variable_set(:@features, {})
+            end
+
+            it "returns false because the feature value is nil"  do 
+                expect(Pugin::Feature::Bandiera.dissolution?).to equal(false)
+            end
+        end
+    end
+
+    context 'when checking if you can still register to vote' do 
+        context 'when the feature exists in Bandiera' do 
+
+            before :each do
+                Pugin::Feature::Bandiera.instance_variable_set(:@features, {'show-register' => true})
+            end
+
+            it "returns the value of show-dissolution in the Bandiera database if it exists"  do 
+                expect(Pugin::Feature::Bandiera.register_to_vote?).to equal(true)
+            end
+        end
+
+        context 'when the feature does not exist in Bandiera' do 
+
+            before :each do
+                Pugin::Feature::Bandiera.instance_variable_set(:@features, {})
+            end
+
+            it "returns false because the feature value is nil"  do 
+                expect(Pugin::Feature::Bandiera.register_to_vote?).to equal(false)
+            end
+        end
+    end
+
+    context 'when trying to reset the @features instance variable' do
+
+        before :each do
+            Pugin::Feature::Bandiera.instance_variable_set(:@features, {'show-register' => true})
+        end
+
+        it 'resets the @features instance variable back to nil' do
+            Pugin::Feature::Bandiera.reset
+            expect(@features).to equal(nil)
+        end
+    end
+end

data/spec/lib/pugin/feature/simple_feature_spec.rb

@@ -0,0 +1,53 @@
+require 'spec_helper'
+
+describe 'Pugin simple feature' do
+
+    before :example do 
+        Pugin::Feature::SimpleFeature.disable
+    end
+
+    after :example do
+        Pugin::Feature::SimpleFeature.disable
+    end
+
+    it 'is initially disabled' do 
+        expect(Pugin::Feature::SimpleFeature.enabled?).to equal(false)
+    end
+
+    context 'without being altered' do 
+        it "returns false when checked if it's enabled" do
+            expect(Pugin::Feature::SimpleFeature.enabled?).to equal(false)
+        end
+        it "returns true when checked if it's disabled" do 
+            expect(Pugin::Feature::SimpleFeature.disabled?).to equal(true)
+        end
+    end
+
+    context 'when being enabled' do
+
+        before :each do
+            Pugin::Feature::SimpleFeature.enable
+        end 
+
+        it "returns true when checked if it's enabled" do
+            expect(Pugin::Feature::SimpleFeature.enabled?).to equal(true)
+        end
+        it "returns false when checked if it's disabled" do 
+            expect(Pugin::Feature::SimpleFeature.disabled?).to equal(false)
+        end
+    end
+
+    context 'when being disabled' do 
+
+        before :each do
+            Pugin::Feature::SimpleFeature.disable
+        end 
+
+        it "returns false when checked if it's enabled" do
+            expect(Pugin::Feature::SimpleFeature.enabled?).to equal(false)
+        end
+        it "returns true when checked if it's disabled" do 
+            expect(Pugin::Feature::SimpleFeature.disabled?).to equal(true)
+        end
+    end
+end

data/spec/lib/pugin/feature/status_banner_spec.rb

@@ -0,0 +1,53 @@
+require 'spec_helper'
+
+describe 'Pugin status banner' do
+
+    before :example do 
+        Pugin::Feature::StatusBanner.disable
+    end
+
+    after :example do
+        Pugin::Feature::StatusBanner.disable
+    end
+
+    it 'is initially disabled' do 
+        expect(Pugin::Feature::StatusBanner.enabled?).to equal(false)
+    end
+
+	context 'without being altered' do 
+		it "returns false when checked if it's enabled" do
+		    expect(Pugin::Feature::StatusBanner.enabled?).to equal(false)
+		end
+		it "returns true when checked if it's disabled" do 
+			expect(Pugin::Feature::StatusBanner.disabled?).to equal(true)
+		end
+    end
+
+    context 'when being enabled' do
+
+    	before :each do
+ 		    Pugin::Feature::StatusBanner.enable
+    	end 
+
+    	it "returns true when checked if it's enabled" do
+    		expect(Pugin::Feature::StatusBanner.enabled?).to equal(true)
+    	end
+    	it "returns false when checked if it's disabled" do 
+    		expect(Pugin::Feature::StatusBanner.disabled?).to equal(false)
+    	end
+ 	end
+
+ 	context 'when being disabled' do 
+
+ 		before :each do
+ 		    Pugin::Feature::StatusBanner.disable
+    	end 
+
+    	it "returns false when checked if it's enabled" do
+    		expect(Pugin::Feature::StatusBanner.enabled?).to equal(false)
+    	end
+    	it "returns true when checked if it's disabled" do 
+    		expect(Pugin::Feature::StatusBanner.disabled?).to equal(true)
+    	end
+ 	end
+end

data/spec/lib/pugin/feature/top_navigation_spec.rb

@@ -0,0 +1,53 @@
+require 'spec_helper'
+
+describe 'Pugin top navigation' do
+
+    before :example do 
+        Pugin::Feature::TopNavigation.disable
+    end
+
+    after :example do
+        Pugin::Feature::TopNavigation.disable
+    end
+
+    it 'is initially disabled' do 
+        expect(Pugin::Feature::TopNavigation.enabled?).to equal(false)
+    end
+
+	context 'without being altered' do 
+		it "returns false when checked if it's enabled" do
+		    expect(Pugin::Feature::TopNavigation.enabled?).to equal(false)
+		end
+		it "returns true when checked if it's disabled" do 
+			expect(Pugin::Feature::TopNavigation.disabled?).to equal(true)
+		end
+    end
+
+    context 'when being enabled' do
+
+    	before :each do
+ 		    Pugin::Feature::TopNavigation.enable
+    	end 
+
+    	it "returns true when checked if it's enabled" do
+    		expect(Pugin::Feature::TopNavigation.enabled?).to equal(true)
+    	end
+    	it "returns false when checked if it's disabled" do 
+    		expect(Pugin::Feature::TopNavigation.disabled?).to equal(false)
+    	end
+ 	end
+
+ 	context 'when being disabled' do 
+
+ 		before :each do
+ 		    Pugin::Feature::TopNavigation.disable
+    	end 
+
+    	it "returns false when checked if it's enabled" do
+    		expect(Pugin::Feature::TopNavigation.enabled?).to equal(false)
+    	end
+    	it "returns true when checked if it's disabled" do 
+    		expect(Pugin::Feature::TopNavigation.disabled?).to equal(true)
+    	end
+ 	end
+end

data/spec/lib/pugin/feature_spec.rb

@@ -0,0 +1,30 @@
+require 'spec_helper'
+
+describe 'Pugin feature module' do
+   context 'when attempting to enable a feature that exists' do
+
+        before :each do 
+            Pugin::Feature.enable('StatusBanner')
+        end
+
+        it 'should successfully enable the feature' do 
+            expect(Pugin::Feature::StatusBanner.enabled?).to equal(true) 
+        end    
+   end
+
+   context 'when attempting to disable a feature that exists' do 
+        before :each do 
+            Pugin::Feature.disable('StatusBanner')
+        end
+
+        it 'should successfully disable the feature' do 
+            expect(Pugin::Feature::StatusBanner.enabled?).to equal(false)
+        end
+   end
+
+   context "when attempting to enable a feature that doesn't exist" do
+        it 'should raise an ArgumentError' do 
+            expect {Pugin::Feature.enable('NonExistentFeature')}.to raise_error(ArgumentError)
+        end
+   end
+end

data/spec/lib/pugin/helpers/controller_helpers_spec.rb

@@ -0,0 +1,60 @@
+require 'spec_helper'
+require 'pugin/helpers/controller_helpers'
+
+describe 'pugin/helpers/controller_helpers.rb' do 
+
+	before do 
+		class TestClass
+			include Pugin::Helpers::ControllerHelpers
+		end 
+	end
+
+	subject { TestClass.new }
+
+	context '#enable_status_banner' do 
+		it 'enables the status banner' do 
+			subject.enable_status_banner
+			expect(Pugin::Feature::TopNavigation.enabled?).to equal(true)
+		end
+	end
+	context '#disable_status_banner' do 
+
+		before :each do 
+			Pugin::Feature::StatusBanner.enable
+		end
+
+		it 'disables the status banner' do 
+			expect(Pugin::Feature::StatusBanner.disabled?).to equal(false)
+			subject.disable_status_banner
+			expect(Pugin::Feature::StatusBanner.disabled?).to equal(true)
+		end
+	end
+	context '#enable_top_navigation' do 
+		it 'enables the top navigation' do 
+			subject.enable_top_navigation
+			expect(Pugin::Feature::TopNavigation.enabled?).to equal(true)
+		end
+	end
+	context '#disable_top_navigation' do 
+
+		before :each do 
+			Pugin::Feature::TopNavigation.enable
+		end
+
+		it 'disables the top navigation' do 
+			expect(Pugin::Feature::TopNavigation.disabled?).to equal(false)
+			subject.disable_top_navigation
+			expect(Pugin::Feature::TopNavigation.disabled?).to equal(true)
+		end
+	end
+	context '#reset_bandiera_features' do 
+		before :each do
+			Pugin::Feature::Bandiera.instance_variable_set(:@features, {'show-register' => true, 'show-dissolution' => true})
+		end
+
+		it 'resets @features back to nil' do
+			subject.reset_bandiera_features
+			expect(Pugin::Feature::Bandiera.instance_variable_get(:@features)).to equal(nil)
+		end
+	end
+end

data/spec/spec_helper.rb

@@ -1,5 +1,6 @@
 require 'simplecov'
 require 'coveralls'
+
 SimpleCov.formatter = SimpleCov::Formatter::MultiFormatter.new([
                                                                  Coveralls::SimpleCov::Formatter,
                                                                  SimpleCov::Formatter::HTMLFormatter
@@ -8,6 +9,8 @@ SimpleCov.start
 
 ENV['RAILS_ENV'] ||= 'test'
 
+require 'bandiera/client'
+
 require File.expand_path('../dummy/config/environment.rb', __FILE__)
 require 'rspec/rails'
 require 'factory_girl_rails'

data/spec/views/pugin/components/_status.html.haml_spec.rb

@@ -6,14 +6,8 @@ describe 'pugin/components/_status.html.haml', type: :view do
   context 'while not in a dissolution' do
 
   	before :each do 
-  	  module FlagHelper
-  		def self.dissolution? 
-  			false
-  		end
-  		def self.register_to_vote? 
-  			false
-  		end
-  	  end
+      allow(Pugin::Feature::Bandiera).to receive(:dissolution?).and_return(false)
+      allow(Pugin::Feature::Bandiera).to receive(:register_to_vote?).and_return(false)
     end
 
     it 'renders a message not related to the dissolution' do
@@ -38,14 +32,8 @@ DATA
   context 'while in a dissolution and still able to register to vote' do
 
   	before :each do 
-  	  module FlagHelper
-  		def self.dissolution? 
-  			true
-  		end
-  		def self.register_to_vote? 
-  			true
-  		end
-  	  end
+      allow(Pugin::Feature::Bandiera).to receive(:dissolution?).and_return(true)
+      allow(Pugin::Feature::Bandiera).to receive(:register_to_vote?).and_return(true)
     end
 
     it 'renders a message that shows a link to register to vote' do
@@ -71,14 +59,8 @@ DATA
   context 'while in a dissolution and registration has closed' do
 
   	before :each do 
-  	  module FlagHelper
-  		def self.dissolution? 
-  			true
-  		end
-  		def self.register_to_vote? 
-  			false
-  		end
-  	  end
+  	  allow(Pugin::Feature::Bandiera).to receive(:dissolution?).and_return(true)
+      allow(Pugin::Feature::Bandiera).to receive(:register_to_vote?).and_return(false)
     end
 
     it 'renders a message that reminds the user of the general election date' do

data/spec/views/pugin/constituencies/list/_list.html.haml_spec.rb

@@ -0,0 +1,78 @@
+require 'spec_helper'
+require 'haml'
+
+describe 'pugin/constituencies/list/_list.html.haml', type: :view do 
+
+  context 'with a current constituency data set' do
+
+    before :each do  
+      constituency = Class.new
+      allow(constituency).to receive(:current?).and_return(true)
+      allow(constituency).to receive(:name).and_return('Aberavon')
+      allow(constituency).to receive(:graph_id).and_return('123')
+      allow(constituency).to receive(:members).and_return([]) 
+      @constituencies = [constituency]
+    end
+
+    it "shows the constituency's name as a link" do 
+      render partial: "pugin/constituencies/list/list", collection: @constituencies, as: "constituencies".to_sym
+      expect(response).to include("<a href='/constituencies/123'>Aberavon</a>")
+    end
+
+    context 'with a current member' do 
+
+      before :each do 
+        constituency = Class.new
+        members = Class.new
+        allow(members).to receive(:display_name).and_return('Test Name')
+        allow(constituency).to receive(:members).and_return([members]) 
+        allow(constituency).to receive(:current?).and_return(true)
+        allow(constituency).to receive(:name).and_return('Aberavon')
+        allow(constituency).to receive(:graph_id).and_return('123')
+        @constituencies = [constituency]
+      end
+
+      it "displays the member's name" do 
+        render partial: "pugin/constituencies/list/list", collection: @constituencies, as: "constituencies".to_sym
+        expect(response).to include("<h2>\n<a href='/constituencies/123'>Aberavon</a>\n</h2>")
+        expect(response).to include("<p>Test Name</p>")
+      end
+    end
+
+    context 'without a current member' do 
+
+      before :each do 
+        constituency = Class.new
+        allow(constituency).to receive(:current?).and_return(true)
+        allow(constituency).to receive(:name).and_return('Aberavon')
+        allow(constituency).to receive(:graph_id).and_return('123')
+        allow(constituency).to receive(:members).and_return([]) 
+        @constituencies = [constituency]
+      end
+
+      it 'displays the vacant i18n message' do 
+        render partial: "pugin/constituencies/list/list", collection: @constituencies, as: "constituencies".to_sym
+        expect(response).to include("<h2>\n<a href='/constituencies/123'>Aberavon</a>\n</h2>")
+        expect(response).to include("<p>Vacant</p>")
+      end
+    end
+  end
+
+  context 'with a previous constituency data set' do 
+
+    before :each do 
+      constituency = Class.new
+      allow(constituency).to receive(:current?).and_return(false)
+      allow(constituency).to receive(:name).and_return('Aberavon')
+      allow(constituency).to receive(:graph_id).and_return('123')
+      @constituencies = [constituency]
+    end
+
+    it 'displays the previous constituency i18n message' do 
+      render partial: "pugin/constituencies/list/list", collection: @constituencies, as: "constituencies".to_sym
+      expect(response).to include("<h2>\n<a href='/constituencies/123'>Aberavon</a>\n</h2>")
+      expect(response).to include("<p>Former constituency</p>")
+    end
+  end
+
+end