From e31077c9aaec05bdf5ea0386eb42fcc039d86a0a Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Jos=C3=A9=20Valim?= <jose.valim@gmail.com>
Date: Wed, 23 Dec 2009 12:28:02 +0100
Subject: Small clean up in Naming and TTranslation tests.

---
 activemodel/test/cases/naming_test.rb      |  3 ++-
 activemodel/test/cases/translation_test.rb | 36 +++++++++++++-----------------
 activemodel/test/models/person.rb          |  4 ++++
 activemodel/test/models/track_back.rb      |  4 ++++
 4 files changed, 25 insertions(+), 22 deletions(-)
 create mode 100644 activemodel/test/models/track_back.rb

(limited to 'activemodel/test')

diff --git a/activemodel/test/cases/naming_test.rb b/activemodel/test/cases/naming_test.rb
index fe1ea36450..dc39b84ed8 100644
--- a/activemodel/test/cases/naming_test.rb
+++ b/activemodel/test/cases/naming_test.rb
@@ -1,8 +1,9 @@
 require 'cases/helper'
+require 'models/track_back'
 
 class NamingTest < ActiveModel::TestCase
   def setup
-    @model_name = ActiveModel::Name.new(self, 'Post::TrackBack')
+    @model_name = ActiveModel::Name.new(Post::TrackBack)
   end
 
   def test_singular
diff --git a/activemodel/test/cases/translation_test.rb b/activemodel/test/cases/translation_test.rb
index d171784963..bfc1ca12e6 100644
--- a/activemodel/test/cases/translation_test.rb
+++ b/activemodel/test/cases/translation_test.rb
@@ -1,11 +1,5 @@
 require 'cases/helper'
-
-class SuperUser
-  extend ActiveModel::Translation
-end
-
-class User < SuperUser
-end
+require 'models/person'
 
 class ActiveModelI18nTests < ActiveModel::TestCase
 
@@ -14,38 +8,38 @@ class ActiveModelI18nTests < ActiveModel::TestCase
   end
   
   def test_translated_model_attributes
-    I18n.backend.store_translations 'en', :activemodel => {:attributes => {:super_user => {:name => 'super_user name attribute'} } }
-    assert_equal 'super_user name attribute', SuperUser.human_attribute_name('name')
+    I18n.backend.store_translations 'en', :activemodel => {:attributes => {:person => {:name => 'person name attribute'} } }
+    assert_equal 'person name attribute', Person.human_attribute_name('name')
   end
   
   def test_translated_model_attributes_with_symbols
-    I18n.backend.store_translations 'en', :activemodel => {:attributes => {:super_user => {:name => 'super_user name attribute'} } }
-    assert_equal 'super_user name attribute', SuperUser.human_attribute_name(:name)
+    I18n.backend.store_translations 'en', :activemodel => {:attributes => {:person => {:name => 'person name attribute'} } }
+    assert_equal 'person name attribute', Person.human_attribute_name(:name)
   end
 
   def test_translated_model_attributes_with_ancestor
-    I18n.backend.store_translations 'en', :activemodel => {:attributes => {:user => {:name => 'user name attribute'} } }
-    assert_equal 'user name attribute', User.human_attribute_name('name')
+    I18n.backend.store_translations 'en', :activemodel => {:attributes => {:child => {:name => 'child name attribute'} } }
+    assert_equal 'child name attribute', Child.human_attribute_name('name')
   end
 
   def test_translated_model_attributes_with_ancestors_fallback
-    I18n.backend.store_translations 'en', :activemodel => {:attributes => {:super_user => {:name => 'super_user name attribute'} } }
-    assert_equal 'super_user name attribute', User.human_attribute_name('name')
+    I18n.backend.store_translations 'en', :activemodel => {:attributes => {:person => {:name => 'person name attribute'} } }
+    assert_equal 'person name attribute', Child.human_attribute_name('name')
   end
 
   def test_translated_model_names
-    I18n.backend.store_translations 'en', :activemodel => {:models => {:super_user => 'super_user model'} }
-    assert_equal 'super_user model', SuperUser.model_name.human
+    I18n.backend.store_translations 'en', :activemodel => {:models => {:person => 'person model'} }
+    assert_equal 'person model', Person.model_name.human
   end
 
   def test_translated_model_names_with_sti
-    I18n.backend.store_translations 'en', :activemodel => {:models => {:user => 'user model'} }
-    assert_equal 'user model', User.model_name.human
+    I18n.backend.store_translations 'en', :activemodel => {:models => {:child => 'child model'} }
+    assert_equal 'child model', Child.model_name.human
   end
 
   def test_translated_model_names_with_ancestors_fallback
-    I18n.backend.store_translations 'en', :activemodel => {:models => {:super_user => 'super_user model'} }
-    assert_equal 'super_user model', User.model_name.human
+    I18n.backend.store_translations 'en', :activemodel => {:models => {:person => 'person model'} }
+    assert_equal 'person model', Child.model_name.human
   end
 end
 
diff --git a/activemodel/test/models/person.rb b/activemodel/test/models/person.rb
index d98420f900..fad29a51ec 100644
--- a/activemodel/test/models/person.rb
+++ b/activemodel/test/models/person.rb
@@ -1,5 +1,9 @@
 class Person
   include ActiveModel::Validations
+  extend  ActiveModel::Translation
 
   attr_accessor :title, :karma
 end
+
+class Child < Person
+end
diff --git a/activemodel/test/models/track_back.rb b/activemodel/test/models/track_back.rb
new file mode 100644
index 0000000000..d137e4ff8f
--- /dev/null
+++ b/activemodel/test/models/track_back.rb
@@ -0,0 +1,4 @@
+class Post
+  class TrackBack
+  end
+end
\ No newline at end of file
-- 
cgit v1.2.3