unclebilly-activerecord-oracle_enhanced-adapter 1.2.4

data/spec/active_record/connection_adapters/oracle_enhanced_adapter_structure_dumper_spec.rb

@@ -0,0 +1,266 @@
+require File.expand_path(File.dirname(__FILE__) + '/../../spec_helper')
+
+describe "OracleEnhancedAdapter structure dump" do
+  include LoggerSpecHelper
+
+  before(:all) do
+    ActiveRecord::Base.establish_connection(CONNECTION_PARAMS)
+    @conn = ActiveRecord::Base.connection
+  end
+  describe "structure dump" do
+    before(:each) do
+      @conn.create_table :test_posts, :force => true do |t|
+        t.string      :title
+        t.string      :foo
+        t.integer     :foo_id
+      end
+      @conn.create_table :foos do |t|
+      end
+      class ::TestPost < ActiveRecord::Base
+      end
+      TestPost.set_table_name "test_posts"
+    end
+  
+    after(:each) do
+      @conn.drop_table :test_posts 
+      @conn.drop_table :foos
+      @conn.execute "DROP SEQUENCE test_posts_seq" rescue nil
+      @conn.execute "ALTER TABLE test_posts drop CONSTRAINT fk_test_post_foo" rescue nil
+      @conn.execute "DROP TRIGGER test_post_trigger" rescue nil
+      @conn.execute "DROP TYPE TEST_TYPE" rescue nil
+      @conn.execute "DROP TABLE bars" rescue nil
+    end
+  
+    it "should dump single primary key" do
+      dump = ActiveRecord::Base.connection.structure_dump
+      dump.should =~ /CONSTRAINT (.+) PRIMARY KEY \(ID\)\n/
+    end
+  
+    it "should dump composite primary keys" do
+      pk = @conn.send(:select_one, <<-SQL)
+        select constraint_name from user_constraints where table_name = 'TEST_POSTS' and constraint_type='P'
+      SQL
+      @conn.execute <<-SQL
+        alter table test_posts drop constraint #{pk["constraint_name"]}
+      SQL
+      @conn.execute <<-SQL
+        ALTER TABLE TEST_POSTS
+        add CONSTRAINT pk_id_title PRIMARY KEY (id, title)
+      SQL
+      dump = ActiveRecord::Base.connection.structure_dump
+      dump.should =~ /CONSTRAINT (.+) PRIMARY KEY \(ID,TITLE\)\n/
+    end
+  
+    it "should dump foreign keys" do
+      @conn.execute <<-SQL
+        ALTER TABLE TEST_POSTS 
+        ADD CONSTRAINT fk_test_post_foo FOREIGN KEY (foo_id) REFERENCES foos(id)
+      SQL
+      dump = ActiveRecord::Base.connection.structure_dump_fk_constraints
+      dump.split('\n').length.should == 1
+      dump.should =~ /ALTER TABLE TEST_POSTS ADD CONSTRAINT fk_test_post_foo FOREIGN KEY \(foo_id\) REFERENCES foos\(id\)/
+    end
+  
+    it "should not error when no foreign keys are present" do
+      dump = ActiveRecord::Base.connection.structure_dump_fk_constraints
+      dump.split('\n').length.should == 0
+      dump.should == ''
+    end
+  
+    it "should dump triggers" do
+      @conn.execute <<-SQL
+        create or replace TRIGGER TEST_POST_TRIGGER
+          BEFORE INSERT
+          ON TEST_POSTS
+          FOR EACH ROW
+        BEGIN
+          SELECT 'bar' INTO :new.FOO FROM DUAL;
+        END;
+      SQL
+      dump = ActiveRecord::Base.connection.structure_dump_db_stored_code.gsub(/\n|\s+/,' ')
+      dump.should =~ /create or replace TRIGGER TEST_POST_TRIGGER/
+    end
+  
+    it "should dump types" do
+      @conn.execute <<-SQL
+        create or replace TYPE TEST_TYPE AS TABLE OF VARCHAR2(10);
+      SQL
+      dump = ActiveRecord::Base.connection.structure_dump_db_stored_code.gsub(/\n|\s+/,' ')
+      dump.should =~ /create or replace TYPE TEST_TYPE/
+    end
+  
+    it "should dump virtual columns" do
+      @conn.execute <<-SQL
+        CREATE TABLE bars (
+          id          NUMBER(38,0) NOT NULL,
+          id_plus     NUMBER GENERATED ALWAYS AS(id + 2) VIRTUAL,
+          PRIMARY KEY (ID)
+        )
+      SQL
+      dump = ActiveRecord::Base.connection.structure_dump
+      dump.should =~ /id_plus number GENERATED ALWAYS AS \(ID\+2\) VIRTUAL/
+    end
+  
+    it "should dump unique keys" do
+      @conn.execute <<-SQL
+        ALTER TABLE test_posts
+          add CONSTRAINT uk_foo_foo_id UNIQUE (foo, foo_id)
+      SQL
+      dump = ActiveRecord::Base.connection.structure_dump_unique_keys("test_posts")
+      dump.should == [" CONSTRAINT UK_FOO_FOO_ID UNIQUE (FOO,FOO_ID)"]
+    
+      dump = ActiveRecord::Base.connection.structure_dump
+      dump.should =~ /CONSTRAINT UK_FOO_FOO_ID UNIQUE \(FOO,FOO_ID\)/
+    end
+  
+    it "should dump indexes" do
+      ActiveRecord::Base.connection.add_index(:test_posts, :foo, :name => :ix_test_posts_foo)
+      ActiveRecord::Base.connection.add_index(:test_posts, :foo_id, :name => :ix_test_posts_foo_id, :unique => true)
+      
+      @conn.execute <<-SQL
+        ALTER TABLE test_posts
+          add CONSTRAINT uk_foo_foo_id UNIQUE (foo, foo_id)
+      SQL
+      
+      dump = ActiveRecord::Base.connection.structure_dump
+      dump.should =~ /create unique index ix_test_posts_foo_id on test_posts \(foo_id\)/i
+      dump.should =~ /create  index ix_test_posts_foo on test_posts \(foo\)/i
+      dump.should_not =~ /create unique index uk_test_posts_/i
+    end
+  end
+  describe "temporary tables" do
+    after(:all) do
+      @conn.drop_table :test_comments rescue nil
+    end
+    it "should dump correctly" do
+      @conn.create_table :test_comments, :temporary => true, :id => false do |t|
+        t.integer :post_id
+      end
+      dump = ActiveRecord::Base.connection.structure_dump
+      dump.should =~ /create global temporary table test_comments/i
+    end
+  end
+
+  describe "database stucture dump extentions" do
+    before(:all) do
+      @conn.execute <<-SQL
+        CREATE TABLE nvarchartable (
+          unq_nvarchar  NVARCHAR2(255) DEFAULT NULL
+        )
+      SQL
+    end
+
+    after(:all) do
+      @conn.execute "DROP TABLE nvarchartable"
+    end
+
+    it "should return the character size of nvarchar fields" do
+      if /.*unq_nvarchar nvarchar2\((\d+)\).*/ =~ @conn.structure_dump
+         "#$1".should == "255"
+      end
+    end
+  end
+  
+  describe "temp_table_drop" do
+    before(:each) do
+      @conn.create_table :temp_tbl, :temporary => true do |t|
+        t.string :foo
+      end
+      @conn.create_table :not_temp_tbl do |t|
+        t.string :foo
+      end
+    end
+    it "should dump drop sql for just temp tables" do
+      dump = @conn.temp_table_drop
+      dump.should =~ /drop table temp_tbl/i
+      dump.should_not =~ /drop table not_temp_tbl/i
+    end
+    after(:each) do
+      @conn.drop_table :temp_tbl 
+      @conn.drop_table :not_temp_tbl
+    end
+  end
+  
+  describe "full drop" do
+    before(:each) do 
+      @conn.create_table :full_drop_test do |t|
+        t.integer :id
+      end
+      @conn.create_table :full_drop_test_temp, :temporary => true do |t|
+        t.string :foo
+      end
+      #view
+      @conn.execute <<-SQL
+        create or replace view full_drop_test_view (foo) as select id as "foo" from full_drop_test
+      SQL
+      #package
+      @conn.execute <<-SQL
+        create or replace package full_drop_test_package as
+          function test_func return varchar2;
+        end test_package;
+      SQL
+      @conn.execute <<-SQL
+        create or replace package body full_drop_test_package as 
+          function test_func return varchar2 is
+            begin
+              return ('foo');
+          end test_func;
+        end test_package;
+      SQL
+      #function
+      @conn.execute <<-SQL
+        create or replace function full_drop_test_function
+          return varchar2 
+        is
+          foo varchar2(3);
+        begin 
+          return('foo');
+        end;
+      SQL
+      #procedure
+      @conn.execute <<-SQL
+        create or replace procedure full_drop_test_procedure
+        begin
+          delete from full_drop_test where id=1231231231
+        exception
+        when no_data_found then
+          dbms_output.put_line('foo');
+        end;
+      SQL
+      #synonym
+      @conn.execute <<-SQL
+        create or replace synonym full_drop_test_synonym for full_drop_test
+      SQL
+      #type
+      @conn.execute <<-SQL
+        create or replace type full_drop_test_type as table of number
+      SQL
+    end
+    after(:each) do
+      @conn.drop_table :full_drop_test
+      @conn.drop_table :full_drop_test_temp
+      @conn.execute "DROP VIEW FULL_DROP_TEST_VIEW" rescue nil
+      @conn.execute "DROP SYNONYM FULL_DROP_TEST_SYNONYM" rescue nil
+      @conn.execute "DROP PACKAGE FULL_DROP_TEST_PACKAGE" rescue nil
+      @conn.execute "DROP FUNCTION FULL_DROP_TEST_FUNCTION" rescue nil
+      @conn.execute "DROP PROCEDURE FULL_DROP_TEST_PROCEDURE" rescue nil
+      @conn.execute "DROP TYPE FULL_DROP_TEST_TYPE" rescue nil
+    end
+    it "should contain correct sql" do
+      drop = @conn.full_drop
+      drop.should =~ /drop table full_drop_test cascade constraints/i
+      drop.should =~ /drop sequence full_drop_test_seq/i
+      drop.should =~ /drop view "full_drop_test_view"/i
+      drop.should =~ /drop package full_drop_test_package/i
+      drop.should =~ /drop function full_drop_test_function/i
+      drop.should =~ /drop procedure full_drop_test_procedure/i
+      drop.should =~ /drop synonym "full_drop_test_synonym"/i
+      drop.should =~ /drop type "full_drop_test_type"/i
+    end
+    it "should not drop tables when preserve_tables is true" do
+      drop = @conn.full_drop(true)
+      drop.should =~ /drop table full_drop_test_temp/i
+      drop.should_not =~ /drop table full_drop_test cascade constraints/i
+    end
+  end
+end

