rm `Type#number?`

This predicate is only used in `query_attribute`, and is relatively easy
to remove without adding a bunch of is a checks.

14 files changed, 0 insertions, 19 deletions

diff --git a/activerecord/test/cases/adapters/postgresql/array_test.rb b/activerecord/test/cases/adapters/postgresql/array_test.rb
index 35b792b1c2..19ebef4981 100644
--- a/activerecord/test/cases/adapters/postgresql/array_test.rb
+++ b/activerecord/test/cases/adapters/postgresql/array_test.rb
@@ -36,14 +36,12 @@ class PostgresqlArrayTest < ActiveRecord::TestCase
     assert_equal :string, @column.type
     assert_equal "character varying", @column.sql_type
     assert @column.array?
-    assert_not @type.number?
     assert_not @type.binary?
 
     ratings_column = PgArray.columns_hash['ratings']
     assert_equal :integer, ratings_column.type
     type = PgArray.type_for_attribute("ratings")
     assert ratings_column.array?
-    assert_not type.number?
   end
 
   def test_default
diff --git a/activerecord/test/cases/adapters/postgresql/bit_string_test.rb b/activerecord/test/cases/adapters/postgresql/bit_string_test.rb
index 52fa46bd01..6c6b4dc22a 100644
--- a/activerecord/test/cases/adapters/postgresql/bit_string_test.rb
+++ b/activerecord/test/cases/adapters/postgresql/bit_string_test.rb
@@ -30,7 +30,6 @@ class PostgresqlBitStringTest < ActiveRecord::TestCase
     assert_not column.array?
 
     type = PostgresqlBitString.type_for_attribute("a_bit")
-    assert_not type.number?
     assert_not type.binary?
   end
 
@@ -41,7 +40,6 @@ class PostgresqlBitStringTest < ActiveRecord::TestCase
     assert_not column.array?
 
     type = PostgresqlBitString.type_for_attribute("a_bit_varying")
-    assert_not type.number?
     assert_not type.binary?
   end
 
diff --git a/activerecord/test/cases/adapters/postgresql/citext_test.rb b/activerecord/test/cases/adapters/postgresql/citext_test.rb
index 077f0271e2..0ee2a21484 100644
--- a/activerecord/test/cases/adapters/postgresql/citext_test.rb
+++ b/activerecord/test/cases/adapters/postgresql/citext_test.rb
@@ -34,7 +34,6 @@ if ActiveRecord::Base.connection.supports_extensions?
       assert_not column.array?
 
       type = Citext.type_for_attribute('cival')
-      assert_not type.number?
       assert_not type.binary?
     end
 
diff --git a/activerecord/test/cases/adapters/postgresql/composite_test.rb b/activerecord/test/cases/adapters/postgresql/composite_test.rb
index 0c0d2465b2..83dfb18e95 100644
--- a/activerecord/test/cases/adapters/postgresql/composite_test.rb
+++ b/activerecord/test/cases/adapters/postgresql/composite_test.rb
@@ -52,7 +52,6 @@ class PostgresqlCompositeTest < ActiveRecord::TestCase
     assert_not column.array?
 
     type = PostgresqlComposite.type_for_attribute("address")
-    assert_not type.number?
     assert_not type.binary?
   end
 
@@ -115,7 +114,6 @@ class PostgresqlCompositeWithCustomOIDTest < ActiveRecord::TestCase
     assert_not column.array?
 
     type = PostgresqlComposite.type_for_attribute("address")
-    assert_not type.number?
     assert_not type.binary?
   end
 
diff --git a/activerecord/test/cases/adapters/postgresql/domain_test.rb b/activerecord/test/cases/adapters/postgresql/domain_test.rb
index 702de07597..b7d776b40c 100644
--- a/activerecord/test/cases/adapters/postgresql/domain_test.rb
+++ b/activerecord/test/cases/adapters/postgresql/domain_test.rb
@@ -31,7 +31,6 @@ class PostgresqlDomainTest < ActiveRecord::TestCase
     assert_not column.array?
 
     type = PostgresqlDomain.type_for_attribute("price")
-    assert type.number?
     assert_not type.binary?
   end
 
diff --git a/activerecord/test/cases/adapters/postgresql/enum_test.rb b/activerecord/test/cases/adapters/postgresql/enum_test.rb
index 7739d2ee3b..acb09b0607 100644
--- a/activerecord/test/cases/adapters/postgresql/enum_test.rb
+++ b/activerecord/test/cases/adapters/postgresql/enum_test.rb
@@ -33,7 +33,6 @@ class PostgresqlEnumTest < ActiveRecord::TestCase
     assert_not column.array?
 
     type = PostgresqlEnum.type_for_attribute("current_mood")
-    assert_not type.number?
     assert_not type.binary?
   end
 
diff --git a/activerecord/test/cases/adapters/postgresql/full_text_test.rb b/activerecord/test/cases/adapters/postgresql/full_text_test.rb
index 8357dcb3dc..81891a90fa 100644
--- a/activerecord/test/cases/adapters/postgresql/full_text_test.rb
+++ b/activerecord/test/cases/adapters/postgresql/full_text_test.rb
@@ -23,7 +23,6 @@ class PostgresqlFullTextTest < ActiveRecord::TestCase
     assert_not column.array?
 
     type = Tsvector.type_for_attribute("text_vector")
