ruby-plsql 0.6.0 → 0.7.0

data/ruby-plsql.gemspec

@@ -1,19 +1,20 @@
-# Generated by jeweler
+# Generated by juwelier
 # DO NOT EDIT THIS FILE DIRECTLY
-# Instead, edit Jeweler::Tasks in Rakefile, and run 'rake gemspec'
+# Instead, edit Juwelier::Tasks in Rakefile, and run 'rake gemspec'
 # -*- encoding: utf-8 -*-
-# stub: ruby-plsql 0.6.0 ruby lib
+
+# stub: ruby-plsql 0.7.0 ruby lib
 
 Gem::Specification.new do |s|
-  s.name = "ruby-plsql"
-  s.version = "0.6.0"
+  s.name = "ruby-plsql".freeze
+  s.version = "0.7.0"
 
-  s.required_rubygems_version = Gem::Requirement.new(">= 0") if s.respond_to? :required_rubygems_version=
-  s.require_paths = ["lib"]
-  s.authors = ["Raimonds Simanovskis"]
-  s.date = "2016-03-13"
-  s.description = "ruby-plsql gem provides simple Ruby API for calling Oracle PL/SQL procedures.\nIt could be used both for accessing Oracle PL/SQL API procedures in legacy applications\nas well as it could be used to create PL/SQL unit tests using Ruby testing libraries.\n"
-  s.email = "raimonds.simanovskis@gmail.com"
+  s.required_rubygems_version = Gem::Requirement.new(">= 0".freeze) if s.respond_to? :required_rubygems_version=
+  s.require_paths = ["lib".freeze]
+  s.authors = ["Raimonds Simanovskis".freeze]
+  s.date = "2018-09-03"
+  s.description = "  ruby-plsql gem provides simple Ruby API for calling Oracle PL/SQL procedures.\n  It could be used both for accessing Oracle PL/SQL API procedures in legacy applications\n  as well as it could be used to create PL/SQL unit tests using Ruby testing libraries.\n".freeze
+  s.email = "raimonds.simanovskis@gmail.com".freeze
   s.extra_rdoc_files = [
     "README.md"
   ]
@@ -70,38 +71,41 @@ Gem::Specification.new do |s|
     "spec/support/test_db.rb",
     "spec/support/unlock_and_setup_hr_user.sql"
   ]
-  s.homepage = "http://github.com/rsim/ruby-plsql"
-  s.rubygems_version = "2.5.1"
-  s.summary = "Ruby API for calling Oracle PL/SQL procedures."
+  s.homepage = "http://github.com/rsim/ruby-plsql".freeze
+  s.licenses = ["MIT".freeze]
+  s.rubygems_version = "2.6.4".freeze
+  s.summary = "Ruby API for calling Oracle PL/SQL procedures.".freeze
 
   if s.respond_to? :specification_version then
     s.specification_version = 4
 
-    if Gem::Version.new(Gem::VERSION) >= Gem::Version.new('1.2.0') then
-      s.add_development_dependency(%q<jeweler>, ["~> 2.0.1"])
-      s.add_development_dependency(%q<rspec>, ["~> 3.1"])
-      s.add_development_dependency(%q<rspec_junit_formatter>, [">= 0"])
-      s.add_development_dependency(%q<activerecord>, ["< 4.3.0", ">= 3.2.3"])
-      s.add_development_dependency(%q<activerecord-oracle_enhanced-adapter>, ["< 1.7.0", ">= 1.4.1"])
-      s.add_development_dependency(%q<simplecov>, [">= 0"])
-      s.add_development_dependency(%q<ruby-oci8>, ["~> 2.1"])
+    if Gem::Version.new(Gem::VERSION) >= Gem::Version.new("1.2.0") then
+      s.add_development_dependency("juwelier".freeze, ["~> 2.0"])
+      s.add_development_dependency("rspec_junit_formatter".freeze, [">= 0"])
+      s.add_development_dependency("rake".freeze, [">= 10.0"])
+      s.add_development_dependency("rspec".freeze, ["~> 3.1"])
+      s.add_development_dependency("activerecord".freeze, "~> 5.0")
+      s.add_development_dependency("activerecord-oracle_enhanced-adapter".freeze, "~> 1.7")
+      s.add_development_dependency("simplecov".freeze, [">= 0"])
+      s.add_development_dependency("ruby-oci8".freeze, ["~> 2.1"])
     else
