ruby-plsql 0.6.0 → 0.7.0

data/spec/plsql/variable_spec.rb

@@ -1,6 +1,6 @@
 # encoding: utf-8
 
-require 'spec_helper'
+require "spec_helper"
 
 describe "Package variables /" do
 
@@ -35,24 +35,24 @@ describe "Package variables /" do
     end
 
     it "should set and get VARCHAR variable" do
-      plsql.test_package.varchar_variable = 'abc'
-      expect(plsql.test_package.varchar_variable).to eq('abc')
+      plsql.test_package.varchar_variable = "abc"
+      expect(plsql.test_package.varchar_variable).to eq("abc")
     end
 
     it "should set and get VARCHAR2 variable" do
-      plsql.test_package.varchar2_variable = 'abc'
-      expect(plsql.test_package.varchar2_variable).to eq('abc')
+      plsql.test_package.varchar2_variable = "abc"
+      expect(plsql.test_package.varchar2_variable).to eq("abc")
     end
 
     it "should set and get VARCHAR2 variable with comment" do
-      plsql.test_package.varchar2_variable2 = 'abc'
-      expect(plsql.test_package.varchar2_variable2).to eq('abc')
+      plsql.test_package.varchar2_variable2 = "abc"
+      expect(plsql.test_package.varchar2_variable2).to eq("abc")
     end
 
     it "should get VARCHAR2 variable default value" do
-      expect(plsql.test_package.varchar2_default).to eq('default')
-      expect(plsql.test_package.varchar2_default2).to eq('default')
-      expect(plsql.test_package.varchar2_default3).to eq('default')
+      expect(plsql.test_package.varchar2_default).to eq("default")
+      expect(plsql.test_package.varchar2_default2).to eq("default")
+      expect(plsql.test_package.varchar2_default3).to eq("default")
     end
 
     describe "with character or byte limit" do
@@ -70,33 +70,33 @@ describe "Package variables /" do
       end
 
       it "should set and get VARCHAR2(n CHAR) variable" do
-        plsql.test_package.varchar2_3_char = 'āčē'
-        expect(plsql.test_package.varchar2_3_char).to eq('āčē')
-        expect { plsql.test_package.varchar2_3_char = 'aceg' }.to raise_error(/buffer too small/)
+        plsql.test_package.varchar2_3_char = "āčē"
+        expect(plsql.test_package.varchar2_3_char).to eq("āčē")
+        expect { plsql.test_package.varchar2_3_char = "aceg" }.to raise_error(/buffer too small/)
       end
 
       it "should set and get VARCHAR2(n BYTE) variable" do
-        plsql.test_package.varchar2_3_byte = 'ace'
-        expect(plsql.test_package.varchar2_3_byte).to eq('ace')
-        expect { plsql.test_package.varchar2_3_byte = 'āce' }.to raise_error(/buffer too small/)
-        expect { plsql.test_package.varchar2_3_byte = 'aceg' }.to raise_error(/buffer too small/)
+        plsql.test_package.varchar2_3_byte = "ace"
+        expect(plsql.test_package.varchar2_3_byte).to eq("ace")
+        expect { plsql.test_package.varchar2_3_byte = "āce" }.to raise_error(/buffer too small/)
+        expect { plsql.test_package.varchar2_3_byte = "aceg" }.to raise_error(/buffer too small/)
       end
 
     end
 
     it "should set and get CHAR variable" do
-      plsql.test_package.char_variable = 'abc'
-      expect(plsql.test_package.char_variable).to eq('abc' + ' '*7)
+      plsql.test_package.char_variable = "abc"
+      expect(plsql.test_package.char_variable).to eq("abc" + " " * 7)
     end
 
     it "should set and get NVARCHAR2 variable" do
-      plsql.test_package.nvarchar2_variable = 'abc'
-      expect(plsql.test_package.nvarchar2_variable).to eq('abc')
+      plsql.test_package.nvarchar2_variable = "abc"
+      expect(plsql.test_package.nvarchar2_variable).to eq("abc")
     end
 
     it "should set and get NCHAR variable" do
