pugin 0.7.1 → 0.8.0

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

@@ -1,52 +1,34 @@
 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)
+describe 'The bandiera client feature set' do
+
+    method_hash = {
+        'show-dissolution': :dissolution?,
+        'show-register': :register_to_vote?,
+        'show-election': :election?,
+        'show-post-election': :post_election?
+    }
+
+    method_hash.each do |flag_name, method_name|
+        context flag_name do
+            context 'when the feature exists in Bandiera' do
+                before :each do
+                    Pugin::Feature::Bandiera.instance_variable_set(:@features, {"#{flag_name}" => true})
+                end
+
+                it "returns the value of #{flag_name} in the Bandiera database if it exists" do
+                    expect(Pugin::Feature::Bandiera.send(method_name)).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.send(method_name)).to equal(false)
+                end
             end
         end
     end
@@ -62,4 +44,4 @@ describe 'The bandiera client' do
             expect(@features).to equal(nil)
         end
     end
-end
+end

data/spec/views/pugin/components/_navigation-letter.html.haml_spec.rb

@@ -51,23 +51,11 @@ describe 'pugin/components/_navigation-letter.html.haml', type: :view do
       end
     end
 
-    # Test for primary id and letter
-    context 'and a primary id' do
-      before :each do
-        render partial: 'pugin/components/navigation-letter', locals: { route_symbol: :party_members_letter_path, primary_id: @primary_id }
-      end
-
-      ('a'..'z').each do |letter|
-        it "renders a navigation link for /parties/1234/a-z/#{letter}" do
-          expect(rendered).to include("<a data-atoz-see=\"true\" href=\"/parties/1234/members/a-z/#{letter}\">#{letter}</a>")
-        end
+    context 'and with route arguments' do 
+      before :each do 
+        render partial: 'pugin/components/navigation-letter', locals: { route_symbol: :house_party_members_letter_path, route_args:[@primary_id,@secondary_id] }
       end
-    end
 
-    context 'and both a primary and secondary id' do
-      before :each do
-        render partial: 'pugin/components/navigation-letter', locals: { route_symbol: :house_party_members_letter_path, primary_id: @primary_id, secondary_id: @secondary_id }
-      end
       ('a'..'z').each do |letter|
         it "renders a navigation link for /houses/1234/parties/5678/members/a-z/#{letter}" do
           expect(rendered).to include("<a data-atoz-see=\"true\" href=\"/houses/1234/parties/5678/members/a-z/#{letter}\">#{letter}</a>")
@@ -78,7 +66,7 @@ describe 'pugin/components/_navigation-letter.html.haml', type: :view do
     context "doesn't render links for empty letters" do
       before :each do
         @letters = []
-        render partial: 'pugin/components/navigation-letter', locals: { route_symbol: :party_members_letter_path, primary_id: @primary_id }
+        render partial: 'pugin/components/navigation-letter', locals: { route_symbol: :party_members_letter_path, route_args: [@primary_id] }
       end
 
       ('a'..'z').each do |letter|

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

@@ -8,6 +8,7 @@ describe 'pugin/components/_status.html.haml', type: :view do
   	before :each do 
       allow(Pugin::Feature::Bandiera).to receive(:dissolution?).and_return(false)
       allow(Pugin::Feature::Bandiera).to receive(:register_to_vote?).and_return(false)
+      allow(Pugin::Feature::Bandiera).to receive(:election?).and_return(false)
     end
 
     it 'renders a message not related to the dissolution' do
@@ -85,4 +86,34 @@ DATA
     end
   end
 
+  context 'while on election day' do
+
+    before :each do
+      allow(Pugin::Feature::Bandiera).to receive(:dissolution?).and_return(false)
+      allow(Pugin::Feature::Bandiera).to receive(:register_to_vote?).and_return(false)
+      allow(Pugin::Feature::Bandiera).to receive(:election?).and_return(true)
+    end
+
+    it 'renders a message that reminds the user of the general election date' do
+      render partial: 'pugin/components/status', locals: { status: nil }
+
+      expect(response).to eq(
+<<DATA
+<div class='highlight--status highlight--default'>
+<div class='container'>
+<p>
+This is a test website, so may be inaccurate.
+<a href='http://www.smartsurvey.co.uk/s/ukparliament-beta-website-feedback/' target='_blank' title='website opens in a new window'>Give feedback</a>
+to help improve it.
+Thursday 8 June is election day, find out
+<a href='http://www.parliament.uk/get-involved/elections/voting/' target='_blank' title='website opens in a new window'>how to vote</a>.
+</p>
+</div>
+</div>
+DATA
+      )
+    end
+  end
+
+
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: pugin
 version: !ruby/object:Gem::Version
-  version: 0.7.1
+  version: 0.8.0
 platform: ruby
 authors:
 - Matt Rayner
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2017-06-01 00:00:00.000000000 Z
+date: 2017-06-05 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rails