-    assert_not type.number?
     assert_not type.binary?
   end
 
diff --git a/activerecord/test/cases/adapters/postgresql/geometric_test.rb b/activerecord/test/cases/adapters/postgresql/geometric_test.rb
index 5d2e86c5a0..4b25381a83 100644
--- a/activerecord/test/cases/adapters/postgresql/geometric_test.rb
+++ b/activerecord/test/cases/adapters/postgresql/geometric_test.rb
@@ -28,7 +28,6 @@ class PostgresqlPointTest < ActiveRecord::TestCase
     assert_not column.array?
 
     type = PostgresqlPoint.type_for_attribute("x")
-    assert_not type.number?
     assert_not type.binary?
   end
 
diff --git a/activerecord/test/cases/adapters/postgresql/hstore_test.rb b/activerecord/test/cases/adapters/postgresql/hstore_test.rb
index 121f347986..11053a6e38 100644
--- a/activerecord/test/cases/adapters/postgresql/hstore_test.rb
+++ b/activerecord/test/cases/adapters/postgresql/hstore_test.rb
@@ -56,7 +56,6 @@ if ActiveRecord::Base.connection.supports_extensions?
       assert_equal "hstore", @column.sql_type
       assert_not @column.array?
 
-      assert_not @type.number?
       assert_not @type.binary?
     end
 
diff --git a/activerecord/test/cases/adapters/postgresql/json_test.rb b/activerecord/test/cases/adapters/postgresql/json_test.rb
index dd7b67bad7..cbe7e62870 100644
--- a/activerecord/test/cases/adapters/postgresql/json_test.rb
+++ b/activerecord/test/cases/adapters/postgresql/json_test.rb
@@ -36,7 +36,6 @@ module PostgresqlJSONSharedTestCases
     assert_not column.array?
 
     type = JsonDataType.type_for_attribute("payload")
-    assert_not type.number?
     assert_not type.binary?
   end
 
diff --git a/activerecord/test/cases/adapters/postgresql/ltree_test.rb b/activerecord/test/cases/adapters/postgresql/ltree_test.rb
index ca17edfd03..2b3823f9f1 100644
--- a/activerecord/test/cases/adapters/postgresql/ltree_test.rb
+++ b/activerecord/test/cases/adapters/postgresql/ltree_test.rb
@@ -32,7 +32,6 @@ class PostgresqlLtreeTest < ActiveRecord::TestCase
     assert_not column.array?
 
     type = Ltree.type_for_attribute('path')
-    assert_not type.number?
     assert_not type.binary?
   end
 
diff --git a/activerecord/test/cases/adapters/postgresql/money_test.rb b/activerecord/test/cases/adapters/postgresql/money_test.rb
index f5621fd3ec..ba9af4be6f 100644
--- a/activerecord/test/cases/adapters/postgresql/money_test.rb
+++ b/activerecord/test/cases/adapters/postgresql/money_test.rb
@@ -27,7 +27,6 @@ class PostgresqlMoneyTest < ActiveRecord::TestCase
     assert_not column.array?
 
     type = PostgresqlMoney.type_for_attribute("wealth")
-    assert type.number?
     assert_not type.binary?
   end
 
diff --git a/activerecord/test/cases/adapters/postgresql/network_test.rb b/activerecord/test/cases/adapters/postgresql/network_test.rb
index efe754ac7c..4cd2d4d5f3 100644
--- a/activerecord/test/cases/adapters/postgresql/network_test.rb
+++ b/activerecord/test/cases/adapters/postgresql/network_test.rb
@@ -25,7 +25,6 @@ class PostgresqlNetworkTest < ActiveRecord::TestCase
     assert_not column.array?
 
     type = PostgresqlNetworkAddress.type_for_attribute("cidr_address")
-    assert_not type.number?
     assert_not type.binary?
   end
 
@@ -36,7 +35,6 @@ class PostgresqlNetworkTest < ActiveRecord::TestCase
     assert_not column.array?
 
     type = PostgresqlNetworkAddress.type_for_attribute("inet_address")
-    assert_not type.number?
     assert_not type.binary?
   end
 
@@ -47,7 +45,6 @@ class PostgresqlNetworkTest < ActiveRecord::TestCase
     assert_not column.array?
 
     type = PostgresqlNetworkAddress.type_for_attribute("mac_address")
-    assert_not type.number?
     assert_not type.binary?
   end
 
diff --git a/activerecord/test/cases/adapters/postgresql/uuid_test.rb b/activerecord/test/cases/adapters/postgresql/uuid_test.rb
index 1d0f013a26..6693843497 100644
--- a/activerecord/test/cases/adapters/postgresql/uuid_test.rb
+++ b/activerecord/test/cases/adapters/postgresql/uuid_test.rb
@@ -51,7 +51,6 @@ class PostgresqlUUIDTest < ActiveRecord::TestCase
     assert_not column.array?
 
     type = UUIDType.type_for_attribute("guid")
-    assert_not type.number?
     assert_not type.binary?
   end