aboutsummaryrefslogtreecommitdiffstats
path: root/spec
diff options
context:
space:
mode:
Diffstat (limited to 'spec')
-rw-r--r--spec/helpers/public_body_helper_spec.rb93
-rw-r--r--spec/models/public_body_spec.rb106
-rw-r--r--spec/views/public_body/show.html.erb_spec.rb2
3 files changed, 93 insertions, 108 deletions
diff --git a/spec/helpers/public_body_helper_spec.rb b/spec/helpers/public_body_helper_spec.rb
index 89a4d0641..0bf55abb4 100644
--- a/spec/helpers/public_body_helper_spec.rb
+++ b/spec/helpers/public_body_helper_spec.rb
@@ -1,3 +1,4 @@
+# encoding: UTF-8
require File.expand_path(File.dirname(__FILE__) + '/../spec_helper')
describe PublicBodyHelper do
@@ -23,7 +24,7 @@ describe PublicBodyHelper do
@body.tag_string = 'defunct'
msg = 'This authority no longer exists, so you cannot make a request to it.'
expect(public_body_not_requestable_reasons(@body)).to include(msg)
- end
+ end
it 'links to the request page if the body has no contact email' do
@body.request_email = ''
@@ -47,4 +48,94 @@ describe PublicBodyHelper do
end
+
+ describe :type_of_authority do
+
+ it 'falls back to "A public authority"' do
+ public_body = FactoryGirl.build(:public_body)
+ expect(type_of_authority(public_body)).to eq('A public authority')
+ end
+
+ it 'handles Unicode' do
+ category = FactoryGirl.create(:public_body_category, :category_tag => 'spec',
+ :description => 'ünicode category')
+ heading = FactoryGirl.create(:public_body_heading)
+ heading.add_category(category)
+ public_body = FactoryGirl.create(:public_body, :tag_string => 'spec')
+
+
+ expect(type_of_authority(public_body)).to eq('<a href="/body/list/spec">Ünicode category</a>')
+ end
+
+ it 'constructs the correct string if there are tags which are not categories' do
+ heading = FactoryGirl.create(:public_body_heading)
+ 3.times do |i|
+ category = FactoryGirl.create(:public_body_category, :category_tag => "spec_#{i}",
+ :description => "spec category #{i}")
+ heading.add_category(category)
+ end
+ public_body = FactoryGirl.create(:public_body, :tag_string => 'spec_0 spec_2 unknown')
+ expected = '<a href="/body/list/spec_0">Spec category 0</a> and <a href="/body/list/spec_2">spec category 2</a>'
+ expect(type_of_authority(public_body)).to eq(expected)
+ end
+
+
+ context 'when associated with one category' do
+
+ it 'returns the description wrapped in an anchor tag' do
+ category = FactoryGirl.create(:public_body_category, :category_tag => 'spec',
+ :description => 'spec category')
+ heading = FactoryGirl.create(:public_body_heading)
+ heading.add_category(category)
+ public_body = FactoryGirl.create(:public_body, :tag_string => 'spec')
+
+ anchor = %Q(<a href="/body/list/spec">Spec category</a>)
+ expect(type_of_authority(public_body)).to eq(anchor)
+ end
+ end
+
+ context 'when associated with several categories' do
+
+ it 'joins the category descriptions and capitalizes the first letter' do
+ heading = FactoryGirl.create(:public_body_heading)
+ 3.times do |i|
+ category = FactoryGirl.create(:public_body_category, :category_tag => "spec_#{i}",
+ :description => "spec category #{i}")
+ heading.add_category(category)
+ end
+ public_body = FactoryGirl.create(:public_body, :tag_string => 'spec_0 spec_1 spec_2')
+
+ description = [
+ %Q(<a href="/body/list/spec_0">Spec category 0</a>),
+ ', ',
+ %Q(<a href="/body/list/spec_1">spec category 1</a>),
+ ' and ',
+ %Q(<a href="/body/list/spec_2">spec category 2</a>)
+ ].join('')
+
+ expect(type_of_authority(public_body)).to eq(description)
+ end
+
+ end
+
+ context 'when in a non-default locale' do
+
+ it 'creates the anchor href in the correct locale' do
+ # Activate the routing filter, normally turned off for helper tests
+ RoutingFilter.active = true
+ category = FactoryGirl.create(:public_body_category, :category_tag => 'spec',
+ :description => 'spec category')
+ heading = FactoryGirl.create(:public_body_heading)
+ heading.add_category(category)
+ public_body = FactoryGirl.create(:public_body, :tag_string => 'spec')
+
+ anchor = %Q(<a href="/es/body/list/spec">Spec category</a>)
+ I18n.with_locale(:es) { expect(type_of_authority(public_body)
+).to eq(anchor) }
+ end
+
+ end
+
+ end
+
end
diff --git a/spec/models/public_body_spec.rb b/spec/models/public_body_spec.rb
index b90696c25..d91021315 100644
--- a/spec/models/public_body_spec.rb
+++ b/spec/models/public_body_spec.rb
@@ -30,112 +30,6 @@ require File.expand_path(File.dirname(__FILE__) + '/../spec_helper')
describe PublicBody do
- describe :type_of_authority do
-
- it 'falls back to "A public authority"' do
- public_body = FactoryGirl.build(:public_body)
- expect(public_body.type_of_authority).to eq('A public authority')
- end
-
- it 'handles Unicode' do
- category = FactoryGirl.create(:public_body_category, :category_tag => 'spec',
- :description => 'ünicode category')
- heading = FactoryGirl.create(:public_body_heading)
- heading.add_category(category)
- public_body = FactoryGirl.create(:public_body, :tag_string => 'spec')
-
-
- expect(public_body.type_of_authority).to eq('Ünicode category')
- end
-
- it 'constructs the correct string if there are tags which are not categories' do
- heading = FactoryGirl.create(:public_body_heading)
- 3.times do |i|
- category = FactoryGirl.create(:public_body_category, :category_tag => "spec_#{i}",
- :description => "spec category #{i}")
- heading.add_category(category)
- end
- public_body = FactoryGirl.create(:public_body, :tag_string => 'spec_0 spec_2 unknown')
-
- expect(public_body.type_of_authority).to eq('Spec category 0 and spec category 2')
- end
-
- context 'when associated with one category' do
-
- it 'returns the capitalised category description' do
- category = FactoryGirl.create(:public_body_category, :category_tag => 'spec',
- :description => 'spec category')
- heading = FactoryGirl.create(:public_body_heading)
- heading.add_category(category)
- public_body = FactoryGirl.create(:public_body, :tag_string => 'spec')
-
- expect(public_body.type_of_authority).to eq('Spec category')
- end
-
- end
-
- context 'when associated with several categories' do
-
- it 'joins the category descriptions and capitalizes the first letter' do
- heading = FactoryGirl.create(:public_body_heading)
- 3.times do |i|
- category = FactoryGirl.create(:public_body_category, :category_tag => "spec_#{i}",
- :description => "spec category #{i}")
- heading.add_category(category)
- end
- public_body = FactoryGirl.create(:public_body, :tag_string => 'spec_0 spec_1 spec_2')
-
- description = 'Spec category 0, spec category 1 and spec category 2'
- expect(public_body.type_of_authority).to eq(description)
- end
-
- end
-
- context 'when the html parameter is true' do
-
- context 'when associated with one category' do
-
- it 'returns the description wrapped in an anchor tag' do
- category = FactoryGirl.create(:public_body_category, :category_tag => 'spec',
- :description => 'spec category')
- heading = FactoryGirl.create(:public_body_heading)
- heading.add_category(category)
- public_body = FactoryGirl.create(:public_body, :tag_string => 'spec')
-
- anchor = %Q(<a href="/body/list/spec">Spec category</a>)
- expect(public_body.type_of_authority(true)).to eq(anchor)
- end
-
- end
-
- context 'when associated with several categories' do
-
- it 'joins the category descriptions and capitalizes the first letter' do
- heading = FactoryGirl.create(:public_body_heading)
- 3.times do |i|
- category = FactoryGirl.create(:public_body_category, :category_tag => "spec_#{i}",
- :description => "spec category #{i}")
- heading.add_category(category)
- end
- public_body = FactoryGirl.create(:public_body, :tag_string => 'spec_0 spec_1 spec_2')
-
- description = [
- %Q(<a href="/body/list/spec_0">Spec category 0</a>),
- ', ',
- %Q(<a href="/body/list/spec_1">spec category 1</a>),
- ' and ',
- %Q(<a href="/body/list/spec_2">spec category 2</a>)
- ].join('')
-
- expect(public_body.type_of_authority(true)).to eq(description)
- end
-
- end
-
- end
-
- end
-
describe :translations_attributes= do
context 'translation_attrs is a Hash' do
diff --git a/spec/views/public_body/show.html.erb_spec.rb b/spec/views/public_body/show.html.erb_spec.rb
index e4217ca4d..6ebc39caa 100644
--- a/spec/views/public_body/show.html.erb_spec.rb
+++ b/spec/views/public_body/show.html.erb_spec.rb
@@ -7,7 +7,7 @@ describe "public_body/show" do
:short_name => 'tq',
:url_name => 'testquango',
:notes => '',
- :type_of_authority => 'A public body',
+ :tags => [],
:eir_only? => nil,
:info_requests => [1, 2, 3, 4], # out of sync with Xapian
:publication_scheme => '',