activerecord-oracle_enhanced-adapter 1.8.2 → 5.2.0.beta1

data/spec/active_record/oracle_enhanced/type/binary_spec.rb

@@ -0,0 +1,119 @@
+# frozen_string_literal: true
+
+describe "OracleEnhancedAdapter handling of BLOB columns" do
+  include SchemaSpecHelper
+
+  before(:all) do
+    ActiveRecord::Base.establish_connection(CONNECTION_PARAMS)
+    @conn = ActiveRecord::Base.connection
+    schema_define do
+      create_table :test_employees, force: true do |t|
+        t.string  :first_name, limit: 20
+        t.string  :last_name, limit: 25
+        t.binary  :binary_data
+      end
+    end
+    class ::TestEmployee < ActiveRecord::Base
+    end
+    @binary_data = "\0\1\2\3\4\5\6\7\8\9" * 10000
+    @binary_data2 = "\1\2\3\4\5\6\7\8\9\0" * 10000
+  end
+
+  after(:all) do
+    @conn.drop_table :test_employees, if_exists: true
+    Object.send(:remove_const, "TestEmployee")
+  end
+
+  after(:each) do
+    ActiveRecord::Base.clear_cache!
+  end
+
+  it "should create record with BLOB data" do
+    @employee = TestEmployee.create!(
+      first_name: "First",
+      last_name: "Last",
+      binary_data: @binary_data
+    )
+    @employee.reload
+    expect(@employee.binary_data).to eq(@binary_data)
+  end
+
+  it "should update record with BLOB data" do
+    @employee = TestEmployee.create!(
+      first_name: "First",
+      last_name: "Last"
+    )
+    @employee.reload
+    expect(@employee.binary_data).to be_nil
+    @employee.binary_data = @binary_data
+    @employee.save!
+    @employee.reload
+    expect(@employee.binary_data).to eq(@binary_data)
+  end
+
+  it "should update record with zero-length BLOB data" do
+    @employee = TestEmployee.create!(
+      first_name: "First",
+      last_name: "Last"
+    )
+    @employee.reload
+    expect(@employee.binary_data).to be_nil
+    @employee.binary_data = ""
+    @employee.save!
+    @employee.reload
+    expect(@employee.binary_data).to eq("")
+  end
+
+  it "should update record that has existing BLOB data with different BLOB data" do
+    @employee = TestEmployee.create!(
+      first_name: "First",
+      last_name: "Last",
+      binary_data: @binary_data
+    )
+    @employee.reload
+    @employee.binary_data = @binary_data2
+    @employee.save!
+    @employee.reload
+    expect(@employee.binary_data).to eq(@binary_data2)
+  end
+
+  it "should update record that has existing BLOB data with nil" do
+    @employee = TestEmployee.create!(
+      first_name: "First",
+      last_name: "Last",
+      binary_data: @binary_data
+    )
+    @employee.reload
+    @employee.binary_data = nil
+    @employee.save!
+    @employee.reload
+    expect(@employee.binary_data).to be_nil
+  end
+
+  it "should update record that has existing BLOB data with zero-length BLOB data" do
+    @employee = TestEmployee.create!(
+      first_name: "First",
+      last_name: "Last",
+      binary_data: @binary_data
+    )
+    @employee.reload
+    @employee.binary_data = ""
+    @employee.save!
+    @employee.reload
+    expect(@employee.binary_data).to eq("")
+  end
+
+  it "should update record that has zero-length BLOB data with non-empty BLOB data" do
+    @employee = TestEmployee.create!(
+      first_name: "First",
+      last_name: "Last",
+      binary_data: ""
+    )
+    @employee.reload
+    expect(@employee.binary_data).to eq("")
+    @employee.binary_data = @binary_data
+    @employee.save!
+    @employee.reload
+    expect(@employee.binary_data).to eq(@binary_data)
+  end
+end

data/spec/active_record/oracle_enhanced/type/boolean_spec.rb