data/spec/active_record/connection_adapters/oracle_enhanced_connection_spec.rb

@@ -0,0 +1,206 @@
+require File.expand_path(File.dirname(__FILE__) + '/../../spec_helper')
+
+describe "OracleEnhancedConnection create connection" do
+
+  before(:all) do
+    @conn = ActiveRecord::ConnectionAdapters::OracleEnhancedConnection.create(CONNECTION_PARAMS)
+  end
+  
+  before(:each) do
+    @conn = ActiveRecord::ConnectionAdapters::OracleEnhancedConnection.create(CONNECTION_PARAMS) unless @conn.active?
+  end
+
+  after(:all) do
+    @conn.logoff if @conn.active?
+  end
+
+  it "should create new connection" do
+    @conn.should be_active
+  end
+
+  it "should ping active connection" do
+    @conn.ping.should be_true
+  end
+
+  it "should not ping inactive connection" do
+    @conn.logoff
+    lambda { @conn.ping }.should raise_error(ActiveRecord::ConnectionAdapters::OracleEnhancedConnectionException)
+  end
+
+  it "should reset active connection" do
+    @conn.reset!
+    @conn.should be_active
+  end
+
+  it "should be in autocommit mode after connection" do
+    @conn.should be_autocommit
+  end
+
+end
+
+if defined?(RUBY_ENGINE) && RUBY_ENGINE == 'jruby'
+
+  describe "OracleEnhancedConnection create JDBC connection" do
+    after(:each) do
+      @conn.logoff if @conn.active?
+    end
+
+    it "should create new connection using :url" do
+      params = CONNECTION_PARAMS.dup
+      params[:url] = "jdbc:oracle:thin:@#{DATABASE_HOST}:#{DATABASE_PORT}:#{DATABASE_NAME}"
+      params[:host] = nil
+      params[:database] = nil
+      @conn = ActiveRecord::ConnectionAdapters::OracleEnhancedConnection.create(params)
+      @conn.should be_active
+    end
+
+    it "should create new connection using :url and tnsnames alias" do
+      params = CONNECTION_PARAMS.dup
+      params[:url] = "jdbc:oracle:thin:@#{DATABASE_NAME}"
+      params[:host] = nil
+      params[:database] = nil
+      @conn = ActiveRecord::ConnectionAdapters::OracleEnhancedConnection.create(params)
+      @conn.should be_active
+    end
+
+    it "should create new connection using just tnsnames alias" do
+      params = CONNECTION_PARAMS.dup
+      params[:host] = nil
+      @conn = ActiveRecord::ConnectionAdapters::OracleEnhancedConnection.create(params)
+      @conn.should be_active
+    end
+
+  end
+
+end
+
+describe "OracleEnhancedConnection SQL execution" do
+
+  before(:all) do
+    @conn = ActiveRecord::ConnectionAdapters::OracleEnhancedConnection.create(CONNECTION_PARAMS)
+  end
+  
+  before(:each) do
+    @conn = ActiveRecord::ConnectionAdapters::OracleEnhancedConnection.create(CONNECTION_PARAMS) unless @conn.active?
+  end
+
+  after(:all) do
+    @conn.logoff if @conn.active?
+  end
+
+  it "should execute SQL statement" do
+    @conn.exec("SELECT * FROM dual").should_not be_nil
+  end
+
+  it "should execute SQL select" do
+    @conn.select("SELECT * FROM dual").should == [{'dummy' => 'X'}]
+  end
+
+  it "should execute SQL select and return also columns" do
+    @conn.select("SELECT * FROM dual", nil, true).should == [ [{'dummy' => 'X'}], ['dummy'] ]
+  end
+
+end
+
+describe "OracleEnhancedConnection auto reconnection" do
+
+  before(:all) do
+    ActiveRecord::Base.establish_connection(CONNECTION_PARAMS)
+    @conn = ActiveRecord::Base.connection.instance_variable_get("@connection")
+    @sys_conn = ActiveRecord::ConnectionAdapters::OracleEnhancedConnection.create(SYS_CONNECTION_PARAMS)
+  end
+  
+  before(:each) do
+    ActiveRecord::Base.connection.reconnect! unless @conn.active?
+  end
+
+  after(:all) do
+    ActiveRecord::Base.connection.disconnect! if @conn.active?
+  end
+
+  def kill_current_session
+    audsid = @conn.select("SELECT userenv('sessionid') audsid FROM dual").first['audsid']
+    sid_serial = @sys_conn.select("SELECT s.sid||','||s.serial# sid_serial
+        FROM   v$session s
+        WHERE  audsid = '#{audsid}'").first['sid_serial']
+    @sys_conn.exec "ALTER SYSTEM KILL SESSION '#{sid_serial}' IMMEDIATE"
+  end
+
+  it "should reconnect and execute SQL statement if connection is lost and auto retry is enabled" do
+    # @conn.auto_retry = true
+    ActiveRecord::Base.connection.auto_retry = true
+    kill_current_session
+    @conn.exec("SELECT * FROM dual").should_not be_nil
+  end
+
+  it "should not reconnect and execute SQL statement if connection is lost and auto retry is disabled" do
+    # @conn.auto_retry = false
+    ActiveRecord::Base.connection.auto_retry = false
+    kill_current_session
+    lambda { @conn.exec("SELECT * FROM dual") }.should raise_error
+  end
+
+  it "should reconnect and execute SQL select if connection is lost and auto retry is enabled" do
+    # @conn.auto_retry = true
+    ActiveRecord::Base.connection.auto_retry = true
+    kill_current_session
+    @conn.select("SELECT * FROM dual").should == [{'dummy' => 'X'}]
+  end
+
+  it "should not reconnect and execute SQL select if connection is lost and auto retry is disabled" do
+    # @conn.auto_retry = false
+    ActiveRecord::Base.connection.auto_retry = false
+    kill_current_session
+    lambda { @conn.select("SELECT * FROM dual") }.should raise_error
+  end
+
+end
+
+describe "OracleEnhancedConnection describe table" do
+
+  before(:all) do
+    @conn = ActiveRecord::ConnectionAdapters::OracleEnhancedConnection.create(CONNECTION_PARAMS)
+    @owner = CONNECTION_PARAMS[:username].upcase
+  end
+  
+  after(:all) do
+    @conn.logoff if @conn.active?
+  end
+
+  it "should describe existing table" do
+    @conn.exec "CREATE TABLE test_employees (first_name VARCHAR2(20))" rescue nil
+    @conn.describe("test_employees").should == [@owner, "TEST_EMPLOYEES"]
+    @conn.exec "DROP TABLE test_employees" rescue nil
+  end
+
+  it "should not describe non-existing table" do
+    lambda { @conn.describe("test_xxx") }.should raise_error(ActiveRecord::ConnectionAdapters::OracleEnhancedConnectionException)
+  end
+
+  it "should describe table in other schema" do
+    @conn.describe("sys.dual").should == ["SYS", "DUAL"]
+  end
+
+  it "should describe existing view" do
+    @conn.exec "CREATE TABLE test_employees (first_name VARCHAR2(20))" rescue nil
+    @conn.exec "CREATE VIEW test_employees_v AS SELECT * FROM test_employees" rescue nil
+    @conn.describe("test_employees_v").should == [@owner, "TEST_EMPLOYEES_V"]
+    @conn.exec "DROP VIEW test_employees_v" rescue nil
+    @conn.exec "DROP TABLE test_employees" rescue nil
+  end
+
+  it "should describe view in other schema" do
+    @conn.describe("sys.v_$version").should == ["SYS", "V_$VERSION"]
+  end
+
+  it "should describe existing private synonym" do
+    @conn.exec "CREATE SYNONYM test_dual FOR sys.dual" rescue nil
+    @conn.describe("test_dual").should == ["SYS", "DUAL"]
+    @conn.exec "DROP SYNONYM test_dual" rescue nil
+  end
+
+  it "should describe existing public synonym" do
+    @conn.describe("all_tables").should == ["SYS", "ALL_TABLES"]
+  end
+
+end