-      plsql.test_package.nchar_variable = 'abc'
-      expect(plsql.test_package.nchar_variable).to eq('abc' + ' '*7)
+      plsql.test_package.nchar_variable = "abc"
+      expect(plsql.test_package.nchar_variable).to eq("abc" + " " * 7)
     end
 
   end
@@ -134,22 +134,22 @@ describe "Package variables /" do
   end
 
   [
-      {:ora_data_type => 'INTEGER',        :default => nil, :class => Fixnum, :given => 1, :expected => 1},
-      {:ora_data_type => 'NUMBER(10)',     :default => nil, :class => Fixnum, :given => 1, :expected => 1},
-      {:ora_data_type => 'NUMBER(10)',     :default => 5,   :class => Fixnum, :given => 1, :expected => 1},
-      {:ora_data_type => 'NUMBER',         :default => nil, :class => BigDecimal, :given => 123.456, :expected => 123.456},
-      {:ora_data_type => 'NUMBER(15,2)',   :default => nil, :class => BigDecimal, :given => 123.456, :expected => 123.46},
-      {:ora_data_type => 'PLS_INTEGER',    :default => nil, :class => Fixnum, :given => 1, :expected => 1},
-      {:ora_data_type => 'BINARY_INTEGER', :default => nil, :class => Fixnum, :given => 1, :expected => 1},
-      {:ora_data_type => 'SIMPLE_INTEGER', :default => 10,  :class => Fixnum, :given => 1, :expected => 1},
-      {:ora_data_type => 'NATURAL',        :default => nil, :class => Fixnum, :given => 1, :expected => 1},
-      {:ora_data_type => 'NATURALN',       :default => 0,   :class => Fixnum, :given => 1, :expected => 1},
-      {:ora_data_type => 'POSITIVE',       :default => nil, :class => Fixnum, :given => 1, :expected => 1},
-      {:ora_data_type => 'POSITIVEN',      :default => 5,   :class => Fixnum, :given => 1, :expected => 1},
-      {:ora_data_type => 'SIGNTYPE',       :default => -1,  :class => Fixnum, :given => 1, :expected => 1},
+      { ora_data_type: "INTEGER",        default: nil, class: Integer, given: 1, expected: 1 },
+      { ora_data_type: "NUMBER(10)",     default: nil, class: Integer, given: 1, expected: 1 },
+      { ora_data_type: "NUMBER(10)",     default: 5,   class: Integer, given: 1, expected: 1 },
+      { ora_data_type: "NUMBER",         default: nil, class: BigDecimal, given: 123.456, expected: 123.456 },
+      { ora_data_type: "NUMBER(15,2)",   default: nil, class: BigDecimal, given: 123.456, expected: 123.46 },
+      { ora_data_type: "PLS_INTEGER",    default: nil, class: Integer, given: 1, expected: 1 },
+      { ora_data_type: "BINARY_INTEGER", default: nil, class: Integer, given: 1, expected: 1 },
+      { ora_data_type: "SIMPLE_INTEGER", default: 10,  class: Integer, given: 1, expected: 1 },
+      { ora_data_type: "NATURAL",        default: nil, class: Integer, given: 1, expected: 1 },
+      { ora_data_type: "NATURALN",       default: 0,   class: Integer, given: 1, expected: 1 },
+      { ora_data_type: "POSITIVE",       default: nil, class: Integer, given: 1, expected: 1 },
+      { ora_data_type: "POSITIVEN",      default: 5,   class: Integer, given: 1, expected: 1 },
+      { ora_data_type: "SIGNTYPE",       default: -1,  class: Integer, given: 1, expected: 1 },
   ].each do |row|
     ora_data_type, default, class_, given, expected = row.values