@@ -0,0 +1,207 @@
+# frozen_string_literal: true
+
+describe "OracleEnhancedAdapter boolean type detection based on string column types and names" do
+  before(:all) do
+    ActiveRecord::Base.establish_connection(CONNECTION_PARAMS)
+    @conn = ActiveRecord::Base.connection
+    @conn.execute <<-SQL
+      CREATE TABLE test3_employees (
+        id            NUMBER PRIMARY KEY,
+        first_name    VARCHAR2(20),
+        last_name     VARCHAR2(25),
+        email         VARCHAR2(25),
+        phone_number  VARCHAR2(20),
+        hire_date     DATE,
+        job_id        NUMBER,
+        salary        NUMBER,
+        commission_pct  NUMBER(2,2),
+        manager_id    NUMBER(6),
+        department_id NUMBER(4,0),
+        created_at    DATE,
+        has_email     CHAR(1),
+        has_phone     VARCHAR2(1) DEFAULT 'Y',
+        active_flag   VARCHAR2(2),
+        manager_yn    VARCHAR2(3) DEFAULT 'N',
+        test_boolean  VARCHAR2(3)
+      )
+    SQL
+    @conn.execute <<-SQL
+      CREATE SEQUENCE test3_employees_seq  MINVALUE 1
+        INCREMENT BY 1 START WITH 10040 CACHE 20 NOORDER NOCYCLE
+    SQL
+  end
+
+  after(:all) do
+    @conn.execute "DROP TABLE test3_employees"
+    @conn.execute "DROP SEQUENCE test3_employees_seq"
+  end
+
+  before(:each) do
+    class ::Test3Employee < ActiveRecord::Base
+    end
+  end
+
+  after(:each) do
+    Object.send(:remove_const, "Test3Employee")
+    ActiveRecord::Base.clear_cache!
+  end
+
+  describe "default values in new records" do
+    context "when emulate_booleans_from_strings is false" do
+      before do
+        ActiveRecord::ConnectionAdapters::OracleEnhancedAdapter.emulate_booleans_from_strings = false
+      end
+
+      it "are Y or N" do
+        subject = Test3Employee.new
+        expect(subject.has_phone).to eq("Y")
+        expect(subject.manager_yn).to eq("N")
+      end
+    end
+
+    context "when emulate_booleans_from_strings is true" do
+      before do
+        ActiveRecord::ConnectionAdapters::OracleEnhancedAdapter.emulate_booleans_from_strings = true
+      end
+
+      it "are True or False" do
+        class ::Test3Employee < ActiveRecord::Base
+          attribute :has_phone, :boolean
+          attribute :manager_yn, :boolean, default: false
+        end
+        subject = Test3Employee.new
+        expect(subject.has_phone).to be_a(TrueClass)
+        expect(subject.manager_yn).to be_a(FalseClass)
+      end
+    end
+  end
+
+  it "should translate boolean type to NUMBER(1) if emulate_booleans_from_strings is false" do
+    ActiveRecord::ConnectionAdapters::OracleEnhancedAdapter.emulate_booleans_from_strings = false
+    sql_type = ActiveRecord::Base.connection.type_to_sql(:boolean)
+    expect(sql_type).to eq("NUMBER(1)")
+  end
+
+  describe "/ VARCHAR2 boolean values from ActiveRecord model" do
+    before(:each) do
+      ActiveRecord::ConnectionAdapters::OracleEnhancedAdapter.emulate_booleans_from_strings = false
+    end
+
+    after(:each) do
+      ActiveRecord::Base.clear_cache!
+    end
+
+    def create_employee3(params = {})
+      @employee3 = Test3Employee.create(
+        {
+        first_name: "First",
+        last_name: "Last",
+        has_email: true,
+        has_phone: false,
+        active_flag: true,
+        manager_yn: false
+        }.merge(params)
+      )
+      @employee3.reload
+    end
+
+    it "should return String value from VARCHAR2 boolean column if emulate_booleans_from_strings is false" do
+      ActiveRecord::ConnectionAdapters::OracleEnhancedAdapter.emulate_booleans_from_strings = false
+      create_employee3
+      %w(has_email has_phone active_flag manager_yn).each do |col|
+        expect(@employee3.send(col.to_sym).class).to eq(String)
+      end
+    end
+
+    it "should return boolean value from VARCHAR2 boolean column if emulate_booleans_from_strings is true" do
+      ActiveRecord::ConnectionAdapters::OracleEnhancedAdapter.emulate_booleans_from_strings = true
+      class ::Test3Employee < ActiveRecord::Base
+        attribute :has_email, :boolean
+        attribute :active_flag, :boolean
+        attribute :has_phone, :boolean, default: false
+        attribute :manager_yn, :boolean, default: false
+      end
+      create_employee3
+      %w(has_email active_flag).each do |col|
+        expect(@employee3.send(col.to_sym).class).to eq(TrueClass)
+        expect(@employee3.send((col + "_before_type_cast").to_sym)).to eq("Y")
+      end
+      %w(has_phone manager_yn).each do |col|
+        expect(@employee3.send(col.to_sym).class).to eq(FalseClass)
+        expect(@employee3.send((col + "_before_type_cast").to_sym)).to eq("N")
+      end
+    end
+
+    it "should return string value from VARCHAR2 column if it is not boolean column and emulate_booleans_from_strings is true" do
+      ActiveRecord::ConnectionAdapters::OracleEnhancedAdapter.emulate_booleans_from_strings = true
+      create_employee3
+      expect(@employee3.first_name.class).to eq(String)
+    end
+
+    it "should return boolean value from VARCHAR2 boolean column if attribute is set to :boolean" do
+      ActiveRecord::ConnectionAdapters::OracleEnhancedAdapter.emulate_booleans_from_strings = true
+      class ::Test3Employee < ActiveRecord::Base
+        attribute :test_boolean, :boolean
+      end
+      create_employee3(test_boolean: true)
+      expect(@employee3.test_boolean.class).to eq(TrueClass)
+      expect(@employee3.test_boolean_before_type_cast).to eq("Y")
+      create_employee3(test_boolean: false)
+      expect(@employee3.test_boolean.class).to eq(FalseClass)
+      expect(@employee3.test_boolean_before_type_cast).to eq("N")
+      create_employee3(test_boolean: nil)
+      expect(@employee3.test_boolean.class).to eq(NilClass)
+      expect(@employee3.test_boolean_before_type_cast).to eq(nil)
+      create_employee3(test_boolean: "")
+      expect(@employee3.test_boolean.class).to eq(NilClass)
+      expect(@employee3.test_boolean_before_type_cast).to eq(nil)
+    end
+
+    it "should return string value from VARCHAR2 column with boolean column name but attribute is set to :string" do
+      ActiveRecord::ConnectionAdapters::OracleEnhancedAdapter.emulate_booleans_from_strings = true
+      class ::Test3Employee < ActiveRecord::Base
+        attribute :active_flag, :string
+      end
+      create_employee3
+      expect(@employee3.active_flag.class).to eq(String)
+    end
+
+  end
+
+end
+
+describe "OracleEnhancedAdapter boolean support when emulate_booleans_from_strings = true" do
+  include SchemaSpecHelper
+
+  before(:all) do
+    ActiveRecord::ConnectionAdapters::OracleEnhancedAdapter.emulate_booleans_from_strings = true
+    ActiveRecord::Base.establish_connection(CONNECTION_PARAMS)
+    schema_define do
+      create_table :posts, force: true do |t|
+        t.string  :name,        null: false
+        t.boolean :is_default, default: false
+      end
+    end
+  end
+
+  after(:all) do
+    ActiveRecord::ConnectionAdapters::OracleEnhancedAdapter.emulate_booleans_from_strings = false
+  end
+
+  before(:each) do
+    class ::Post < ActiveRecord::Base
+    end
+  end
+
+  after(:each) do
+    Object.send(:remove_const, "Post")
+    ActiveRecord::Base.clear_cache!
+  end
+
+  it "boolean should not change after reload" do
+    post = Post.create(name: "Test 1", is_default: false)
+    expect(post.is_default).to be false
+    post.reload
+    expect(post.is_default).to be false
+  end
+end

