Move updating of derived attributes to a concern.

Add spec to demonstrate that admin-added bodies aren't getting derived attributes.
Add a spec to demonstrate the problem setting the first letter for translations.
Demonstrate failure to update derived attributes in translations.

1 files changed, 25 insertions, 0 deletions

diff --git a/spec/models/public_body_spec.rb b/spec/models/public_body_spec.rb
index d6abf7b5f..ca94c59a8 100644
--- a/spec/models/public_body_spec.rb
+++ b/spec/models/public_body_spec.rb
@@ -282,6 +282,31 @@ describe PublicBody, " when saving" do
         pb.first_letter.should == 'Å'
     end
 
+    it 'should save the first letter of a translation' do
+        existing = FactoryGirl.create(:public_body, :first_letter => 'T', :name => 'Test body')
+        I18n.with_locale(:es) { existing.update_attributes :name => 'Prueba body' }
+        PublicBody::Translation.
+            where(:public_body_id => existing.id, :locale => :es).
+            pluck('first_letter').first.should == 'P'
+    end
+
+    it 'should save the first letter of a translation, even when it is the same as the
+        first letter in the default locale' do
+        existing = FactoryGirl.create(:public_body, :first_letter => 'T', :name => 'Test body')
+        I18n.with_locale(:es) { existing.update_attributes :name => existing.name }
+        PublicBody::Translation.
+            where(:public_body_id => existing.id, :locale => :es).
+            pluck('first_letter').first.should == 'T'
+    end
+
+    it 'should create a url_name for a translation' do
+        existing = FactoryGirl.create(:public_body, :first_letter => 'T', :short_name => 'Test body')
+        I18n.with_locale(:es) do
+            existing.update_attributes :short_name => 'Prueba', :name => 'Prueba body'
+            existing.url_name.should == 'prueba'
+        end
+    end
+
     it "should not save if the url_name is already taken" do
         existing = FactoryGirl.create(:public_body)
         pb = PublicBody.new(existing.attributes)