rom-sql 2.0.0.beta2 → 2.0.0.beta3

data/spec/integration/auto_migrations/indexes_spec.rb

@@ -1,253 +0,0 @@
-RSpec.describe ROM::SQL::Gateway, :postgres, :helpers do
-  include_context 'database setup'
-
-  before do
-    conn.drop_table?(:users)
-  end
-
-  let(:table_name) { :users }
-  let(:relation_name) { ROM::Relation::Name.new(table_name) }
-
-  subject(:gateway) { container.gateways[:default] }
-
-  let(:inferrer) { ROM::SQL::Schema::Inferrer.new }
-
-  let(:migrated_schema) do
-    empty = define_schema(table_name)
-    empty.with(inferrer.(empty, gateway))
-  end
-
-  let(:attributes) { migrated_schema.to_a }
-
-  def indexdef(index)
-    gateway.connection[<<-SQL, index].first[:indexdef]
-      select indexdef
-      from   pg_indexes
-      where  indexname = ?
-    SQL
-  end
-
-  describe 'create table' do
-    describe 'one-column indexes' do
-      before do
-        conf.relation(:users) do
-          schema do
-            attribute :id,    ROM::SQL::Types::Serial
-            attribute :name,  ROM::SQL::Types::String.meta(index: true)
-
-            indexes do
-              index :name, name: :unique_name, unique: true
-            end
-          end
-        end
-      end
-
-      it 'creates ordinary b-tree indexes' do
-        gateway.auto_migrate!(conf)
-
-        expect(attributes.map(&:to_ast))
-          .to eql([
-                    [:attribute,
-                     [:id,
-                      [:definition, [Integer, {}]],
-                      primary_key: true, source: :users]],
-                    [:attribute,
-                     [:name,
-                      [:definition, [String, {}]],
-                      source: :users]],
-                  ])
-
-        expect(migrated_schema.indexes.first).
-          to eql(ROM::SQL::Index.new(
-                   [define_attribute(:name, :String, source: relation_name)],
-                   name: :unique_name,
-                   unique: true
-                 ))
-      end
-    end
-  end
-
-  describe 'alter table' do
-    describe 'one-column indexes' do
-      context 'adding' do
-        before do
-        end
-
-        it 'adds indexed column' do
-          conn.create_table :users do
-            primary_key :id
-          end
-
-          conf.relation(:users) do
-            schema do
-              attribute :id,    ROM::SQL::Types::Serial
-              attribute :name,  ROM::SQL::Types::String.meta(index: true)
-            end
-          end
-
-          gateway.auto_migrate!(conf)
-
-          name_index = migrated_schema.indexes.first
-
-          expect(migrated_schema.attributes[1].name).to eql(:name)
-          expect(migrated_schema.indexes.size).to eql(1)
-          expect(name_index.name).to eql(:users_name_index)
-          expect(name_index.attributes.map(&:name)).to eql(%i(name))
-        end
-
-        it 'supports custom names' do
-          conn.create_table :users do
-            primary_key :id
-          end
-
-          conf.relation(:users) do
-            schema do
-              attribute :id,    ROM::SQL::Types::Serial
-              attribute :name,  ROM::SQL::Types::String
-
-              indexes do
-                index :name, name: :custom_idx
-              end
-            end
-          end
-
-          gateway.auto_migrate!(conf)
-
-          name_index = migrated_schema.indexes.first
-
-          expect(migrated_schema.attributes[1].name).to eql(:name)
-          expect(migrated_schema.indexes.size).to eql(1)
-          expect(name_index.name).to eql(:custom_idx)
-          expect(name_index.attributes.map(&:name)).to eql(%i(name))
-        end
-
-        it 'adds index to existing column' do
-          conn.create_table :users do
-            primary_key :id
-            column :name, String
-          end
-
-          conf.relation(:users) do
-            schema do
-              attribute :id,    ROM::SQL::Types::Serial
-              attribute :name,  ROM::SQL::Types::String
-
-              indexes do
-                index :name
-              end
-            end
-          end
-
-          gateway.auto_migrate!(conf)
-
-          name_index = migrated_schema.indexes.first
-
-          expect(name_index.name).to eql(:users_name_index)
-          expect(name_index.attributes.map(&:name)).to eql(%i(name))
-          expect(name_index).not_to be_unique
-        end
-
-        it 'supports unique indexes' do
-          conn.create_table :users do
-            primary_key :id
-            column :name, String
-          end
-
-          conf.relation(:users) do
-            schema do
-              attribute :id,    ROM::SQL::Types::Serial
-              attribute :name,  ROM::SQL::Types::String
-
-              indexes do
-                index :name, unique: true
-              end
-            end
-          end
-
-          gateway.auto_migrate!(conf)
-
-          name_index = migrated_schema.indexes.first
-
-          expect(name_index.name).to eql(:users_name_index)
-          expect(name_index.attributes.map(&:name)).to eql(%i(name))
-          expect(name_index).to be_unique
-        end
-
-        if metadata[:postgres]
-          it 'uses index method' do
-            conn.create_table :users do
-              primary_key :id
-              column :props, :jsonb, null: false
-            end
-
-            conf.relation(:users) do
-              schema do
-                attribute :id,     ROM::SQL::Types::Serial
-                attribute :props,  ROM::SQL::Types::PG::JSONB
-
-                indexes do
-                  index :props, type: :gin
-                end
-              end
-            end
-
-            gateway.auto_migrate!(conf)
-
-            expect(indexdef('users_props_index')).
-              to eql('CREATE INDEX users_props_index ON users USING gin (props)')
-          end
-
-          it 'supports partial indexes' do
-            conn.create_table :users do
-              primary_key :id
-              column :name, String
-            end
-
-            conf.relation(:users) do
-              schema do
-                attribute :id,     ROM::SQL::Types::Serial
-                attribute :name,   ROM::SQL::Types::String
-
-                indexes do
-                  index :name, name: :long_names_only, predicate: 'length(name) > 10'
-                end
-              end
-            end
-
-            gateway.auto_migrate!(conf)
-
-            expect(indexdef('long_names_only')).
-              to eql('CREATE INDEX long_names_only ON users USING btree (name) WHERE (length(name) > 10)')
-          end
-        end
-      end
-
-      context 'removing' do
-        before do
-          conf.relation(:users) do
-            schema do
-              attribute :id,    ROM::SQL::Types::Serial
-              attribute :name,  ROM::SQL::Types::String
-              attribute :email, ROM::SQL::Types::String
-            end
-          end
-
-          conn.create_table :users do
-            primary_key :id
-            column :name, String
-            column :email, String
-
-            index :name
-            index :email, name: :email_idx
-          end
-        end
-
-        it 'removes index' do
-          gateway.auto_migrate!(conf)
-
-          expect(migrated_schema.indexes).to be_empty
-        end
-      end
-    end
-  end
-end

