activerecord-oracle_enhanced-adapter 1.7.11 → 1.8.2

data/spec/active_record/connection_adapters/oracle_enhanced_connection_spec.rb

@@ -1,5 +1,3 @@
-require 'spec_helper'
-
 describe "OracleEnhancedConnection" do
 
   describe "create connection" do
@@ -61,26 +59,26 @@ describe "OracleEnhancedConnection" do
 
   describe "create connection with NLS parameters" do
     after do
-      ENV['NLS_DATE_FORMAT'] = nil
+      ENV["NLS_DATE_FORMAT"] = nil
     end
 
     it "should use NLS_DATE_FORMAT environment variable" do
-      ENV['NLS_DATE_FORMAT'] = 'YYYY-MM-DD'
+      ENV["NLS_DATE_FORMAT"] = "YYYY-MM-DD"
       @conn = ActiveRecord::ConnectionAdapters::OracleEnhancedConnection.create(CONNECTION_PARAMS)
-      expect(@conn.select("SELECT value FROM v$nls_parameters WHERE parameter = 'NLS_DATE_FORMAT'")).to eq([{'value' => 'YYYY-MM-DD'}])
+      expect(@conn.select("select SYS_CONTEXT('userenv', 'NLS_DATE_FORMAT') as value from dual")).to eq([{ "value" => "YYYY-MM-DD" }])
     end
 
     it "should use configuration value and ignore NLS_DATE_FORMAT environment variable" do
-      ENV['NLS_DATE_FORMAT'] = 'YYYY-MM-DD'
-      @conn = ActiveRecord::ConnectionAdapters::OracleEnhancedConnection.create(CONNECTION_PARAMS.merge(:nls_date_format => 'YYYY-MM-DD HH24:MI'))
-      expect(@conn.select("SELECT value FROM v$nls_parameters WHERE parameter = 'NLS_DATE_FORMAT'")).to eq([{'value' => 'YYYY-MM-DD HH24:MI'}])
+      ENV["NLS_DATE_FORMAT"] = "YYYY-MM-DD"
+      @conn = ActiveRecord::ConnectionAdapters::OracleEnhancedConnection.create(CONNECTION_PARAMS.merge(nls_date_format: "YYYY-MM-DD HH24:MI"))
+      expect(@conn.select("select SYS_CONTEXT('userenv', 'NLS_DATE_FORMAT') as value from dual")).to eq([{ "value" => "YYYY-MM-DD HH24:MI" }])
     end
 
     it "should use default value when NLS_DATE_FORMAT environment variable is not set" do
-      ENV['NLS_DATE_FORMAT'] = nil
+      ENV["NLS_DATE_FORMAT"] = nil
       @conn = ActiveRecord::ConnectionAdapters::OracleEnhancedConnection.create(CONNECTION_PARAMS)
       default = ActiveRecord::ConnectionAdapters::OracleEnhancedAdapter::DEFAULT_NLS_PARAMETERS[:nls_date_format]
-      expect(@conn.select("SELECT value FROM v$nls_parameters WHERE parameter = 'NLS_DATE_FORMAT'")).to eq([{'value' => default}])
+      expect(@conn.select("select SYS_CONTEXT('userenv', 'NLS_DATE_FORMAT') as value from dual")).to eq([{ "value" => default }])
     end
   end
 
@@ -116,7 +114,7 @@ describe "OracleEnhancedConnection" do
     before(:all) do
       ActiveRecord::Base.establish_connection(CONNECTION_WITH_TIMEZONE_PARAMS)
       schema_define do
-        create_table :posts, :force => true do |t|
+        create_table :posts, force: true do |t|
           t.timestamps null: false
         end
       end