-    describe ora_data_type+(default ? ' with default' : '') do
+    describe ora_data_type + (default ? " with default" : "") do
       include_examples "Numeric", ora_data_type, default, class_, given, expected
     end
   end
@@ -170,8 +170,8 @@ describe "Package variables /" do
         CREATE OR REPLACE PACKAGE BODY test_package IS
         END;
       SQL
-      @date = Time.local(2009,12,21)
-      @timestamp = Time.local(2009,12,21,14,10,30,11)
+      @date = Time.local(2009, 12, 21)
+      @timestamp = Time.local(2009, 12, 21, 14, 10, 30, 11)
     end
 
     after(:all) do
@@ -233,17 +233,17 @@ describe "Package variables /" do
     end
 
     it "should set and get CLOB variable" do
-      plsql.test_package.clob_variable = 'abc'
-      expect(plsql.test_package.clob_variable).to eq('abc')
+      plsql.test_package.clob_variable = "abc"
+      expect(plsql.test_package.clob_variable).to eq("abc")
     end
 
     it "should get CLOB variable default value" do
-      expect(plsql.test_package.clob_default).to eq('default')
+      expect(plsql.test_package.clob_default).to eq("default")
     end
 
     it "should set and get NCLOB variable" do
-      plsql.test_package.nclob_variable = 'abc'
-      expect(plsql.test_package.nclob_variable).to eq('abc')
+      plsql.test_package.nclob_variable = "abc"
+      expect(plsql.test_package.nclob_variable).to eq("abc")
     end
 
     it "should set and get BLOB variable" do
@@ -291,12 +291,12 @@ describe "Package variables /" do
     end
 
     it "should set and get VARCHAR2 variable" do
-      plsql.test_package.first_name = 'First'
-      expect(plsql.test_package.first_name).to eq('First')
+      plsql.test_package.first_name = "First"
+      expect(plsql.test_package.first_name).to eq("First")
     end
 
     it "should set and get DATE variable" do
-      today = Time.local(2009,12,22)
+      today = Time.local(2009, 12, 22)
       plsql.test_package.hire_date = today
       expect(plsql.test_package.hire_date).to eq(today)
     end
@@ -329,7 +329,7 @@ describe "Package variables /" do
     end
 
     it "should get VARCHAR2 constant" do
-      expect(plsql.test_package.string_constant).to eq('constant')
+      expect(plsql.test_package.string_constant).to eq("constant")
     end
 
     it "should raise error when trying to set constant" do
@@ -371,14 +371,14 @@ describe "Package variables /" do
           phones        t_phones
         )
       SQL
-      @phones = [{:type => 'mobile', :phone_number => '123456'}, {:type => 'home', :phone_number => '654321'}]
+      @phones = [{ type: "mobile", phone_number: "123456" }, { type: "home", phone_number: "654321" }]
       @employee = {
-        :employee_id => 1,
-        :first_name => 'First',
-        :last_name => 'Last',
-        :hire_date => Time.local(2000,01,31),
-        :address => {:street => 'Main street 1', :city => 'Riga', :country => 'Latvia'},
-        :phones => @phones
+        employee_id: 1,
+        first_name: "First",
+        last_name: "Last",
+        hire_date: Time.local(2000, 01, 31),
+        address: { street: "Main street 1", city: "Riga", country: "Latvia" },
+        phones: @phones
       }
 
       plsql.execute <<-SQL
@@ -433,10 +433,10 @@ describe "Package variables /" do
         )
       SQL
       @employee = {
-        :employee_id => 1,
-        :first_name => 'First',
-        :last_name => 'Last',
-        :hire_date => Time.local(2000,01,31)
+        employee_id: 1,
+        first_name: "First",
+        last_name: "Last",
+        hire_date: Time.local(2000, 01, 31)
       }
 
       plsql.execute <<-SQL

data/spec/plsql/version_spec.rb

@@ -1,8 +1,8 @@
-require 'spec_helper'
+require "spec_helper"
 
 describe "Version" do
   it "should return ruby-plsql version" do