data/spec/active_record/{connection_adapters/oracle_enhanced_dirty_spec.rb → oracle_enhanced/type/dirty_spec.rb}

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 describe "OracleEnhancedAdapter dirty object tracking" do
   include SchemaSpecHelper
 
@@ -113,7 +115,7 @@ describe "OracleEnhancedAdapter dirty object tracking" do
     @conn = nil
     @connection = nil
     @employee = TestEmployee.create!(
-        comments: "initial"
+      comments: "initial"
     )
     expect(@employee.save!).to be_truthy
     @employee.reload

data/spec/active_record/oracle_enhanced/type/float_spec.rb

@@ -0,0 +1,48 @@
+# frozen_string_literal: true
+
+describe "OracleEnhancedAdapter handling of BINARY_FLOAT columns" do
+  include SchemaSpecHelper
+
+  before(:all) do
+    ActiveRecord::Base.establish_connection(CONNECTION_PARAMS)
+    @conn = ActiveRecord::Base.connection
+    schema_define do
+      create_table :test2_employees, force: true do |t|
+        t.string  :first_name, limit: 20
+        t.string  :last_name, limit: 25
+        t.string  :email, limit: 25
+        t.string  :phone_number, limit: 25
+        t.date    :hire_date
+        t.integer :job_id
+        t.integer :salary
+        t.decimal :commission_pct, scale: 2, precision: 2
+        t.float   :hourly_rate
+        t.integer :manager_id,  limit: 6
+        t.integer :is_manager,  limit: 1
+        t.decimal :department_id, scale: 0, precision: 4
+        t.timestamps
+      end
+    end
+    class ::Test2Employee < ActiveRecord::Base
+    end
+  end
+
+  after(:all) do
+    Object.send(:remove_const, "Test2Employee")
+    @conn.drop_table :test2_employees, if_exists:  true
+  end
+
+  it "should set BINARY_FLOAT column type as float" do
+    columns = @conn.columns("test2_employees")
+    column = columns.detect { |c| c.name == "hourly_rate" }
+    expect(column.type).to eq(:float)
+  end
+
+  it "should BINARY_FLOAT column type returns an approximate value" do
+    employee = Test2Employee.create(hourly_rate: 4.4)
+
+    employee.reload
+
+    expect(employee.hourly_rate).to eq(4.400000095367432)
+  end
+end