@@ -143,9 +141,9 @@ describe "OracleEnhancedConnection" do
     let(:username) { CONNECTION_PARAMS[:username] }
     let(:password) { CONNECTION_PARAMS[:password] }
     let(:connection_string) { "(DESCRIPTION=(ADDRESS_LIST=(ADDRESS=(PROTOCOL=tcp)(HOST=#{DATABASE_HOST})(PORT=#{DATABASE_PORT})))(CONNECT_DATA=(SERVICE_NAME=#{DATABASE_NAME})))" }
-    let(:params) { { username: username, password: password, host: 'connection-string', database: connection_string } }
+    let(:params) { { username: username, password: password, host: "connection-string", database: connection_string } }
 
-    it 'uses the database param as the connection string' do
+    it "uses the database param as the connection string" do
       if ORACLE_ENHANCED_CONNECTION == :jdbc
         expect(java.sql.DriverManager).to receive(:getConnection).with("jdbc:oracle:thin:@#{connection_string}", anything).and_call_original
       else
@@ -156,7 +154,7 @@ describe "OracleEnhancedConnection" do
     end
   end
 
-  if defined?(RUBY_ENGINE) && RUBY_ENGINE == 'jruby'
+  if defined?(RUBY_ENGINE) && RUBY_ENGINE == "jruby"
 
     describe "create JDBC connection" do
 
@@ -189,11 +187,11 @@ describe "OracleEnhancedConnection" do
       it "should create a new connection using JNDI" do
 
         begin
-          import 'oracle.jdbc.driver.OracleDriver'
-          import 'org.apache.commons.pool.impl.GenericObjectPool'
-          import 'org.apache.commons.dbcp.PoolingDataSource'
-          import 'org.apache.commons.dbcp.PoolableConnectionFactory'
-          import 'org.apache.commons.dbcp.DriverManagerConnectionFactory'
+          import "oracle.jdbc.driver.OracleDriver"
+          import "org.apache.commons.pool.impl.GenericObjectPool"
+          import "org.apache.commons.dbcp.PoolingDataSource"
+          import "org.apache.commons.dbcp.PoolableConnectionFactory"
+          import "org.apache.commons.dbcp.DriverManagerConnectionFactory"
         rescue NameError => e
           return skip e.message
         end
@@ -203,12 +201,12 @@ describe "OracleEnhancedConnection" do
             connection_pool = GenericObjectPool.new(nil)
             uri = "jdbc:oracle:thin:@#{DATABASE_HOST && "#{DATABASE_HOST}:"}#{DATABASE_PORT && "#{DATABASE_PORT}:"}#{DATABASE_NAME}"
             connection_factory = DriverManagerConnectionFactory.new(uri, DATABASE_USER, DATABASE_PASSWORD)
-            poolable_connection_factory = PoolableConnectionFactory.new(connection_factory,connection_pool,nil,nil,false,true)
+            poolable_connection_factory = PoolableConnectionFactory.new(connection_factory, connection_pool, nil, nil, false, true)
             @data_source = PoolingDataSource.new(connection_pool)
             @data_source.access_to_underlying_connection_allowed = true
           end
           def lookup(path)
-            if (path == 'java:/comp/env')
+            if (path == "java:/comp/env")
               return self
             else
               return @data_source
@@ -219,7 +217,7 @@ describe "OracleEnhancedConnection" do
         allow(javax.naming.InitialContext).to receive(:new).and_return(InitialContextMock.new)
 
         params = {}
-        params[:jndi] = 'java:comp/env/jdbc/test'
+        params[:jndi] = "java:comp/env/jdbc/test"
         @conn = ActiveRecord::ConnectionAdapters::OracleEnhancedConnection.create(params)
         expect(@conn).to be_active
       end
@@ -231,7 +229,7 @@ describe "OracleEnhancedConnection" do
       params[:url] = "jdbc:oracle:thin:@#{DATABASE_HOST && "//#{DATABASE_HOST}#{DATABASE_PORT && ":#{DATABASE_PORT}"}/"}#{DATABASE_NAME}"
       params[:host] = nil
       params[:database] = nil
