rsim-activerecord-oracle_enhanced-adapter 1.1.9.90

data/spec/active_record/connection_adapters/oracle_enhanced_dirty_spec.rb

@@ -0,0 +1,82 @@
+require File.dirname(__FILE__) + '/../../spec_helper.rb'
+
+if ActiveRecord::Base.instance_methods.include?('changed?')
+
+  describe "OracleEnhancedAdapter dirty object tracking" do
+
+    before(:all) do
+      ActiveRecord::Base.establish_connection(CONNECTION_PARAMS)
+      @conn = ActiveRecord::Base.connection
+      @conn.execute <<-SQL
+        CREATE TABLE test_employees (
+          id            NUMBER,
+          first_name    VARCHAR2(20),
+          last_name     VARCHAR2(25),
+          job_id        NUMBER(6,0),
+          salary        NUMBER(8,2),
+          comments      CLOB,
+          hire_date     DATE
+        )
+      SQL
+      @conn.execute <<-SQL
+        CREATE SEQUENCE test_employees_seq  MINVALUE 1
+          INCREMENT BY 1 CACHE 20 NOORDER NOCYCLE
+      SQL
+      class TestEmployee < ActiveRecord::Base
+      end
+    end
+  
+    after(:all) do
+      Object.send(:remove_const, "TestEmployee")
+      @conn.execute "DROP TABLE test_employees"
+      @conn.execute "DROP SEQUENCE test_employees_seq"
+    end  
+
+    it "should not mark empty string (stored as NULL) as changed when reassigning it" do
+      @employee = TestEmployee.create!(:first_name => '')
+      @employee.first_name = ''
+      @employee.should_not be_changed
+      @employee.reload
+      @employee.first_name = ''
+      @employee.should_not be_changed
+    end
+
+    it "should not mark empty integer (stored as NULL) as changed when reassigning it" do
+      @employee = TestEmployee.create!(:job_id => '')
+      @employee.job_id = ''
+      @employee.should_not be_changed
+      @employee.reload
+      @employee.job_id = ''
+      @employee.should_not be_changed
+    end
+
+    it "should not mark empty decimal (stored as NULL) as changed when reassigning it" do
+      @employee = TestEmployee.create!(:salary => '')
+      @employee.salary = ''
+      @employee.should_not be_changed
+      @employee.reload
+      @employee.salary = ''
+      @employee.should_not be_changed
+    end
+
+    it "should not mark empty text (stored as NULL) as changed when reassigning it" do
+      @employee = TestEmployee.create!(:comments => '')
+      @employee.comments = ''
+      @employee.should_not be_changed
+      @employee.reload
+      @employee.comments = ''
+      @employee.should_not be_changed
+    end
+
+    it "should not mark empty date (stored as NULL) as changed when reassigning it" do
+      @employee = TestEmployee.create!(:hire_date => '')
+      @employee.hire_date = ''
+      @employee.should_not be_changed
+      @employee.reload
+      @employee.hire_date = ''
+      @employee.should_not be_changed
+    end
+
+  end
+
+end

data/spec/active_record/connection_adapters/oracle_enhanced_emulate_oracle_adapter_spec.rb

@@ -0,0 +1,25 @@
+require File.dirname(__FILE__) + '/../../spec_helper.rb'
+
+
+
+describe "OracleEnhancedAdapter emulate OracleAdapter" do
+
+  before(:all) do
+    if defined?(ActiveRecord::ConnectionAdapters::OracleAdapter)
+      ActiveRecord::ConnectionAdapters.send(:remove_const, :OracleAdapter)
+    end
+  end
+
+  it "should be an OracleAdapter" do
+    @conn = ActiveRecord::Base.establish_connection(CONNECTION_PARAMS.merge(:emulate_oracle_adapter => true))
+    ActiveRecord::Base.connection.should_not be_nil
+    ActiveRecord::Base.connection.is_a?(ActiveRecord::ConnectionAdapters::OracleAdapter).should be_true
+  end
+
+  after(:all) do
+    if defined?(ActiveRecord::ConnectionAdapters::OracleAdapter)
+      ActiveRecord::ConnectionAdapters.send(:remove_const, :OracleAdapter)
+    end    
+  end
+
+end

data/spec/active_record/connection_adapters/oracle_enhanced_procedures_spec.rb

