Have explicit :text option for a String column take priority over :si…

…ze option on PostgreSQL (Fixes #1750)

CHANGELOG

@@ -1,5 +1,7 @@
 === master
 
+* Have explicit :text option for a String column take priority over :size option on PostgreSQL (jeremyevans) (#1750)
+
 * Support a :skip_invalid option in auto_validations plugin for not adding errors to a column that already has an error (jeremyevans)
 
 * Support a :skip_invalid option in validation_helpers for not adding an error to a column that already has an error (jeremyevans)

lib/sequel/adapters/shared/postgres.rb

@@ -1500,9 +1500,11 @@ def type_literal_generic_integer(column)
       # disallowed or there is a size specified, use the varchar type.
       # Otherwise use the text type.
       def type_literal_generic_string(column)
-        if column[:fixed]
+        if column[:text]
+          :text
+        elsif column[:fixed]
           "char(#{column[:size]||255})"
-        elsif column[:text] == false or column[:size]
+        elsif column[:text] == false || column[:size]
           "varchar(#{column[:size]||255})"
         else
           :text

spec/adapters/postgres_spec.rb

@@ -235,6 +235,11 @@
     proc{@db.create_table(:some_table, :temp => true, :on_commit => :unsupported){text :name}}.must_raise(Sequel::Error)
   end
 
+  it "should not use a size for text columns" do
+    @db.create_table(:tmp_dolls){String :description, text: true, size: :long}
+    @db.tables.must_include :tmp_dolls
+  end
+
   it "should create an unlogged table" do
     @db.create_table(:unlogged_dolls, :unlogged => true){text :name}
   end if DB.server_version >= 90100