-      allow(java.sql.DriverManager).to receive(:getConnection).and_raise('no suitable driver found')
+      allow(java.sql.DriverManager).to receive(:getConnection).and_raise("no suitable driver found")
       @conn = ActiveRecord::ConnectionAdapters::OracleEnhancedConnection.create(params)
       expect(@conn).to be_active
     end
@@ -248,11 +246,11 @@ describe "OracleEnhancedConnection" do
     end
 
     it "should execute SQL select" do
-      expect(@conn.select("SELECT * FROM dual")).to eq([{'dummy' => 'X'}])
+      expect(@conn.select("SELECT * FROM dual")).to eq([{ "dummy" => "X" }])
     end
 
     it "should execute SQL select and return also columns" do
-      expect(@conn.select("SELECT * FROM dual", nil, true)).to eq([ [{'dummy' => 'X'}], ['dummy'] ])
+      expect(@conn.select("SELECT * FROM dual", nil, true)).to eq([ [{ "dummy" => "X" }], ["dummy"] ])
     end
 
   end
@@ -266,7 +264,7 @@ describe "OracleEnhancedConnection" do
       cursor = @conn.prepare("SELECT * FROM dual WHERE :1 = 1")
       cursor.bind_param(1, 1)
       cursor.exec
-      expect(cursor.get_col_names).to eq(['DUMMY'])
+      expect(cursor.get_col_names).to eq(["DUMMY"])
       expect(cursor.fetch).to eq(["X"])
       cursor.close
     end
@@ -285,20 +283,20 @@ describe "OracleEnhancedConnection" do
 
   describe "SQL with bind parameters when NLS_NUMERIC_CHARACTERS is set to ', '" do
     before(:all) do
-      ENV['NLS_NUMERIC_CHARACTERS'] = ", "
+      ENV["NLS_NUMERIC_CHARACTERS"] = ", "
       @conn = ActiveRecord::ConnectionAdapters::OracleEnhancedConnection.create(CONNECTION_PARAMS)
       @conn.exec "CREATE TABLE test_employees (age NUMBER(10,2))"
     end
 
     after(:all) do
-      ENV['NLS_NUMERIC_CHARACTERS'] = nil
+      ENV["NLS_NUMERIC_CHARACTERS"] = nil
       @conn.exec "DROP TABLE test_employees" rescue nil
     end
 
     it "should execute prepared statement with decimal bind parameter " do
       cursor = @conn.prepare("INSERT INTO test_employees VALUES(:1)")
       type_metadata = ActiveRecord::ConnectionAdapters::SqlTypeMetadata.new(sql_type: "NUMBER", type: :decimal, limit: 10, precision: nil, scale: 2)
-      column = ActiveRecord::ConnectionAdapters::OracleEnhancedColumn.new('age', nil, type_metadata, false, "test_employees", false, false, nil)
+      column = ActiveRecord::ConnectionAdapters::OracleEnhancedColumn.new("age", nil, type_metadata, false, "test_employees", false, false, nil)
       expect(column.type).to eq(:decimal)
       # Here 1.5 expects that this value has been type casted already
       # it should use bind_params in the long term.
@@ -324,10 +322,10 @@ describe "OracleEnhancedConnection" do
     end
 
     def kill_current_session