-      s.add_dependency(%q<jeweler>, ["~> 2.0.1"])
-      s.add_dependency(%q<rspec>, ["~> 3.1"])
-      s.add_dependency(%q<rspec_junit_formatter>, [">= 0"])
-      s.add_dependency(%q<activerecord>, ["< 4.3.0", ">= 3.2.3"])
-      s.add_dependency(%q<activerecord-oracle_enhanced-adapter>, ["< 1.7.0", ">= 1.4.1"])
-      s.add_dependency(%q<simplecov>, [">= 0"])
-      s.add_dependency(%q<ruby-oci8>, ["~> 2.1"])
+      s.add_dependency("juwelier".freeze, ["~> 2.0"])
+      s.add_dependency("rspec_junit_formatter".freeze, [">= 0"])
+      s.add_dependency("rake".freeze, [">= 10.0"])
+      s.add_dependency("rspec".freeze, ["~> 3.1"])
+      s.add_dependency("activerecord".freeze, "~> 5.0")
+      s.add_dependency("activerecord-oracle_enhanced-adapter".freeze, "~> 1.7")
+      s.add_dependency("simplecov".freeze, [">= 0"])
+      s.add_dependency("ruby-oci8".freeze, ["~> 2.1"])
     end
   else
-    s.add_dependency(%q<jeweler>, ["~> 2.0.1"])
-    s.add_dependency(%q<rspec>, ["~> 3.1"])
-    s.add_dependency(%q<rspec_junit_formatter>, [">= 0"])
-    s.add_dependency(%q<activerecord>, ["< 4.3.0", ">= 3.2.3"])
-    s.add_dependency(%q<activerecord-oracle_enhanced-adapter>, ["< 1.7.0", ">= 1.4.1"])
-    s.add_dependency(%q<simplecov>, [">= 0"])
-    s.add_dependency(%q<ruby-oci8>, ["~> 2.1"])
+    s.add_dependency("juwelier".freeze, ["~> 2.0"])
+    s.add_dependency("rspec_junit_formatter".freeze, [">= 0"])
+    s.add_dependency("rake".freeze, [">= 10.0"])
+    s.add_dependency("rspec".freeze, ["~> 3.1"])
+    s.add_dependency("activerecord".freeze, "~> 5.0")
+    s.add_dependency("activerecord-oracle_enhanced-adapter".freeze, "~> 1.7")
+    s.add_dependency("simplecov".freeze, [">= 0"])
+    s.add_dependency("ruby-oci8".freeze, ["~> 2.1"])
   end
 end
-

data/spec/plsql/connection_spec.rb

@@ -1,12 +1,12 @@
 # encoding: utf-8
 
-require 'spec_helper'
+require "spec_helper"
 
 describe "Connection" do
 
   before(:all) do
     @raw_conn = get_connection
-    @conn = PLSQL::Connection.create( @raw_conn )
+    @conn = PLSQL::Connection.create(@raw_conn)
   end
 
   after(:all) do
@@ -23,7 +23,7 @@ describe "Connection" do
     end
 
     before(:each) do
-      @conn1 = PLSQL::Connection.create( @raw_conn1 )
+      @conn1 = PLSQL::Connection.create(@raw_conn1)
     end
 
     it "should create connection" do
@@ -52,42 +52,42 @@ describe "Connection" do
   unless defined?(JRuby)
     describe "OCI data type conversions" do
       it "should translate PL/SQL VARCHAR to Ruby String" do
