ruby-plsql 0.5.3 → 0.8.0

data/spec/plsql/type_spec.rb

@@ -1,4 +1,4 @@
-require 'spec_helper'
+require "spec_helper"
 
 describe "Type" do
   before(:all) do
@@ -80,7 +80,7 @@ describe "Type" do
       CREATE OR REPLACE TYPE t_employee AS OBJECT (
         employee_id   NUMBER(15),
         first_name    VARCHAR2(50),
-        last_name     VARCHAR2(50),
+        last_name     VARCHAR(50),
         hire_date     DATE,
         address       t_address,
         phones        t_phones
@@ -167,28 +167,22 @@ describe "Type" do
     end
 
     it "should get attributes metadata" do
-      expect(plsql.t_employee.attributes).to eq({
-        :employee_id =>
-          {:position=>1, :data_type=>"NUMBER", :data_length=>nil, :data_precision=>15, :data_scale=>0, :type_owner=>nil, :type_name=>nil, :sql_type_name=>nil},
-        :first_name =>
-          {:position=>2, :data_type=>"VARCHAR2", :data_length=>50, :data_precision=>nil, :data_scale=>nil, :type_owner=>nil, :type_name=>nil, :sql_type_name=>nil},
-        :last_name =>
-          {:position=>3, :data_type=>"VARCHAR2", :data_length=>50, :data_precision=>nil, :data_scale=>nil, :type_owner=>nil, :type_name=>nil, :sql_type_name=>nil},
-        :hire_date => 
-          {:position=>4, :data_type=>"DATE", :data_length=>nil, :data_precision=>nil, :data_scale=>nil, :type_owner=>nil, :type_name=>nil, :sql_type_name=>nil},
-        :address => 
-          {:position=>5, :data_type=>"OBJECT", :data_length=>nil, :data_precision=>nil, :data_scale=>nil, :type_owner=>"HR", :type_name=>"T_ADDRESS", :sql_type_name=>"HR.T_ADDRESS"},
-        :phones => 
-          {:position=>6, :data_type=>"TABLE", :data_length=>nil, :data_precision=>nil, :data_scale=>nil, :type_owner=>"HR", :type_name=>"T_PHONES", :sql_type_name=>"HR.T_PHONES"}
-      })
+      expect(plsql.t_employee.attributes).to eq(
+        employee_id:           { position: 1, data_type: "NUMBER", data_length: nil, data_precision: 15, data_scale: 0, type_owner: nil, type_name: nil, sql_type_name: nil },
+        first_name:           { position: 2, data_type: "VARCHAR2", data_length: 50, data_precision: nil, data_scale: nil, type_owner: nil, type_name: nil, sql_type_name: nil },
+        last_name:           { position: 3, data_type: "VARCHAR2", data_length: 50, data_precision: nil, data_scale: nil, type_owner: nil, type_name: nil, sql_type_name: nil },
+        hire_date:           { position: 4, data_type: "DATE", data_length: nil, data_precision: nil, data_scale: nil, type_owner: nil, type_name: nil, sql_type_name: nil },
+        address:           { position: 5, data_type: "OBJECT", data_length: nil, data_precision: nil, data_scale: nil, type_owner: "HR", type_name: "T_ADDRESS", sql_type_name: "HR.T_ADDRESS" },
+        phones:           { position: 6, data_type: "TABLE", data_length: nil, data_precision: nil, data_scale: nil, type_owner: "HR", type_name: "T_PHONES", sql_type_name: "HR.T_PHONES" }
+      )
     end
 
   end
 
   describe "object instance" do
     before(:all) do
-      @phone_attributes = {:type => 'mobile', :phone_number => '123456'}
-      @address_attributes = {:street => 'Street', :city => 'City', :country => 'Country'}
+      @phone_attributes = { type: "mobile", phone_number: "123456" }
+      @address_attributes = { street: "Street", city: "City", country: "Country" }
       @full_address = "#{@address_attributes[:street]}, #{@address_attributes[:city]}, #{@address_attributes[:country]}"
     end
 
@@ -206,7 +200,7 @@ describe "Type" do
 
     it "should get new object instance using custom constructor" do
       expect(plsql.t_address(@full_address)).to eq(@address_attributes)
-      expect(plsql.t_address(:p_full_address => @full_address)).to eq(@address_attributes)
+      expect(plsql.t_address(p_full_address: @full_address)).to eq(@address_attributes)
     end
 
     it "should get new object instance using default constructor when custom constructor exists" do
@@ -230,7 +224,7 @@ describe "Type" do
 
   describe "member procedures" do
     before(:all) do
-      @address_attributes = {:street => 'Street', :city => 'City', :country => 'Country'}
+      @address_attributes = { street: "Street", city: "City", country: "Country" }
       @full_address = "#{@address_attributes[:street]}, #{@address_attributes[:city]}, #{@address_attributes[:country]}"
     end
 
@@ -239,36 +233,36 @@ describe "Type" do
     end
 
     it "should call object instance member function with parameters" do
-      expect(plsql.t_address(@address_attributes).display_address(',')).to eq(@full_address)
+      expect(plsql.t_address(@address_attributes).display_address(",")).to eq(@full_address)
     end
 
     it "should call object instance member function with named parameters" do
-      expect(plsql.t_address(@address_attributes).display_address(:p_separator => ',')).to eq(@full_address)
+      expect(plsql.t_address(@address_attributes).display_address(p_separator: ",")).to eq(@full_address)
     end
 
     it "should call object overloaded instance member function" do
       expect(plsql.t_address(@address_attributes).display_address(true)).to eq(@full_address.upcase)
-      expect(plsql.t_address(@address_attributes).display_address(true, ',')).to eq(@full_address.upcase)
+      expect(plsql.t_address(@address_attributes).display_address(true, ",")).to eq(@full_address.upcase)
     end
 
     it "should call object instance member function with explicit first SELF parameter" do
-      expect(plsql.t_address.display_address(@address_attributes, ',')).to eq(@full_address)
+      expect(plsql.t_address.display_address(@address_attributes, ",")).to eq(@full_address)
     end
 
     it "should call object instance member function with explicit named SELF parameter" do
-      expect(plsql.t_address.display_address(:self => @address_attributes, :p_separator => ',')).to eq(@full_address)
+      expect(plsql.t_address.display_address(self: @address_attributes, p_separator: ",")).to eq(@full_address)
     end
 
     it "should call object instance member procedure" do
       other_country = "Other"
-      expect(plsql.t_address(@address_attributes).set_country(other_country)).to eq(@address_attributes.merge(:country => other_country))
+      expect(plsql.t_address(@address_attributes).set_country(other_country)).to eq(@address_attributes.merge(country: other_country))
     end
 
     it "should call object instance member procedure with output parameters" do
       other_country = "Other"
       expect(plsql.t_address(@address_attributes).set_country2(other_country)).to eq(
-        [@address_attributes.merge(:country => other_country),
-        {:x_display_address => "#{@address_attributes[:street]}, #{@address_attributes[:city]}, #{other_country}"}]
+        [@address_attributes.merge(country: other_country),
+        { x_display_address: "#{@address_attributes[:street]}, #{@address_attributes[:city]}, #{other_country}" }]
       )
     end
 
@@ -282,7 +276,7 @@ describe "Type" do
 
   describe "static procedures" do
     before(:all) do
-      @address_attributes = {:street => 'Street', :city => 'City', :country => 'Country'}
+      @address_attributes = { street: "Street", city: "City", country: "Country" }
       @full_address = "#{@address_attributes[:street]}, #{@address_attributes[:city]}, #{@address_attributes[:country]}"
     end
 
@@ -291,7 +285,7 @@ describe "Type" do
     end
 
     it "should call object type static function with named parameters" do
-      expect(plsql.t_address.create_address(:p_full_address => @full_address)).to eq(@address_attributes)
+      expect(plsql.t_address.create_address(p_full_address: @full_address)).to eq(@address_attributes)
     end
 
     it "should raise error if invalid static procedure is called" do

data/spec/plsql/variable_spec.rb

@@ -1,6 +1,6 @@
 # encoding: utf-8
 
-require 'spec_helper'
+require "spec_helper"
 
 describe "Package variables /" do
 
@@ -16,6 +16,7 @@ describe "Package variables /" do
           varchar2_default3 varchar2(50) NOT NULL := 'default';
           varchar2_3_char VARCHAR2(3 CHAR);
           varchar2_3_byte VARCHAR2(3 BYTE);
+          varchar_variable VARCHAR(50);
           char_variable char(10) ;
           nvarchar2_variable NVARCHAR2(50);
           nchar_variable NCHAR(10);
@@ -33,20 +34,25 @@ describe "Package variables /" do
       plsql.logoff
     end
 
+    it "should set and get VARCHAR variable" do
+      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
@@ -64,56 +70,46 @@ 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
 
-  describe "Numeric" do
+  shared_examples "Numeric" do |ora_data_type, default, class_, given, expected|
+
     before(:all) do
       plsql.connect! CONNECTION_PARAMS
       plsql.execute <<-SQL
         CREATE OR REPLACE PACKAGE test_package IS
-          integer_variable INTEGER;
-          integer10_variable NUMBER(10);
-          integer10_default NUMBER(10) := 1;
-          number_variable NUMBER;
-          number_with_scale NUMBER(15,2);
-          pls_int_variable PLS_INTEGER;
-          bin_int_variable BINARY_INTEGER;
+          numeric_var #{ora_data_type}#{default ? ':= ' + default.to_s : nil};
         END;
       SQL
-      plsql.execute <<-SQL
-        CREATE OR REPLACE PACKAGE BODY test_package IS
-        END;
-      SQL
-
     end
 
     after(:all) do
@@ -121,45 +117,41 @@ describe "Package variables /" do
       plsql.logoff
     end
 
-    it "should set and get INTEGER variable" do
-      plsql.test_package.integer_variable = 1
-      expect(plsql.test_package.integer_variable).to be_a Fixnum
-      expect(plsql.test_package.integer_variable).to eq(1)
-    end
+    it "should get #{ora_data_type} variable default value" do
+      expect(plsql.test_package.numeric_var).to eq(default)
+    end if default
 
-    it "should set and get integer variable with precision" do
-      plsql.test_package.integer10_variable = 1
-      expect(plsql.test_package.integer10_variable).to be_a Fixnum
-      expect(plsql.test_package.integer10_variable).to eq(1)
+    it "should get #{ora_data_type} variable type mapped to #{class_.to_s}" do
+      plsql.test_package.numeric_var = given
+      expect(plsql.test_package.numeric_var).to be_a class_
     end
 
-    it "should get integer variable default value" do
-      expect(plsql.test_package.integer10_default).to eq(1)
+    it "should set and get #{ora_data_type} variable" do
+      plsql.test_package.numeric_var = given
+      expect(plsql.test_package.numeric_var).to eq(expected)
     end
 
-    it "should set and get PLS_INTEGER variable" do
-      plsql.test_package.pls_int_variable = 1
-      expect(plsql.test_package.pls_int_variable).to be_a Fixnum
-      expect(plsql.test_package.pls_int_variable).to eq(1)
-    end
-
-    it "should set and get BINARY_INTEGER variable" do
-      plsql.test_package.bin_int_variable = 1
-      expect(plsql.test_package.bin_int_variable).to be_a Fixnum
-      expect(plsql.test_package.bin_int_variable).to eq(1)
-    end
-
-    it "should set and get NUMBER variable" do
-      plsql.test_package.number_variable = 123.456
-      expect(plsql.test_package.number_variable).to be_a BigDecimal
-      expect(plsql.test_package.number_variable).to eq(123.456)
-    end
+  end
 
-    it "should set and get NUMBER variable with scale" do
-      plsql.test_package.number_with_scale = 123.456
-      expect(plsql.test_package.number_with_scale).to eq(123.46) # rounding to two decimal digits
+  [
+      { 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
+      include_examples "Numeric", ora_data_type, default, class_, given, expected
     end
-
   end
 
   describe "Date and Time" do
@@ -178,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
@@ -241,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
@@ -299,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
@@ -337,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
@@ -379,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
@@ -441,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