-      audsid = @conn.select("SELECT userenv('sessionid') audsid FROM dual").first['audsid']
+      audsid = @conn.select("SELECT userenv('sessionid') audsid FROM dual").first["audsid"]
       sid_serial = @sys_conn.select("SELECT s.sid||','||s.serial# sid_serial
           FROM   v$session s
-          WHERE  audsid = '#{audsid}'").first['sid_serial']
+          WHERE  audsid = '#{audsid}'").first["sid_serial"]
       @sys_conn.exec "ALTER SYSTEM KILL SESSION '#{sid_serial}' IMMEDIATE"
     end
 
@@ -342,7 +340,7 @@ describe "OracleEnhancedConnection" do
       # @conn.auto_retry = false
       ActiveRecord::Base.connection.auto_retry = false
       kill_current_session
-      if defined?(RUBY_ENGINE) && RUBY_ENGINE == 'jruby'
+      if defined?(RUBY_ENGINE) && RUBY_ENGINE == "jruby"
         expect { @conn.exec("SELECT * FROM dual") }.to raise_error(NativeException)
       else
         expect { @conn.exec("SELECT * FROM dual") }.to raise_error(OCIError)
@@ -353,14 +351,14 @@ describe "OracleEnhancedConnection" do
       # @conn.auto_retry = true
       ActiveRecord::Base.connection.auto_retry = true
       kill_current_session
-      expect(@conn.select("SELECT * FROM dual")).to eq([{'dummy' => 'X'}])
+      expect(@conn.select("SELECT * FROM dual")).to eq([{ "dummy" => "X" }])
     end
 
     it "should not reconnect and execute SQL select if connection is lost and auto retry is disabled" do
       # @conn.auto_retry = false
       ActiveRecord::Base.connection.auto_retry = false
       kill_current_session
-      if defined?(RUBY_ENGINE) && RUBY_ENGINE == 'jruby'
+      if defined?(RUBY_ENGINE) && RUBY_ENGINE == "jruby"
         expect { @conn.select("SELECT * FROM dual") }.to raise_error(NativeException)
       else
         expect { @conn.select("SELECT * FROM dual") }.to raise_error(OCIError)

data/spec/active_record/connection_adapters/oracle_enhanced_context_index_spec.rb

@@ -1,5 +1,3 @@
-require 'spec_helper'
-
 describe "OracleEnhancedAdapter context index" do
   include SchemaSpecHelper
   include LoggerSpecHelper
@@ -68,8 +66,8 @@ describe "OracleEnhancedAdapter context index" do
         has_context_index
       end
       @post0 = Post.create(title: "dummy title", body: "dummy body")
-      @post1 = Post.create(title: @title_words.join(' '), body: @body_words.join(' '))
-      @post2 = Post.create(title: (@title_words*2).join(' '), body: (@body_words*2).join(' '))
+      @post1 = Post.create(title: @title_words.join(" "), body: @body_words.join(" "))
+      @post2 = Post.create(title: (@title_words * 2).join(" "), body: (@body_words * 2).join(" "))
       @post_with_null_body = Post.create(title: "withnull", body: nil)
       @post_with_null_title = Post.create(title: nil, body: "withnull")
     end
@@ -77,7 +75,11 @@ describe "OracleEnhancedAdapter context index" do
     after(:all) do
       drop_table_posts
       Object.send(:remove_const, "Post")
-      ActiveRecord::Base.clear_cache! if ActiveRecord::Base.respond_to?(:"clear_cache!")
+      ActiveRecord::Base.clear_cache!
+    end
+
+    before(:each) do
+      @post = nil
     end
 
     after(:each) do
@@ -102,13 +104,13 @@ describe "OracleEnhancedAdapter context index" do
 
     it "should not include text index secondary tables in user tables list" do
       @conn.add_context_index :posts, :title
-      expect(@conn.tables.any?{|t| t =~ /^dr\$/i}).to be_falsey
+      expect(@conn.tables.any? { |t| t =~ /^dr\$/i }).to be_falsey
       @conn.remove_context_index :posts, :title
     end
 
     it "should create multiple column index" do
       @conn.add_context_index :posts, [:title, :body]
-      (@title_words+@body_words).each do |word|
+      (@title_words + @body_words).each do |word|
         expect(Post.contains(:title, word).to_a).to eq([@post2, @post1])
       end
       @conn.remove_context_index :posts, [:title, :body]
@@ -122,7 +124,7 @@ describe "OracleEnhancedAdapter context index" do
 
     it "should create multiple column index with specified main index column" do
       @conn.add_context_index :posts, [:title, :body],
-        index_column: :all_text, sync: 'ON COMMIT'
+        index_column: :all_text, sync: "ON COMMIT"
       @post = Post.create(title: "abc", body: "def")
       expect(Post.contains(:all_text, "abc").to_a).to eq([@post])
       expect(Post.contains(:all_text, "def").to_a).to eq([@post])
@@ -138,7 +140,7 @@ describe "OracleEnhancedAdapter context index" do
     it "should create multiple column index with trigger updated main index column" do
       @conn.add_context_index :posts, [:title, :body],
         index_column: :all_text, index_column_trigger_on: [:created_at, :updated_at],
-        sync: 'ON COMMIT'
+        sync: "ON COMMIT"
       @post = Post.create(title: "abc", body: "def")
       expect(Post.contains(:all_text, "abc").to_a).to eq([@post])
       expect(Post.contains(:all_text, "def").to_a).to eq([@post])
@@ -151,7 +153,7 @@ describe "OracleEnhancedAdapter context index" do
     it "should use base letter conversion with BASIC_LEXER" do
       @post = Post.create!(title: "āčē", body: "dummy")
       @conn.add_context_index :posts, :title,
-        lexer: { type: "BASIC_LEXER", base_letter_type: 'GENERIC', base_letter: true }
+        lexer: { type: "BASIC_LEXER", base_letter_type: "GENERIC", base_letter: true }
       expect(Post.contains(:title, "āčē").to_a).to eq([@post])
       expect(Post.contains(:title, "ace").to_a).to eq([@post])
       expect(Post.contains(:title, "ACE").to_a).to eq([@post])
@@ -172,7 +174,7 @@ describe "OracleEnhancedAdapter context index" do
     it "should use index when contains has schema_name.table_name syntax" do
       @conn.add_context_index :posts, :title
       @title_words.each do |word|
-        Post.contains('posts.title', word).to_a.should == [@post2, @post1]
+        expect(Post.contains("posts.title", word).to_a).to eq([@post2, @post1])
       end
       @conn.remove_context_index :posts, :title
     end
@@ -195,11 +197,11 @@ describe "OracleEnhancedAdapter context index" do
       drop_tables
       Object.send(:remove_const, "Comment")
       Object.send(:remove_const, "Post")
-      ActiveRecord::Base.clear_cache! if ActiveRecord::Base.respond_to?(:"clear_cache!")
+      ActiveRecord::Base.clear_cache!
     end
 
     after(:each) do
-      @conn.remove_context_index :posts, name: 'post_and_comments_index' rescue nil
+      @conn.remove_context_index :posts, name: "post_and_comments_index" rescue nil
       @conn.remove_context_index :posts, index_column: :all_text rescue nil
       Post.destroy_all
     end
@@ -210,9 +212,9 @@ describe "OracleEnhancedAdapter context index" do
         # specify aliases always with AS keyword
         "SELECT comments.author AS comment_author, comments.body AS comment_body FROM comments WHERE comments.post_id = :id"
         ],
