Merge pull request #409 from rom-rb/fix-specs-in-compat-mode

Fix specs in compat mode

Author: solnic

.github/workflows/ci.yml

@@ -76,7 +76,7 @@ jobs:
     - name: Bundle install
       run: bundle install --jobs 4 --retry 3
     - name: Run all tests
-      run: bundle exec rake
+      run: bundle exec rake spec spec:compat
     - name: Run codacy-coverage-reporter
       uses: codacy/codacy-coverage-reporter-action@master
       if: env.COVERAGE == 'true' && env.COVERAGE_TOKEN != ''

Rakefile

@@ -1,5 +1,13 @@
+# frozen_string_literal: true
+
 require "rspec/core/rake_task"
 
 RSpec::Core::RakeTask.new(:spec)
 
+desc "Run all specs in compat mode"
+task "spec:compat" do
+  ENV["ROM_COMPAT"] = "true"
+  Rake::Task["spec"].invoke
+end
+
 task default: [:spec]

lib/rom/sql/relation.rb

@@ -10,6 +10,7 @@
 
 require "rom/sql/relation/reading"
 require "rom/sql/relation/writing"
+require "rom/sql/schema/dsl"
 
 module ROM
   module SQL
@@ -34,6 +35,7 @@ class Relation < ROM::Relation
         config.attr_class = SQL::Attribute
         config.inferrer = ROM::SQL::Schema::Inferrer.new.freeze
         config.plugins << :indexes
+        config.dsl_class = SQL::Schema::DSL
       end
 
       dataset(abstract: true) do |schema|

lib/rom/sql/schema/dsl.rb

@@ -0,0 +1,42 @@
+# frozen_string_literal: true
+
+require "rom/compat/schema/dsl"
+require_relative "index_dsl"
+
+module ROM
+  module SQL
+    class Schema < ROM::Schema
+      # Specialized schema DSL with SQL-specific features
+      #
+      # @api public
+      # @deprecated
+      class DSL < ROM::Schema::DSL
+        # @!attribute [r] index_dsl
+        #   @return [IndexDSL] Index DSL instance (created only if indexes block is called)
+        attr_reader :index_dsl
+
+        # Define indexes within a block
+        #
+        # @api public
+        def indexes(&block)
+          @index_dsl = IndexDSL.new(**options, &block)
+        end
+
+        private
+
+        # Return schema options
+        #
+        # @api private
+        def opts
+          if index_dsl
+            opts = super
+
+            {**opts, indexes: index_dsl.(relation, opts[:attributes])}
+          else
+            super
+          end
+        end
+      end
+    end
+  end
+end

spec/integration/associations/many_to_many/from_view_spec.rb

@@ -78,6 +78,8 @@
     end
 
     it "prepares joined relations using custom FK" do
+      pending_if_compat_mode
+
       relation = assoc.().order(puzzles[:text].qualified, puzzle_solvers[:user_id].qualified)
 
       expect(relation.schema.map(&:to_sql_name))

spec/integration/associations/many_to_one/from_view_spec.rb

@@ -72,6 +72,8 @@
     end
 
     it "prepares joined relations using custom view in target relation" do
+      pending_if_compat_mode
+
       relation = assoc_inter.()
 
       expect(relation.schema.map(&:to_sql_name))

spec/integration/associations/one_to_many/from_view_spec.rb

@@ -50,6 +50,8 @@
     end
 
     it "prepares joined relations using custom view" do
+      pending_if_compat_mode
+
       relation = assoc.()
 
       expect(relation.schema.map(&:to_sql_name))

spec/integration/auto_migrations/file_based_migrations_spec.rb

@@ -39,6 +39,8 @@ def migrations
     end
 
     it "creates migration files by schema definitions" do
+      pending_if_compat_mode
+
       gateway.auto_migrate!(conf, options)
       expect(migrations.size).to eql(1)
 
@@ -83,6 +85,8 @@ def migrations
       end
 
       it "creates migration files by schema definitions" do
+        pending_if_compat_mode
+
         gateway.auto_migrate!(conf, options)
         expect(migrations.size).to eql(1)
 

spec/integration/auto_migrations/indexes_spec.rb

@@ -45,6 +45,8 @@ def indexdef(index)
       end
 
       it "creates ordinary b-tree indexes" do
+        pending_if_compat_mode
+
         gateway.auto_migrate!(conf, inline: true)
 
         expect(attributes.map(&:to_ast))
@@ -98,6 +100,8 @@ def indexdef(index)
         end
 
         it "supports custom names" do
+          pending_if_compat_mode
+
           conn.create_table :users do
             primary_key :id
           end
@@ -124,6 +128,8 @@ def indexdef(index)
         end
 
         it "adds index to existing column" do
+          pending_if_compat_mode
+
           conn.create_table :users do
             primary_key :id
             column :name, String
@@ -150,6 +156,8 @@ def indexdef(index)
         end
 
         it "supports unique indexes" do
+          pending_if_compat_mode
+
           conn.create_table :users do
             primary_key :id
             column :name, String
@@ -177,6 +185,8 @@ def indexdef(index)
 
         if metadata[:postgres]
           it "uses index method" do
+            pending_if_compat_mode
+
             conn.create_table :users do
               primary_key :id
               column :props, :jsonb, null: false
@@ -200,6 +210,8 @@ def indexdef(index)
           end
 
           it "supports partial indexes" do
+            pending_if_compat_mode
+
             conn.create_table :users do
               primary_key :id
               column :name, String

spec/integration/plugins/auto_restrictions_spec.rb

@@ -70,7 +70,9 @@ class Test::Tasks < ROM::Relation[:sql]
 
       include_context "auto-generated restriction view"
 
-      it "generates restrictrions by a composite index" do
+      it "generates restrictions by a composite index" do
+        pending_if_compat_mode
+
         expect(tasks.by_user_id_and_title(1, "Jane's task").first).to eql(id: 2, user_id: 1, title: "Jane's task")
       end
     end