data/spec/integration/auto_migrations/managing_columns_spec.rb

@@ -1,156 +0,0 @@
-RSpec.describe ROM::SQL::Gateway, :postgres, :helpers do
-  include_context 'database setup'
-
-  before do
-    conn.drop_table?(:users)
-  end
-
-  before do
-    conf.relation(:users) do
-      schema do
-        attribute :id,    ROM::SQL::Types::Serial
-        attribute :name,  ROM::SQL::Types::String
-        attribute :email, ROM::SQL::Types::String.optional
-      end
-    end
-  end
-
-  let(:table_name) { :users }
-
-  subject(:gateway) { container.gateways[:default] }
-
-  let(:inferrer) { ROM::SQL::Schema::Inferrer.new }
-
-  let(:migrated_schema) do
-    empty = define_schema(table_name)
-    empty.with(inferrer.(empty, gateway))
-  end
-
-  let(:attributes) { migrated_schema.to_a }
-
-  describe 'create a table' do
-    it 'creates a table from a relation' do
-      gateway.auto_migrate!(conf)
-
-      expect(attributes.map(&:to_ast))
-        .to eql([
-                  [:attribute,
-                   [:id,
-                    [:definition, [Integer, {}]],
-                    primary_key: true, source: :users]],
-                  [:attribute, [:name, [:definition, [String, {}]], source: :users]],
-                  [:attribute,
-                   [:email,
-                    [:sum,
-                     [[:constrained,
-                       [[:definition, [NilClass, {}]],
-                        [:predicate, [:type?, [[:type, NilClass], [:input, ROM::Undefined]]]],
-                        {}]],
-                      [:definition, [String, {}]],
-                      {}]],
-                    source: :users]]
-                ])
-    end
-  end
-
-  describe 'adding columns' do
-    before do
-      conn.create_table :users do
-        primary_key :id
-      end
-    end
-
-    it 'adds columns to an existing table' do
-      gateway.auto_migrate!(conf)
-
-      expect(attributes[1].to_ast)
-        .to eql(
-              [:attribute, [:name, [:definition, [String, {}]], source: :users]]
-            )
-      expect(attributes[2].to_ast)
-        .to eql(
-              [:attribute,
-               [:email,
-                [:sum,
-                 [[:constrained,
-                   [[:definition, [NilClass, {}]],
-                    [:predicate, [:type?, [[:type, NilClass], [:input, ROM::Undefined]]]],
-                    {}]],
-                  [:definition, [String, {}]],
-                  {}]],
-                source: :users]]
-            )
-    end
-  end
-
-  describe 'removing columns' do
-    before do
-      conn.create_table :users do
-        primary_key :id
-        column :name, String, null: false
-        column :email, String
-        column :age, Integer, null: false
-      end
-    end
-
-    it 'removes columns from a table' do
-      gateway.auto_migrate!(conf)
-
-      expect(attributes.map(&:name)).to eql(%i(id name email))
-    end
-  end
-
-  describe 'empty diff' do
-    before do
-      conn.create_table :users do
-        primary_key :id
-        column :name, String, null: false
-        column :email, String
-      end
-    end
-
-    it 'leaves existing schema' do
-      current = container.relations[:users].schema
-
-      gateway.auto_migrate!(conf)
-
-      expect(attributes).to eql(current.to_a)
-    end
-  end
-
-  describe 'changing NOTNULL' do
-    describe 'adding' do
-      before do
-        conn.create_table :users do
-          primary_key :id
-          column :name, String
-          column :email, String
-        end
-      end
-
-      it 'adds the constraint' do
-        gateway.auto_migrate!(conf)
-
-        expect(attributes[1].name).to eql(:name)
-        expect(attributes[1]).not_to be_optional
-      end
-    end
-
-    describe 'removing' do
-      before do
-        conn.create_table :users do
-          primary_key :id
-          column :name, String, null: false
-          column :email, String, null: false
-        end
-      end
-
-      it 'removes the constraint' do
-        gateway.auto_migrate!(conf)
-
-        expect(attributes[2].name).to eql(:email)
-        expect(attributes[2]).to be_optional
-      end
-    end
-  end
-end