-        name: 'post_and_comments_index',
+        name: "post_and_comments_index",
         index_column: :all_text, index_column_trigger_on: [:updated_at, :comments_count],
-        sync: 'ON COMMIT'
+        sync: "ON COMMIT"
       @post = Post.create!(title: "aaa", body: "bbb")
       @post.comments.create!(author: "ccc", body: "ddd")
       @post.comments.create!(author: "eee", body: "fff")
@@ -231,9 +233,9 @@ describe "OracleEnhancedAdapter context index" do
             FROM comments
             WHERE comments.post_id = :id }
         ],
-        name: 'post_and_comments_index',
+        name: "post_and_comments_index",
         index_column: :all_text, index_column_trigger_on: [:updated_at, :comments_count],
-        sync: 'ON COMMIT'
+        sync: "ON COMMIT"
       @post = Post.create!(title: "aaa", body: "bbb")
       @post.comments.create!(author: "ccc", body: "ddd")
       @post.comments.create!(author: "eee", body: "fff")
@@ -270,7 +272,7 @@ describe "OracleEnhancedAdapter context index" do
       class ::Post < ActiveRecord::Base
         has_context_index
       end
-      @post = Post.create(title: 'aaa', body: 'bbb')
+      @post = Post.create(title: "aaa", body: "bbb")
       @tablespace = @conn.default_tablespace
       set_logger
       @conn = ActiveRecord::Base.connection