-        expect(@conn.plsql_to_ruby_data_type(:data_type => "VARCHAR", :data_length => 100)).to eq [String, 100]
-        expect(@conn.plsql_to_ruby_data_type(:data_type => "VARCHAR", :data_length => nil)).to eq [String, 32767]
+        expect(@conn.plsql_to_ruby_data_type(data_type: "VARCHAR", data_length: 100)).to eq [String, 100]
+        expect(@conn.plsql_to_ruby_data_type(data_type: "VARCHAR", data_length: nil)).to eq [String, 32767]
       end
 
       it "should translate PL/SQL VARCHAR2 to Ruby String" do
-        expect(@conn.plsql_to_ruby_data_type(:data_type => "VARCHAR2", :data_length => 100)).to eq [String, 100]
-        expect(@conn.plsql_to_ruby_data_type(:data_type => "VARCHAR2", :data_length => nil)).to eq [String, 32767]
+        expect(@conn.plsql_to_ruby_data_type(data_type: "VARCHAR2", data_length: 100)).to eq [String, 100]
+        expect(@conn.plsql_to_ruby_data_type(data_type: "VARCHAR2", data_length: nil)).to eq [String, 32767]
       end
 
       it "should translate PL/SQL CLOB to Ruby String" do
-        expect(@conn.plsql_to_ruby_data_type(:data_type => "CLOB", :data_length => 100_000)).to eq [OCI8::CLOB, nil]
-        expect(@conn.plsql_to_ruby_data_type(:data_type => "CLOB", :data_length => nil)).to eq [OCI8::CLOB, nil]
+        expect(@conn.plsql_to_ruby_data_type(data_type: "CLOB", data_length: 100_000)).to eq [OCI8::CLOB, nil]
+        expect(@conn.plsql_to_ruby_data_type(data_type: "CLOB", data_length: nil)).to eq [OCI8::CLOB, nil]
       end
 
       it "should translate PL/SQL NUMBER to Ruby OraNumber" do
-        expect(@conn.plsql_to_ruby_data_type(:data_type => "NUMBER", :data_length => 15)).to eq [OraNumber, nil]
+        expect(@conn.plsql_to_ruby_data_type(data_type: "NUMBER", data_length: 15)).to eq [OraNumber, nil]
       end
 
       it "should translate PL/SQL DATE to Ruby DateTime" do
-        expect(@conn.plsql_to_ruby_data_type(:data_type => "DATE", :data_length => nil)).to eq [DateTime, nil]
+        expect(@conn.plsql_to_ruby_data_type(data_type: "DATE", data_length: nil)).to eq [DateTime, nil]
       end
 
       it "should translate PL/SQL TIMESTAMP to Ruby Time" do
-        expect(@conn.plsql_to_ruby_data_type(:data_type => "TIMESTAMP", :data_length => nil)).to eq [Time, nil]
+        expect(@conn.plsql_to_ruby_data_type(data_type: "TIMESTAMP", data_length: nil)).to eq [Time, nil]
       end
 
-      it "should not translate Ruby Fixnum when OraNumber type specified" do
+      it "should not translate small Ruby Integer when OraNumber type specified" do
         expect(@conn.ruby_value_to_ora_value(100, OraNumber)).to eql(100)
       end
 
-      it "should translate Ruby Bignum value to OraNumber when OraNumber type specified" do
+      it "should not translate big Ruby Integer when OraNumber type specified" do
         ora_number = @conn.ruby_value_to_ora_value(12345678901234567890, OraNumber)
-        expect(ora_number.class).to eq OraNumber
+        expect(ora_number).to be_an Integer
         expect(ora_number.to_s).to eq "12345678901234567890"
         # OraNumber has more numeric comparison methods in ruby-oci8 2.0
-        expect(ora_number).to eq OraNumber.new("12345678901234567890") if OCI8::VERSION >= '2.0.0'
+        expect(ora_number).to eq OraNumber.new("12345678901234567890") if OCI8::VERSION >= "2.0.0"
       end
 
       it "should translate Ruby String value to OCI8::CLOB when OCI8::CLOB type specified" do
@@ -99,7 +99,7 @@ describe "Connection" do
         expect(ora_value.read).to eq large_text
       end
 