-    expect(PLSQL::VERSION).to eq(File.read(File.dirname(__FILE__)+'/../../VERSION').chomp)
+    expect(PLSQL::VERSION).to eq(File.read(File.dirname(__FILE__) + "/../../VERSION").chomp)
   end
-  
-end
+
+end

data/spec/plsql/view_spec.rb

@@ -1,4 +1,4 @@
-require 'spec_helper'
+require "spec_helper"
 
 describe "View" do
   before(:all) do
@@ -17,18 +17,18 @@ describe "View" do
 
     @employees = (1..10).map do |i|
       {
-        :employee_id => i,
-        :first_name => "First #{i}",
-        :last_name => "Last #{i}",
-        :hire_date => Time.local(2000,01,i),
-        :status => 'A'
+        employee_id: i,
+        first_name: "First #{i}",
+        last_name: "Last #{i}",
+        hire_date: Time.local(2000, 01, i),
+        status: "A"
       }
     end
     @employees_all_fields = [:employee_id, :first_name, :last_name, :hire_date, :status]
-    @employees_all_values = @employees.map{|e| @employees_all_fields.map{|f| e[f]}}
+    @employees_all_values = @employees.map { |e| @employees_all_fields.map { |f| e[f] } }
     @employees_some_fields = [:employee_id, :first_name, :last_name]
-    @employees_some_values = @employees.map{|e| @employees_some_fields.map{|f| e[f]}}
-    @employee_default_values = {:hire_date => nil, :status => 'N'}
+    @employees_some_values = @employees.map { |e| @employees_some_fields.map { |f| e[f] } }
+    @employee_default_values = { hire_date: nil, status: "N" }
   end
 
   after(:all) do
@@ -96,23 +96,23 @@ describe "View" do
     end
 
     it "should get columns metadata for view" do
-      expect(plsql.test_employees_v.columns).to eq({
-        :employee_id => {
-          :position=>1, :data_type=>"NUMBER", :data_length=>22, :data_precision=>15, :data_scale=>0, :char_used=>nil,
-          :type_owner=>nil, :type_name=>nil, :sql_type_name=>nil, :nullable => false, :data_default => nil},
-        :first_name => {
-          :position=>2, :data_type=>"VARCHAR2", :data_length=>50, :data_precision=>nil, :data_scale=>nil, :char_used=>"B",
-          :type_owner=>nil, :type_name=>nil, :sql_type_name=>nil, :nullable => true, :data_default => nil},
-        :last_name => {
-          :position=>3, :data_type=>"VARCHAR2", :data_length=>50, :data_precision=>nil, :data_scale=>nil, :char_used=>"B",
-          :type_owner=>nil, :type_name=>nil, :sql_type_name=>nil, :nullable => true, :data_default => nil},
-        :hire_date => {
-          :position=>4, :data_type=>"DATE", :data_length=>7, :data_precision=>nil, :data_scale=>nil, :char_used=>nil,
-          :type_owner=>nil, :type_name=>nil, :sql_type_name=>nil, :nullable => true, :data_default => nil},
-        :status => {
-          :position=>5, :data_type=>"VARCHAR2", :data_length=>1, :data_precision=>nil, :data_scale=>nil, :char_used=>"B",
-          :type_owner=>nil, :type_name=>nil, :sql_type_name=>nil, :nullable => true, :data_default => nil}
-      })
+      expect(plsql.test_employees_v.columns).to eq(
+        employee_id: {
+          position: 1, data_type: "NUMBER", data_length: 22, data_precision: 15, data_scale: 0, char_used: nil,
+          type_owner: nil, type_name: nil, sql_type_name: nil, nullable: false, data_default: nil },
+        first_name: {
+          position: 2, data_type: "VARCHAR2", data_length: 50, data_precision: nil, data_scale: nil, char_used: "B",
+          type_owner: nil, type_name: nil, sql_type_name: nil, nullable: true, data_default: nil },
+        last_name: {
+          position: 3, data_type: "VARCHAR2", data_length: 50, data_precision: nil, data_scale: nil, char_used: "B",
+          type_owner: nil, type_name: nil, sql_type_name: nil, nullable: true, data_default: nil },
+        hire_date: {
+          position: 4, data_type: "DATE", data_length: 7, data_precision: nil, data_scale: nil, char_used: nil,
+          type_owner: nil, type_name: nil, sql_type_name: nil, nullable: true, data_default: nil },
+        status: {
+          position: 5, data_type: "VARCHAR2", data_length: 1, data_precision: nil, data_scale: nil, char_used: "B",
+          type_owner: nil, type_name: nil, sql_type_name: nil, nullable: true, data_default: nil }
+      )
     end
 
   end