data/spec/active_record/oracle_enhanced/type/integer_spec.rb

@@ -0,0 +1,91 @@
+# frozen_string_literal: true
+
+describe "OracleEnhancedAdapter integer type detection based on attribute settings" do
+  before(:all) do
+    ActiveRecord::Base.establish_connection(CONNECTION_PARAMS)
+    @conn = ActiveRecord::Base.connection
+    @conn.execute "DROP TABLE test2_employees" rescue nil
+    @conn.execute <<-SQL
+      CREATE TABLE test2_employees (
+        id            NUMBER PRIMARY KEY,
+        first_name    VARCHAR2(20),
+        last_name     VARCHAR2(25),
+        email         VARCHAR2(25),
+        phone_number  VARCHAR2(20),
+        hire_date     DATE,
+        job_id        NUMBER,
+        salary        NUMBER,
+        commission_pct  NUMBER(2,2),
+        manager_id    NUMBER(6),
+        is_manager    NUMBER(1),
+        department_id NUMBER(4,0),
+        created_at    DATE
+      )
+    SQL
+    @conn.execute "DROP SEQUENCE test2_employees_seq" rescue nil
+    @conn.execute <<-SQL
+      CREATE SEQUENCE test2_employees_seq  MINVALUE 1
+        INCREMENT BY 1 START WITH 10040 CACHE 20 NOORDER NOCYCLE
+    SQL
+  end
+
+  after(:all) do
+    @conn.execute "DROP TABLE test2_employees"
+    @conn.execute "DROP SEQUENCE test2_employees_seq"
+  end
+
+  describe "/ NUMBER values from ActiveRecord model" do
+    before(:each) do
+      class ::Test2Employee < ActiveRecord::Base
+      end
+    end
+
+    after(:each) do
+      Object.send(:remove_const, "Test2Employee")
+      ActiveRecord::ConnectionAdapters::OracleEnhancedAdapter.emulate_booleans = true
+      ActiveRecord::Base.clear_cache!
+    end
+
+    def create_employee2
+      @employee2 = Test2Employee.create(
+        first_name: "First",
+        last_name: "Last",
+        job_id: 1,
+        is_manager: 1,
+        salary: 1000
+      )
+      @employee2.reload
+    end
+
+    it "should return BigDecimal value from NUMBER column if by default" do
+      create_employee2
+      expect(@employee2.job_id.class).to eq(BigDecimal)
+    end
+
+    it "should return Integer value from NUMBER column if attribute is set to integer" do
+      class ::Test2Employee < ActiveRecord::Base
+        attribute :job_id, :integer
+      end
+      create_employee2
+      expect(@employee2.job_id).to be_a(Integer)
+    end
+
+    it "should return Integer value from NUMBER column with integer value using _before_type_cast method" do
+      create_employee2
+      expect(@employee2.job_id_before_type_cast).to be_a(Integer)
+    end
+
+    it "should return Boolean value from NUMBER(1) column if emulate booleans is used" do
+      create_employee2
+      expect(@employee2.is_manager.class).to eq(TrueClass)
+    end
+
+    it "should return Integer value from NUMBER(1) column if attribute is set to integer" do
+      class ::Test2Employee < ActiveRecord::Base
+        attribute :is_manager, :integer
+      end
+      create_employee2
+      expect(@employee2.is_manager).to be_a(Integer)
+    end
+  end
+end