-      it "should translate Oracle OraNumber integer value to Fixnum" do
+      it "should translate Oracle OraNumber integer value to Integer" do
         expect(@conn.ora_value_to_ruby_value(OraNumber.new(100))).to eql(100)
       end
 
@@ -108,7 +108,7 @@ describe "Connection" do
       end
 
       # ruby-oci8 2.0 returns DATE as Time or DateTime
-      if OCI8::VERSION < '2.0.0'
+      if OCI8::VERSION < "2.0.0"
         it "should translate Oracle OraDate value to Time" do
           now = OraDate.now
           expect(@conn.ora_value_to_ruby_value(now)).to eql(now.to_time)
@@ -128,35 +128,35 @@ describe "Connection" do
 
     describe "JDBC data type conversions" do
       it "should translate PL/SQL VARCHAR to Ruby String" do
-        expect(@conn.plsql_to_ruby_data_type(:data_type => "VARCHAR", :data_length => 100)).to eq [String, 100]
-        expect(@conn.plsql_to_ruby_data_type(:data_type => "VARCHAR", :data_length => nil)).to eq [String, 32767]
+        expect(@conn.plsql_to_ruby_data_type(data_type: "VARCHAR", data_length: 100)).to eq [String, 100]
+        expect(@conn.plsql_to_ruby_data_type(data_type: "VARCHAR", data_length: nil)).to eq [String, 32767]
       end
       it "should translate PL/SQL VARCHAR2 to Ruby String" do
-        expect(@conn.plsql_to_ruby_data_type(:data_type => "VARCHAR2", :data_length => 100)).to eq [String, 100]
-        expect(@conn.plsql_to_ruby_data_type(:data_type => "VARCHAR2", :data_length => nil)).to eq [String, 32767]
+        expect(@conn.plsql_to_ruby_data_type(data_type: "VARCHAR2", data_length: 100)).to eq [String, 100]
+        expect(@conn.plsql_to_ruby_data_type(data_type: "VARCHAR2", data_length: nil)).to eq [String, 32767]
       end
 
       it "should translate PL/SQL NUMBER to Ruby BigDecimal" do
-        expect(@conn.plsql_to_ruby_data_type(:data_type => "NUMBER", :data_length => 15)).to eq [BigDecimal, nil]
+        expect(@conn.plsql_to_ruby_data_type(data_type: "NUMBER", data_length: 15)).to eq [BigDecimal, nil]
       end
 
       it "should translate PL/SQL DATE to Ruby DateTime" do
-        expect(@conn.plsql_to_ruby_data_type(:data_type => "DATE", :data_length => nil)).to eq [DateTime, nil]
+        expect(@conn.plsql_to_ruby_data_type(data_type: "DATE", data_length: nil)).to eq [DateTime, nil]
       end
 
       it "should translate PL/SQL TIMESTAMP to Ruby Time" do
-        expect(@conn.plsql_to_ruby_data_type(:data_type => "TIMESTAMP", :data_length => nil)).to eq [Time, nil]
+        expect(@conn.plsql_to_ruby_data_type(data_type: "TIMESTAMP", data_length: nil)).to eq [Time, nil]
       end
 
-      it "should not translate Ruby Fixnum when BigDecimal type specified" do
+      it "should not translate Ruby Integer when BigDecimal type specified" do
         expect(@conn.ruby_value_to_ora_value(100, BigDecimal)).to eq java.math.BigDecimal.new(100)
       end
 
       it "should translate Ruby String to string value" do
-        expect(@conn.ruby_value_to_ora_value(1.1, String)).to eq '1.1'
+        expect(@conn.ruby_value_to_ora_value(1.1, String)).to eq "1.1"
       end
 
-      it "should translate Ruby Bignum value to BigDecimal when BigDecimal type specified" do
+      it "should translate Ruby Integer value to BigDecimal when BigDecimal type specified" do
         big_decimal = @conn.ruby_value_to_ora_value(12345678901234567890, BigDecimal)
         expect(big_decimal).to eq java.math.BigDecimal.new("12345678901234567890")
       end