@@ -125,12 +125,12 @@ describe "View" do
 
     it "should insert a record in view using partial list of columns" do
       plsql.test_employees_v.insert @employees.first.except(:hire_date)
-      expect(plsql.test_employees_v.all).to eq([@employees.first.merge(:hire_date => nil)])
+      expect(plsql.test_employees_v.all).to eq([@employees.first.merge(hire_date: nil)])
     end
 
     it "should insert default value from table definition if value not provided" do
       plsql.test_employees_v.insert @employees.first.except(:status)
-      expect(plsql.test_employees_v.all).to eq([@employees.first.merge(:status => 'N')])
+      expect(plsql.test_employees_v.all).to eq([@employees.first.merge(status: "N")])
     end
 
     it "should insert array of records in view" do
@@ -168,7 +168,7 @@ describe "View" do
 
     it "should insert many records with list of some fields and array of values" do
       plsql.test_employees_v.insert_values @employees_some_fields, *@employees_some_values
-      expect(plsql.test_employees_v.all).to eq(@employees.map{|e| e.merge(@employee_default_values)})
+      expect(plsql.test_employees_v.all).to eq(@employees.map { |e| e.merge(@employee_default_values) })
     end
 
   end
@@ -186,13 +186,13 @@ describe "View" do
     it "should select all records in view" do
       expect(plsql.test_employees_v.select(:all, "ORDER BY employee_id")).to eq(@employees)
       expect(plsql.test_employees_v.all("ORDER BY employee_id")).to eq(@employees)
-      expect(plsql.test_employees_v.all(:order_by => :employee_id)).to eq(@employees)
+      expect(plsql.test_employees_v.all(order_by: :employee_id)).to eq(@employees)
     end
 
     it "should select record in view using WHERE condition" do
       expect(plsql.test_employees_v.select(:first, "WHERE employee_id = :1", @employees.first[:employee_id])).to eq(@employees.first)
       expect(plsql.test_employees_v.first("WHERE employee_id = :1", @employees.first[:employee_id])).to eq(@employees.first)
-      expect(plsql.test_employees_v.first(:employee_id => @employees.first[:employee_id])).to eq(@employees.first)
+      expect(plsql.test_employees_v.first(employee_id: @employees.first[:employee_id])).to eq(@employees.first)
     end
 
     it "should select record in view using :column => nil condition" do
@@ -201,7 +201,7 @@ describe "View" do
       employee[:hire_date] = nil
       plsql.test_employees_v.insert employee
       expect(plsql.test_employees_v.first("WHERE hire_date IS NULL")).to eq(employee)
-      expect(plsql.test_employees_v.first(:hire_date => nil)).to eq(employee)
+      expect(plsql.test_employees_v.first(hire_date: nil)).to eq(employee)
     end
 
     it "should count records in view" do
@@ -220,26 +220,26 @@ describe "View" do
     it "should update a record in view" do
       employee_id = @employees.first[:employee_id]
       plsql.test_employees_v.insert @employees.first