data/spec/integration/auto_migrations/postgres/column_types_spec.rb

@@ -1,63 +0,0 @@
-RSpec.describe ROM::SQL::Gateway, :postgres, :helpers do
-  include_context 'database setup'
-
-  before do
-    conn.drop_table?(:test_pg_types)
-  end
-
-  let(:table_name) { :test_pg_types }
-
-  subject(:gateway) { container.gateways[:default] }
-
-  let(:inferrer) { ROM::SQL::Schema::Inferrer.new }
-
-  let(:migrated_schema) do
-    empty = define_schema(table_name)
-    empty.with(inferrer.(empty, gateway))
-  end
-
-  let(:attributes) { migrated_schema.to_a }
-
-  describe 'common types' do
-    before do
-      conf.relation(:test_pg_types) do
-        schema do
-          attribute :id,              ROM::SQL::Types::Serial
-          attribute :string,          ROM::SQL::Types::String
-          attribute :int,             ROM::SQL::Types::Int
-          attribute :time,            ROM::SQL::Types::Time
-          attribute :date,            ROM::SQL::Types::Date
-          attribute :decimal,         ROM::SQL::Types::Decimal
-          attribute :string_nullable, ROM::SQL::Types::String.optional
-        end
-      end
-    end
-
-    it 'has support for PG data types' do
-      gateway.auto_migrate!(conf)
-
-      expect(attributes.map(&:to_ast))
-        .to eql([
-                  [:attribute,
-                   [:id,
-                    [:definition, [Integer, {}]],
-                    primary_key: true, source: :test_pg_types]],
-                  [:attribute, [:string, [:definition, [String, {}]], source: :test_pg_types]],
-                  [:attribute, [:int, [:definition, [Integer, {}]], source: :test_pg_types]],
-                  [:attribute, [:time, [:definition, [Time, {}]], source: :test_pg_types]],
-                  [:attribute, [:date, [:definition, [Date, {}]], source: :test_pg_types]],
-                  [:attribute, [:decimal, [:definition, [BigDecimal, {}]], source: :test_pg_types]],
-                  [:attribute,
-                   [:string_nullable,
-                    [:sum,
-                     [[:constrained,
-                       [[:definition, [NilClass, {}]],
-                        [:predicate, [:type?, [[:type, NilClass], [:input, ROM::Undefined]]]],
-                        {}]],
-                      [:definition, [String, {}]],
-                      {}]],
-                    source: :test_pg_types]]
-                ])
-    end
-  end
-end