@@ -175,7 +175,7 @@ describe "Connection" do
         expect(ora_value).to be_nil
       end
 
-      it "should translate Oracle BigDecimal integer value to Fixnum" do
+      it "should translate Oracle BigDecimal integer value to Integer" do
         expect(@conn.ora_value_to_ruby_value(BigDecimal("100"))).to eql(100)
       end
 
@@ -202,87 +202,87 @@ describe "Connection" do
   describe "SQL SELECT statements" do
 
     it "should execute SQL statement and return first result" do
-      @now = Time.local(2008,05,31,23,22,11)
+      @now = Time.local(2008, 05, 31, 23, 22, 11)
       expect(@conn.select_first("SELECT 'abc',123,123.456,
         TO_DATE('#{@now.strftime("%Y-%m-%d %H:%M:%S")}','YYYY-MM-DD HH24:MI:SS')
-        FROM dual")).to eq ["abc",123,123.456,@now]
+        FROM dual")).to eq ["abc", 123, 123.456, @now]
     end
 
     it "should execute SQL statement and return first result as hash" do
-      @now = Time.local(2008,05,31,23,22,11)
+      @now = Time.local(2008, 05, 31, 23, 22, 11)
       expect(@conn.select_hash_first("SELECT 'abc' a, 123 b, 123.456 c,
         TO_DATE('#{@now.strftime("%Y-%m-%d %H:%M:%S")}', 'YYYY-MM-DD HH24:MI:SS') d
-        FROM dual")).to eq({:a => "abc", :b => 123, :c => 123.456, :d => @now})
+        FROM dual")).to eq(a: "abc", b: 123, c: 123.456, d: @now)
     end
 
     it "should execute SQL statement with bind parameters and return first result" do
       @today = Date.parse("2008-05-31")
-      @now = Time.local(2008,05,31,23,22,11)
+      @now = Time.local(2008, 05, 31, 23, 22, 11)
       expect(@conn.select_first("SELECT :1,:2,:3,:4,:5 FROM dual",
-        'abc',123,123.456,@now,@today)).to eq ["abc",123,123.456,@now,Time.parse(@today.to_s)]
+        "abc", 123, 123.456, @now, @today)).to eq ["abc", 123, 123.456, @now, Time.parse(@today.to_s)]
     end
 
     it "should execute SQL statement with NULL values and return first result" do
-      @now = Time.local(2008,05,31,23,22,11)
+      @now = Time.local(2008, 05, 31, 23, 22, 11)
       expect(@conn.select_first("SELECT NULL,123,123.456,
         TO_DATE('#{@now.strftime("%Y-%m-%d %H:%M:%S")}','YYYY-MM-DD HH24:MI:SS')
-        FROM dual")).to eq [nil,123,123.456,@now]
+        FROM dual")).to eq [nil, 123, 123.456, @now]
     end
 
     if defined?(JRuby)
 
       it "should execute SQL statement with NULL values as bind parameters and return first result" do
         @today = Date.parse("2008-05-31")
-        @now = Time.local(2008,05,31,23,22,11)
+        @now = Time.local(2008, 05, 31, 23, 22, 11)
         expect(@conn.select_first("SELECT :1,:2,:3,:4,:5 FROM dual",
-          nil,123,123.456,@now,@today)).to eq [nil,123,123.456,@now,Time.parse(@today.to_s)]
+          nil, 123, 123.456, @now, @today)).to eq [nil, 123, 123.456, @now, Time.parse(@today.to_s)]
       end
 
     end
 
     it "should execute SQL statement and return all results" do
-      @now = Time.local(2008,05,31,23,22,11)
+      @now = Time.local(2008, 05, 31, 23, 22, 11)
       expect(@conn.select_all("SELECT 'abc',123,123.456,
           TO_DATE('#{@now.strftime("%Y-%m-%d %H:%M:%S")}','YYYY-MM-DD HH24:MI:SS')
           FROM dual
           UNION ALL SELECT 'abc',123,123.456,
           TO_DATE('#{@now.strftime("%Y-%m-%d %H:%M:%S")}','YYYY-MM-DD HH24:MI:SS')
-          FROM dual")).to eq [["abc",123,123.456,@now],["abc",123,123.456,@now]]
+          FROM dual")).to eq [["abc", 123, 123.456, @now], ["abc", 123, 123.456, @now]]
     end
 
     it "should execute SQL statement and return all results as hash" do
-      @now = Time.local(2008,05,31,23,22,11)
+      @now = Time.local(2008, 05, 31, 23, 22, 11)
       expect(@conn.select_hash_all("SELECT 'abc' a, 123 b, 123.456 c,
           TO_DATE('#{@now.strftime("%Y-%m-%d %H:%M:%S")}','YYYY-MM-DD HH24:MI:SS') d
           FROM dual
           UNION ALL SELECT 'def' a, 123 b, 123.456 c,
           TO_DATE('#{@now.strftime("%Y-%m-%d %H:%M:%S")}','YYYY-MM-DD HH24:MI:SS') d
-          FROM dual")).to eq [{:a=>"abc",:b=>123,:c=>123.456,:d=>@now},{:a=>"def",:b=>123,:c=>123.456,:d=>@now}]
+          FROM dual")).to eq [{ a: "abc", b: 123, c: 123.456, d: @now }, { a: "def", b: 123, c: 123.456, d: @now }]
     end
 
     it "should execute SQL statement with bind parameters and return all results" do
-      @now = Time.local(2008,05,31,23,22,11)
+      @now = Time.local(2008, 05, 31, 23, 22, 11)
       expect(@conn.select_all("SELECT :1,:2,:3,:4 FROM dual UNION ALL SELECT :1,:2,:3,:4 FROM dual",
-        'abc',123,123.456,@now,'abc',123,123.456,@now)).to eq [["abc",123,123.456,@now],["abc",123,123.456,@now]]
+        "abc", 123, 123.456, @now, "abc", 123, 123.456, @now)).to eq [["abc", 123, 123.456, @now], ["abc", 123, 123.456, @now]]
     end
 
     it "should execute SQL statement and yield all results in block" do
-      @now = Time.local(2008,05,31,23,22,11)
+      @now = Time.local(2008, 05, 31, 23, 22, 11)
       expect(@conn.select_all("SELECT 'abc',123,123.456,
           TO_DATE('#{@now.strftime("%Y-%m-%d %H:%M:%S")}','YYYY-MM-DD HH24:MI:SS')
           FROM dual
           UNION ALL SELECT 'abc',123,123.456,
           TO_DATE('#{@now.strftime("%Y-%m-%d %H:%M:%S")}','YYYY-MM-DD HH24:MI:SS')
           FROM dual") do |r|
-        expect(r).to eq ["abc",123,123.456,@now]
+        expect(r).to eq ["abc", 123, 123.456, @now]
       end).to eq 2
     end
 
     it "should execute SQL statement with bind parameters and yield all results in block" do
-      @now = Time.local(2008,05,31,23,22,11)
+      @now = Time.local(2008, 05, 31, 23, 22, 11)
       expect(@conn.select_all("SELECT :1,:2,:3,:4 FROM dual UNION ALL SELECT :1,:2,:3,:4 FROM dual",
-        'abc',123,123.456,@now,'abc',123,123.456,@now) do |r|
-        expect(r).to eq ["abc",123,123.456,@now]
+        "abc", 123, 123.456, @now, "abc", 123, 123.456, @now) do |r|
+        expect(r).to eq ["abc", 123, 123.456, @now]
       end).to eq 2
     end
 
@@ -291,7 +291,7 @@ describe "Connection" do
   describe "PL/SQL procedures" do
     before(:all) do
       @random = rand(1000)
-      @now = Time.local(2008,05,31,23,22,11)
+      @now = Time.local(2008, 05, 31, 23, 22, 11)
       sql = <<-SQL
         CREATE OR REPLACE FUNCTION test_add_random (p_number NUMBER, p_varchar IN OUT VARCHAR2, p_date IN OUT DATE)
           RETURN NUMBER
@@ -314,10 +314,10 @@ describe "Connection" do
         END;
       SQL
       cursor = @conn.parse(sql)
-      cursor.bind_param(":result", nil, :data_type => 'NUMBER', :in_out => 'OUT')
-      cursor.bind_param(":p_number", 100, :data_type => 'NUMBER', :in_out => 'IN')
-      cursor.bind_param(":p_varchar", "abc", :data_type => 'VARCHAR2', :in_out => 'IN/OUT')
-      cursor.bind_param(":p_date", @now, :data_type => 'DATE', :in_out => 'IN/OUT')
+      cursor.bind_param(":result", nil, data_type: "NUMBER", in_out: "OUT")
+      cursor.bind_param(":p_number", 100, data_type: "NUMBER", in_out: "IN")
+      cursor.bind_param(":p_varchar", "abc", data_type: "VARCHAR2", in_out: "IN/OUT")
+      cursor.bind_param(":p_date", @now, data_type: "DATE", in_out: "IN/OUT")
       cursor.exec
       expect(cursor[":result"]).to eq @random + 100
       expect(cursor[":p_varchar"]).to eq "abc"
@@ -401,21 +401,21 @@ describe "Connection" do
     end
 
     it "should describe local synonym" do
-      expect(@conn.describe_synonym('HR','SYNONYM_FOR_DUAL')).to eq ['SYS', 'DUAL']
-      expect(@conn.describe_synonym('hr','synonym_for_dual')).to eq ['SYS', 'DUAL']
-      expect(@conn.describe_synonym(:hr,:synonym_for_dual)).to eq ['SYS', 'DUAL']
+      expect(@conn.describe_synonym("HR", "SYNONYM_FOR_DUAL")).to eq ["SYS", "DUAL"]
+      expect(@conn.describe_synonym("hr", "synonym_for_dual")).to eq ["SYS", "DUAL"]
+      expect(@conn.describe_synonym(:hr, :synonym_for_dual)).to eq ["SYS", "DUAL"]
     end
 
     it "should return nil on non-existing synonym" do
-      expect(@conn.describe_synonym('HR','SYNONYM_FOR_XXX')).to be_nil
-      expect(@conn.describe_synonym('hr','synonym_for_xxx')).to be_nil
-      expect(@conn.describe_synonym(:hr,:synonym_for_xxx)).to be_nil
+      expect(@conn.describe_synonym("HR", "SYNONYM_FOR_XXX")).to be_nil
+      expect(@conn.describe_synonym("hr", "synonym_for_xxx")).to be_nil
+      expect(@conn.describe_synonym(:hr, :synonym_for_xxx)).to be_nil
     end
 
     it "should describe public synonym" do
-      expect(@conn.describe_synonym('PUBLIC','DUAL')).to eq ['SYS', 'DUAL']
-      expect(@conn.describe_synonym('PUBLIC','dual')).to eq ['SYS', 'DUAL']
-      expect(@conn.describe_synonym('PUBLIC',:dual)).to eq ['SYS', 'DUAL']
+      expect(@conn.describe_synonym("PUBLIC", "DUAL")).to eq ["SYS", "DUAL"]
+      expect(@conn.describe_synonym("PUBLIC", "dual")).to eq ["SYS", "DUAL"]
+      expect(@conn.describe_synonym("PUBLIC", :dual)).to eq ["SYS", "DUAL"]
     end
 
   end
@@ -423,7 +423,7 @@ describe "Connection" do
   describe "session information" do
     it "should get database version" do
       # using Oracle version 10.2.0.4 for unit tests
-      expect(@conn.database_version).to eq DATABASE_VERSION.split('.').map{|n| n.to_i}
+      expect(@conn.database_version).to eq DATABASE_VERSION.split(".").map { |n| n.to_i }
     end
 
     it "should get session ID" do
@@ -453,7 +453,7 @@ describe "Connection" do
     end
 
     it "should not drop other session ruby temporary tables" do
-      tmp_table = "ruby_#{@conn.session_id+1}_222_333"
+      tmp_table = "ruby_#{@conn.session_id + 1}_222_333"
       @conn.exec "CREATE GLOBAL TEMPORARY TABLE #{tmp_table} (dummy CHAR(1))"
       expect { @conn.select_first("SELECT * FROM #{tmp_table}") }.not_to raise_error
       @conn.drop_session_ruby_temporary_tables
@@ -474,7 +474,7 @@ describe "Connection" do
 
     def reconnect_connection
       @raw_conn = get_connection
-      @conn = PLSQL::Connection.create( @raw_conn )
+      @conn = PLSQL::Connection.create(@raw_conn)
     end
 
     it "should drop current session ruby temporary tables" do

data/spec/plsql/package_spec.rb

@@ -1,4 +1,4 @@
-require 'spec_helper'
+require "spec_helper"
 
 describe "Package" do
   before(:all) do
@@ -21,15 +21,15 @@ describe "Package" do
       END;
     SQL
   end
-  
+
   after(:all) do
     plsql.execute "DROP PACKAGE test_package"
     plsql.logoff
   end
-  
+
   before(:each) do
   end
-  
+
   it "should find existing package" do
     expect(PLSQL::Package.find(plsql, :test_package)).not_to be_nil
   end
@@ -43,7 +43,7 @@ describe "Package" do
   end
 
   it "should execute package function and return correct value" do
-    expect(plsql.test_package.test_procedure('xxx')).to eq('XXX')
+    expect(plsql.test_package.test_procedure("xxx")).to eq("XXX")
   end
 
   it "should report an existing procedure as existing" do
@@ -57,11 +57,11 @@ describe "Package" do
   it "should search objects via []" do
     package = PLSQL::Package.find(plsql, :test_package)
 
-    [:Test_Procedure, :test_procedure, 'test_procedure', 'TEST_PROCEDURE'].each do |name_variant|
+    [:Test_Procedure, :test_procedure, "test_procedure", "TEST_PROCEDURE"].each do |name_variant|
       expect(package[name_variant]).to be_a PLSQL::Procedure
     end
 
-    [:Test_Variable, :test_variable, 'test_variable', 'TEST_VARIABLE'].each do |name_variant|
+    [:Test_Variable, :test_variable, "test_variable", "TEST_VARIABLE"].each do |name_variant|
       expect(package[name_variant]).to be_a PLSQL::Variable
     end
   end
@@ -114,7 +114,7 @@ describe "Package" do
 end
 
 describe "Synonym to package" do
-  
+
   before(:all) do
     plsql.connection = get_connection
     plsql.execute <<-SQL
@@ -135,38 +135,38 @@ describe "Synonym to package" do
     SQL
     plsql.execute "CREATE SYNONYM test_pkg_synonym FOR hr.test_package"
   end
-  
+
   after(:all) do
     plsql.execute "DROP SYNONYM test_pkg_synonym" rescue nil
     plsql.logoff
   end
-  
+
   it "should find synonym to package" do
     expect(PLSQL::Package.find(plsql, :test_pkg_synonym)).not_to be_nil
   end
 
   it "should execute package function using synonym and return correct value" do
-    expect(plsql.test_pkg_synonym.test_procedure('xxx')).to eq('XXX')
+    expect(plsql.test_pkg_synonym.test_procedure("xxx")).to eq("XXX")
   end
 
 end
 
 describe "Public synonym to package" do
-  
+
   before(:all) do
     plsql.connection = get_connection
   end
-  
+
   after(:all) do
     plsql.logoff
   end
-  
+
   it "should find public synonym to package" do
     expect(PLSQL::Package.find(plsql, :utl_encode)).not_to be_nil
   end
 
   it "should execute package function using public synonym and return correct value" do
-    expect(plsql.utl_encode.base64_encode('abc')).to eq('4372773D')
+    expect(plsql.utl_encode.base64_encode("abc")).to eq("4372773D")
   end
 
 end