-      plsql.test_employees_v.update :first_name => 'Test', :where => {:employee_id => employee_id}
-      expect(plsql.test_employees_v.first(:employee_id => employee_id)[:first_name]).to eq('Test')
+      plsql.test_employees_v.update first_name: "Test", where: { employee_id: employee_id }
+      expect(plsql.test_employees_v.first(employee_id: employee_id)[:first_name]).to eq("Test")
     end
 
     it "should update a record in view using String WHERE condition" do
       employee_id = @employees.first[:employee_id]
       plsql.test_employees_v.insert @employees
-      plsql.test_employees_v.update :first_name => 'Test', :where => "employee_id = #{employee_id}"
-      expect(plsql.test_employees_v.first(:employee_id => employee_id)[:first_name]).to eq('Test')
+      plsql.test_employees_v.update first_name: "Test", where: "employee_id = #{employee_id}"
+      expect(plsql.test_employees_v.first(employee_id: employee_id)[:first_name]).to eq("Test")
       # all other records should not be changed
       plsql.test_employees_v.all("WHERE employee_id > :1", employee_id) do |employee|
-        expect(employee[:first_name]).not_to eq('Test')
+        expect(employee[:first_name]).not_to eq("Test")
       end
     end
 
     it "should update all records in view" do
       plsql.test_employees_v.insert @employees
-      plsql.test_employees_v.update :first_name => 'Test'
+      plsql.test_employees_v.update first_name: "Test"
       plsql.test_employees_v.all do |employee|
-        expect(employee[:first_name]).to eq('Test')
+        expect(employee[:first_name]).to eq("Test")
       end
     end
 
@@ -249,9 +249,9 @@ describe "View" do
     it "should delete record from view" do
       employee_id = @employees.first[:employee_id]
       plsql.test_employees_v.insert @employees
-      plsql.test_employees_v.delete :employee_id => employee_id
-      expect(plsql.test_employees_v.first(:employee_id => employee_id)).to be_nil
-      expect(plsql.test_employees_v.all(:order_by => :employee_id)).to eq(@employees[1, @employees.size-1])
+      plsql.test_employees_v.delete employee_id: employee_id
+      expect(plsql.test_employees_v.first(employee_id: employee_id)).to be_nil
+      expect(plsql.test_employees_v.all(order_by: :employee_id)).to eq(@employees[1, @employees.size - 1])
     end
 
     it "should delete all records from view" do

data/spec/spec_helper.rb

@@ -1,50 +1,50 @@
 require "rubygems"
 require "bundler"
 Bundler.setup(:default, :development)
-require 'simplecov'
+require "simplecov"
 
 SimpleCov.configure do
-  load_profile 'root_filter'
-  load_profile 'test_frameworks'
+  load_profile "root_filter"
+  load_profile "test_frameworks"
 end
 
 ENV["COVERAGE"] && SimpleCov.start do
   add_filter "/.rvm/"
 end
 
-$LOAD_PATH.unshift(File.join(File.dirname(__FILE__), '..', 'lib'))
-require 'rspec'
+$LOAD_PATH.unshift(File.join(File.dirname(__FILE__), "..", "lib"))
+require "rspec"
 
-unless ENV['NO_ACTIVERECORD']
-  require 'active_record'
+unless ENV["NO_ACTIVERECORD"]
+  require "active_record"
 else
-  puts 'Without ActiveRecord'
+  puts "Without ActiveRecord"
 end
 
-require 'ruby-plsql'
+require "ruby-plsql"
 
 # Requires supporting ruby files with custom matchers and macros, etc,
 # in spec/support/ and its subdirectories.
-Dir[File.join(File.dirname(__FILE__), 'support/**/*.rb')].each {|f| require f}
+Dir[File.join(File.dirname(__FILE__), "support/**/*.rb")].each { |f| require f }
 
-if ENV['USE_VM_DATABASE'] == 'Y'
-  DATABASE_NAME = 'XE'
+if ENV["USE_VM_DATABASE"] == "Y"
+  DATABASE_NAME = "XE"
 else