@@ -279,7 +281,7 @@ describe "OracleEnhancedAdapter context index" do
     after(:all) do
       drop_table_posts
       Object.send(:remove_const, "Post")
-      ActiveRecord::Base.clear_cache! if ActiveRecord::Base.respond_to?(:"clear_cache!")
+      ActiveRecord::Base.clear_cache!
     end
 
     after(:each) do
@@ -287,7 +289,7 @@ describe "OracleEnhancedAdapter context index" do
     end
 
     def verify_logged_statements
-      ['K_TABLE_CLAUSE', 'R_TABLE_CLAUSE', 'N_TABLE_CLAUSE', 'I_INDEX_CLAUSE', 'P_TABLE_CLAUSE'].each do |clause|
+      ["K_TABLE_CLAUSE", "R_TABLE_CLAUSE", "N_TABLE_CLAUSE", "I_INDEX_CLAUSE", "P_TABLE_CLAUSE"].each do |clause|
         expect(@logger.output(:debug)).to match(/CTX_DDL\.SET_ATTRIBUTE\('index_posts_on_title_sto', '#{clause}', '.*TABLESPACE #{@tablespace}'\)/)
       end
       expect(@logger.output(:debug)).to match(/CREATE INDEX .* PARAMETERS \('STORAGE index_posts_on_title_sto'\)/)
@@ -296,15 +298,15 @@ describe "OracleEnhancedAdapter context index" do
     it "should create index on single column" do
       @conn.add_context_index :posts, :title, tablespace: @tablespace
       verify_logged_statements
-      expect(Post.contains(:title, 'aaa').to_a).to eq([@post])
+      expect(Post.contains(:title, "aaa").to_a).to eq([@post])
       @conn.remove_context_index :posts, :title
     end
 
     it "should create index on multiple columns" do
-      @conn.add_context_index :posts, [:title, :body], name: 'index_posts_text', tablespace: @conn.default_tablespace
+      @conn.add_context_index :posts, [:title, :body], name: "index_posts_text", tablespace: @conn.default_tablespace
       verify_logged_statements
-      expect(Post.contains(:title, 'aaa AND bbb').to_a).to eq([@post])
-      @conn.remove_context_index :posts, name: 'index_posts_text'
+      expect(Post.contains(:title, "aaa AND bbb").to_a).to eq([@post])
+      @conn.remove_context_index :posts, name: "index_posts_text"
     end
 
   end