@@ -0,0 +1,276 @@
+require File.dirname(__FILE__) + '/../../spec_helper.rb'
+
+describe "OracleEnhancedAdapter custom methods for create, update and destroy" do
+  include LoggerSpecHelper
+  
+  before(:all) do
+    ActiveRecord::Base.establish_connection(CONNECTION_PARAMS)
+    @conn = ActiveRecord::Base.connection
+    plsql.connection = ActiveRecord::Base.connection.raw_connection
+    @conn.execute("DROP TABLE test_employees") rescue nil
+    @conn.execute <<-SQL
+      CREATE TABLE test_employees (
+        employee_id   NUMBER(6,0),
+        first_name    VARCHAR2(20),
+        last_name     VARCHAR2(25),
+        hire_date     DATE,
+        salary        NUMBER(8,2),
+        description   CLOB,
+        version       NUMBER(15,0),
+        create_time   DATE,
+        update_time   DATE
+      )
+    SQL
+    @conn.execute("DROP SEQUENCE test_employees_s") rescue nil
+    @conn.execute <<-SQL
+      CREATE SEQUENCE test_employees_s  MINVALUE 1
+        INCREMENT BY 1 CACHE 20 NOORDER NOCYCLE
+    SQL
+    @conn.execute <<-SQL
+      CREATE OR REPLACE PACKAGE test_employees_pkg IS
+        PROCEDURE create_employee(
+            p_first_name    VARCHAR2,
+            p_last_name     VARCHAR2,
+            p_hire_date     DATE,
+            p_salary        NUMBER,
+            p_description   VARCHAR2,
+            p_employee_id   OUT NUMBER);
+        PROCEDURE update_employee(
+            p_employee_id   NUMBER,
+            p_first_name    VARCHAR2,
+            p_last_name     VARCHAR2,
+            p_hire_date     DATE,
+            p_salary        NUMBER,
+            p_description   VARCHAR2);
+        PROCEDURE delete_employee(
+            p_employee_id   NUMBER);
+      END;
+    SQL
+    @conn.execute <<-SQL
+      CREATE OR REPLACE PACKAGE BODY test_employees_pkg IS
+        PROCEDURE create_employee(
+            p_first_name    VARCHAR2,
+            p_last_name     VARCHAR2,
+            p_hire_date     DATE,
+            p_salary        NUMBER,
+            p_description   VARCHAR2,
+            p_employee_id   OUT NUMBER)
+        IS
+        BEGIN
+          SELECT test_employees_s.NEXTVAL INTO p_employee_id FROM dual;
+          INSERT INTO test_employees (employee_id, first_name, last_name, hire_date, salary, description,
+                                      version, create_time, update_time)
+          VALUES (p_employee_id, p_first_name, p_last_name, p_hire_date, p_salary, p_description,
+                                      1, SYSDATE, SYSDATE);
+        END create_employee;
+        
+        PROCEDURE update_employee(
+            p_employee_id   NUMBER,
+            p_first_name    VARCHAR2,
+            p_last_name     VARCHAR2,
+            p_hire_date     DATE,
+            p_salary        NUMBER,
+            p_description   VARCHAR2)
+        IS
+            v_version       NUMBER;
+        BEGIN
+          SELECT version INTO v_version FROM test_employees WHERE employee_id = p_employee_id FOR UPDATE;
+          UPDATE test_employees
+          SET employee_id = p_employee_id, first_name = p_first_name, last_name = p_last_name,
+              hire_date = p_hire_date, salary = p_salary, description = p_description,
+              version = v_version + 1, update_time = SYSDATE;
+        END update_employee;
+        
+        PROCEDURE delete_employee(
+            p_employee_id   NUMBER)
+        IS
+        BEGIN
+          DELETE FROM test_employees WHERE employee_id = p_employee_id;
+        END delete_employee;
+      END;
+    SQL
+
+    ActiveRecord::ConnectionAdapters::OracleEnhancedAdapter.emulate_dates_by_column_name = true
+
+    class TestEmployee < ActiveRecord::Base
+      set_primary_key :employee_id
+      
+      validates_presence_of :first_name, :last_name, :hire_date
+      
+      # should return ID of new record
+      set_create_method do
+        plsql.test_employees_pkg.create_employee(
+          :p_first_name => first_name,
+          :p_last_name => last_name,
+          :p_hire_date => hire_date,
+          :p_salary => salary,
+          :p_description => "#{first_name} #{last_name}",
+          :p_employee_id => nil
+        )[:p_employee_id]
+      end
+
+      # return value is ignored
+      set_update_method do
+        plsql.test_employees_pkg.update_employee(
+          :p_employee_id => id,
+          :p_first_name => first_name,
+          :p_last_name => last_name,
+          :p_hire_date => hire_date,
+          :p_salary => salary,
+          :p_description => "#{first_name} #{last_name}"
+        )
+      end
+
+      # return value is ignored
+      set_delete_method do
+        plsql.test_employees_pkg.delete_employee(
+          :p_employee_id => id
+        )
+      end
+
+    end
+  end
+  
+  after(:all) do
+    Object.send(:remove_const, "TestEmployee")
+    @conn = ActiveRecord::Base.connection    
+    @conn.execute "DROP TABLE test_employees"
+    @conn.execute "DROP SEQUENCE test_employees_s"
+    @conn.execute "DROP PACKAGE test_employees_pkg"
+  end
+
+  before(:each) do
+    @today = Date.new(2008,6,28)
+    @buffer = StringIO.new
+  end
+
+  it "should create record" do
+    @employee = TestEmployee.create(
+      :first_name => "First",
+      :last_name => "Last",
+      :hire_date => @today
+    )
+    @employee.reload
+    @employee.first_name.should == "First"
+    @employee.last_name.should == "Last"
+    @employee.hire_date.should == @today
+    @employee.description.should == "First Last"
+    @employee.create_time.should_not be_nil
+    @employee.update_time.should_not be_nil
+  end
+
+  it "should update record" do
+    @employee = TestEmployee.create(
+      :first_name => "First",
+      :last_name => "Last",
+      :hire_date => @today,
+      :description => "description"
+    )
+    @employee.reload
+    @employee.first_name = "Second"
+    @employee.save!
+    @employee.reload
+    @employee.description.should == "Second Last"
+  end
+
+  it "should not update record if nothing is changed and partial updates are enabled" do
+    return pending("Not in this ActiveRecord version") unless TestEmployee.respond_to?(:partial_updates=)
+    TestEmployee.partial_updates = true
+    @employee = TestEmployee.create(
+      :first_name => "First",
+      :last_name => "Last",
+      :hire_date => @today
+    )
+    @employee.reload
+    @employee.save!
+    @employee.reload
+    @employee.version.should == 1
+  end
+
+  it "should update record if nothing is changed and partial updates are disabled" do
+    return pending("Not in this ActiveRecord version") unless TestEmployee.respond_to?(:partial_updates=)
+    TestEmployee.partial_updates = false
+    @employee = TestEmployee.create(
+      :first_name => "First",
+      :last_name => "Last",
+      :hire_date => @today
+    )
+    @employee.reload
+    @employee.save!
+    @employee.reload
+    @employee.version.should == 2
+  end
+
+  it "should delete record" do
+    @employee = TestEmployee.create(
+      :first_name => "First",
+      :last_name => "Last",
+      :hire_date => @today
+    )
+    @employee.reload
+    empl_id = @employee.id
+    @employee.destroy
+    @employee.should be_frozen
+    TestEmployee.find_by_employee_id(empl_id).should be_nil
+  end
+
+  it "should log create record" do
+    log_to @buffer
+    # reestablish plsql.connection as log_to might reset existing connection
+    plsql.connection = ActiveRecord::Base.connection.raw_connection
+    @employee = TestEmployee.create(
+      :first_name => "First",
+      :last_name => "Last",
+      :hire_date => @today
+    )
+    @buffer.string.should match(/^TestEmployee Create \(\d+\.\d+(ms)?\)  custom create method$/)
+  end
+
+  it "should log update record" do
+    (TestEmployee.partial_updates = false) rescue nil
+    @employee = TestEmployee.create(
+      :first_name => "First",
+      :last_name => "Last",
+      :hire_date => @today
+    )
+    log_to @buffer
+    # reestablish plsql.connection as log_to might reset existing connection
+    plsql.connection = ActiveRecord::Base.connection.raw_connection
+    @employee.save!
+    @buffer.string.should match(/^TestEmployee Update \(\d+\.\d+(ms)?\)  custom update method with employee_id=#{@employee.id}$/)
+  end
+
+  it "should log delete record" do
+    @employee = TestEmployee.create(
+      :first_name => "First",
+      :last_name => "Last",
+      :hire_date => @today
+    )
+    log_to @buffer
+    # reestablish plsql.connection as log_to might reset existing connection
+    plsql.connection = ActiveRecord::Base.connection.raw_connection
+    @employee.destroy
+    @buffer.string.should match(/^TestEmployee Destroy \(\d+\.\d+(ms)?\)  custom delete method with employee_id=#{@employee.id}$/)
+  end
+
+  it "should validate new record before creation" do
+    @employee = TestEmployee.new(
+      :last_name => "Last",
+      :hire_date => @today
+    )
+    @employee.save.should be_false
+    @employee.errors.on(:first_name).should_not be_nil
+  end
+
+  it "should validate existing record before update" do
+    @employee = TestEmployee.create(
+      :first_name => "First",
+      :last_name => "Last",
+      :hire_date => @today
+    )
+    @employee.first_name = nil
+    @employee.save.should be_false
+    @employee.errors.on(:first_name).should_not be_nil
+  end
+  
+end

data/spec/spec.opts

@@ -0,0 +1,6 @@
+--colour
+--format
+progress
+--loadby
+mtime
+--reverse

data/spec/spec_helper.rb

@@ -0,0 +1,73 @@
+require 'rubygems'
+gem 'rspec'
+require 'spec'
+
+$:.unshift(File.dirname(__FILE__) + '/../lib')
+
+if ENV['RAILS_GEM_VERSION'] =~ /^2.0/
+  gem 'activerecord', '=2.0.2'
+  gem 'actionpack', '=2.0.2'
+  gem 'activesupport', '=2.0.2'
+  gem 'composite_primary_keys', '=0.9.93'
+elsif ENV['RAILS_GEM_VERSION'] =~ /^2.1/
+  gem 'activerecord', '=2.1.2'
+  gem 'actionpack', '=2.1.2'
+  gem 'activesupport', '=2.1.2'
+  gem 'composite_primary_keys', '=1.0.8'
+else
+  gem 'activerecord', '=2.2.2'
+  gem 'actionpack', '=2.2.2'
+  gem 'activesupport', '=2.2.2'
+  gem 'composite_primary_keys', '=2.2.0'
+end
+
+require 'activerecord'
+require 'actionpack'
+require 'action_controller/session/active_record_store'
+if !defined?(RUBY_ENGINE)
+  gem "activerecord-oracle-adapter"
+  require 'active_record/connection_adapters/oracle_adapter'
+elsif RUBY_ENGINE == 'jruby'
+  gem "activerecord-jdbc-adapter"
+  require 'active_record/connection_adapters/jdbc_adapter'
+end
+
+require 'active_record/connection_adapters/oracle_enhanced_adapter'
+
+module LoggerSpecHelper
+  def log_to(stream)
+    ActiveRecord::Base.logger = Logger.new(stream)
+    if ActiveRecord::Base.respond_to?(:connection_pool)
+      ActiveRecord::Base.connection_pool.clear_reloadable_connections!
+    else
+      ActiveRecord::Base.clear_active_connections!
+    end
+    ActiveRecord::Base.colorize_logging = false
+    ActiveRecord::Base.logger.level = Logger::DEBUG
+  end
+end
+
+CONNECTION_PARAMS = {
+  :adapter => "oracle_enhanced",
+  :database => "xe",
+  :host => "ubuntu810",
+  :username => "hr",
+  :password => "hr"
+}
+
+JDBC_CONNECTION_PARAMS = {
+  :adapter => "jdbc",
+  :driver => "oracle.jdbc.driver.OracleDriver",
+  :url => "jdbc:oracle:thin:@ubuntu810:1521:XE",
+  :username => "hr",
+  :password => "hr"
+}
+
+SYS_CONNECTION_PARAMS = {
+  :adapter => "oracle_enhanced",
+  :database => "xe",
+  :host => "ubuntu810",
+  :username => "sys",
+  :password => "manager",
+  :privilege => "SYSDBA"
+}