-  DATABASE_NAME = ENV['DATABASE_NAME'] || 'orcl'
+  DATABASE_NAME = ENV["DATABASE_NAME"] || "orcl"
 end
 
 DATABASE_SERVICE_NAME = (defined?(JRUBY_VERSION) ? "/" : "") +
-                        (ENV['DATABASE_SERVICE_NAME'] || DATABASE_NAME)
-DATABASE_HOST = ENV['DATABASE_HOST'] || 'localhost'
-DATABASE_PORT = (ENV['DATABASE_PORT'] || 1521).to_i
+                        (ENV["DATABASE_SERVICE_NAME"] || DATABASE_NAME)
+DATABASE_HOST = ENV["DATABASE_HOST"] || "localhost"
+DATABASE_PORT = (ENV["DATABASE_PORT"] || 1521).to_i
 DATABASE_USERS_AND_PASSWORDS = [
-  [ENV['DATABASE_USER'] || 'hr', ENV['DATABASE_PASSWORD'] || 'hr'],
-  [ENV['DATABASE_USER2'] || 'arunit', ENV['DATABASE_PASSWORD2'] || 'arunit']
+  [ENV["DATABASE_USER"] || "hr", ENV["DATABASE_PASSWORD"] || "hr"],
+  [ENV["DATABASE_USER2"] || "arunit", ENV["DATABASE_PASSWORD2"] || "arunit"]
 ]
 # specify which database version is used (will be verified in one test)
-DATABASE_VERSION = ENV['DATABASE_VERSION'] || '10.2.0.4'
+DATABASE_VERSION = ENV["DATABASE_VERSION"] || "10.2.0.4"
 
-if ENV['USE_VM_DATABASE'] == 'Y'
+if ENV["USE_VM_DATABASE"] == "Y"
   RSpec.configure do |config|
     config.before(:suite) do
       TestDb.build
@@ -58,13 +58,21 @@ if ENV['USE_VM_DATABASE'] == 'Y'
   end
 end
 
+def oracle_error_class
+  unless defined?(JRUBY_VERSION)
+    OCIError
+  else
+    java.sql.SQLException
+  end
+end
+
 def get_eazy_connect_url(svc_separator = "")
   "#{DATABASE_HOST}:#{DATABASE_PORT}#{svc_separator}#{DATABASE_SERVICE_NAME}"
 end
 
 def get_connection_url
   unless defined?(JRUBY_VERSION)
-    (ENV['DATABASE_USE_TNS'] == 'NO') ? get_eazy_connect_url("/") : DATABASE_NAME
+    (ENV["DATABASE_USE_TNS"] == "NO") ? get_eazy_connect_url("/") : DATABASE_NAME
   else
     "jdbc:oracle:thin:@#{get_eazy_connect_url}"
   end
@@ -94,20 +102,20 @@ def try_to_connect(exception)
 end
 
 CONNECTION_PARAMS = {
-  :adapter => "oracle_enhanced",
-  :database => DATABASE_SERVICE_NAME,
-  :host => DATABASE_HOST,
-  :port => DATABASE_PORT,
-  :username => DATABASE_USERS_AND_PASSWORDS[0][0],
-  :password => DATABASE_USERS_AND_PASSWORDS[0][1]
+  adapter: "oracle_enhanced",
+  database: DATABASE_SERVICE_NAME,
+  host: DATABASE_HOST,
+  port: DATABASE_PORT,
+  username: DATABASE_USERS_AND_PASSWORDS[0][0],
+  password: DATABASE_USERS_AND_PASSWORDS[0][1]
 }
 
 class Hash
   def except(*blacklist)
-    self.reject {|key, value| blacklist.include?(key) }
+    self.reject { |key, value| blacklist.include?(key) }
   end unless method_defined?(:except)
 
   def only(*whitelist)
-    self.reject {|key, value| !whitelist.include?(key) }
+    self.reject { |key, value| !whitelist.include?(key) }
   end unless method_defined?(:only)
 end