@@ -343,56 +345,56 @@ describe "OracleEnhancedAdapter context index" do
 
       it "should dump definition of multiple table index with options" do
         options = {
-          name: 'post_and_comments_index',
+          name: "post_and_comments_index",
           index_column: :all_text, index_column_trigger_on: :updated_at,
           transactional: true,
-          sync: 'ON COMMIT'
+          sync: "ON COMMIT"
         }
         sub_query = "SELECT comments.author AS comment_author, comments.body AS comment_body FROM comments WHERE comments.post_id = :id"
         @conn.add_context_index :posts, [:title, :body, sub_query], options
         expect(standard_dump).to match(/add_context_index "posts", \[:title, :body, "#{sub_query}"\], #{options.inspect[1..-2]}$/)
-        @conn.remove_context_index :posts, name: 'post_and_comments_index'
+        @conn.remove_context_index :posts, name: "post_and_comments_index"
       end
 
       it "should dump definition of multiple table index with options (when definition is larger than 4000 bytes)" do
         options = {
-          name: 'post_and_comments_index',
+          name: "post_and_comments_index",
           index_column: :all_text, index_column_trigger_on: :updated_at,
           transactional: true,
-          sync: 'ON COMMIT'
+          sync: "ON COMMIT"
         }
         sub_query = "SELECT comments.author AS comment_author, comments.body AS comment_body FROM comments WHERE comments.post_id = :id#{' AND 1=1' * 500}"
         @conn.add_context_index :posts, [:title, :body, sub_query], options
         expect(standard_dump).to match(/add_context_index "posts", \[:title, :body, "#{sub_query}"\], #{options.inspect[1..-2]}$/)
-        @conn.remove_context_index :posts, name: 'post_and_comments_index'
+        @conn.remove_context_index :posts, name: "post_and_comments_index"
       end
 
       it "should dump definition of multiple table index with options (when subquery has newlines)" do
         options = {
-          name: 'post_and_comments_index',
+          name: "post_and_comments_index",
           index_column: :all_text, index_column_trigger_on: :updated_at,
           transactional: true,
-          sync: 'ON COMMIT'
+          sync: "ON COMMIT"
         }
         sub_query = "SELECT comments.author AS comment_author, comments.body AS comment_body\nFROM comments\nWHERE comments.post_id = :id"
         @conn.add_context_index :posts, [:title, :body, sub_query], options
         expect(standard_dump).to match(/add_context_index "posts", \[:title, :body, "#{sub_query.gsub(/\n/, ' ')}"\], #{options.inspect[1..-2]}$/)
-        @conn.remove_context_index :posts, name: 'post_and_comments_index'
+        @conn.remove_context_index :posts, name: "post_and_comments_index"
       end
 
     end
 
     describe "with table prefix and suffix" do
       before(:all) do
-        ActiveRecord::Base.table_name_prefix = 'xxx_'
-        ActiveRecord::Base.table_name_suffix = '_xxx'
+        ActiveRecord::Base.table_name_prefix = "xxx_"
+        ActiveRecord::Base.table_name_suffix = "_xxx"
         create_tables
       end
 
       after(:all) do
         drop_tables
-        ActiveRecord::Base.table_name_prefix = ''
-        ActiveRecord::Base.table_name_suffix = ''
+        ActiveRecord::Base.table_name_prefix = ""
+        ActiveRecord::Base.table_name_suffix = ""
       end
 
       it "should dump definition of single column index" do
@@ -409,11 +411,11 @@ describe "OracleEnhancedAdapter context index" do
 
       it "should dump definition of multiple table index with options" do
         options = {
-          name: 'xxx_post_and_comments_i',
+          name: "xxx_post_and_comments_i",
           index_column: :all_text, index_column_trigger_on: :updated_at,
-          lexer: { type: "BASIC_LEXER", base_letter_type: 'GENERIC', base_letter: true },
+          lexer: { type: "BASIC_LEXER", base_letter_type: "GENERIC", base_letter: true },
           wordlist: { type: "BASIC_WORDLIST", prefix_index: true },
-          sync: 'ON COMMIT'
+          sync: "ON COMMIT"
         }
         schema_define do
           add_context_index :posts,
@@ -422,8 +424,8 @@ describe "OracleEnhancedAdapter context index" do
             ], options
         end
         expect(standard_dump).to match(/add_context_index "posts", \[:title, :body, "SELECT comments.author AS comment_author, comments.body AS comment_body FROM comments WHERE comments.post_id = :id"\], #{
-          options.inspect[1..-2].gsub(/[{}]/){|s| '\\'<<s }}$/)
-        schema_define { remove_context_index :posts, name: 'xxx_post_and_comments_i' }
+          options.inspect[1..-2].gsub(/[{}]/) { |s| '\\' << s }}$/)
+        schema_define { remove_context_index :posts, name: "xxx_post_and_comments_i" }
       end
 
     end