rails_sql_views 0.6.1 → 0.8.0

data/lib/rails_sql_views/version.rb

@@ -1,8 +1,8 @@
 module RailsSqlViews
   module VERSION #:nodoc:
     MAJOR = 0
-    MINOR = 6
-    TINY  = 1
+    MINOR = 7
+    TINY  = 0
 
     STRING = [MAJOR, MINOR, TINY].join('.')
   end

data/test/adapter_test.rb

@@ -0,0 +1,82 @@
+require "#{File.dirname(__FILE__)}/test_helper"
+
+class AdapterTest < Test::Unit::TestCase
+  def test_current_database
+    if ActiveRecord::Base.connection.respond_to?(:current_database)
+      assert_equal 'rails_sql_views_unittest', ActiveRecord::Base.connection.current_database
+    end
+  end
+  def test_tables
+    create_view
+    found = ActiveRecord::Base.connection.tables.sort
+    found.delete(ActiveRecord::Migrator.schema_migrations_table_name)
+    assert_equal ["items", "items_people", "people", "people2", "places", "v_people"], found
+  end
+  def test_base_tables
+    create_view
+    found = ActiveRecord::Base.connection.base_tables.sort
+    found.delete(ActiveRecord::Migrator.schema_migrations_table_name)
+    assert_equal ["items", "items_people", "people", "people2", "places"], found
+  end
+  def test_views
+    create_view
+    assert_equal ['v_people'], ActiveRecord::Base.connection.views
+  end
+  def test_columns
+    create_view
+    assert_equal ["f_name", "l_name", "social_security"], ActiveRecord::Base.connection.columns('v_people').collect { |c| c.name }
+  end
+  def test_supports_views
+    assert ActiveRecord::Base.connection.supports_views?
+  end
+  
+  def test_mapped_views
+    create_mapping
+    assert_equal ['v_people'], ActiveRecord::Base.connection.views
+  end
+  def test_mapped_columns
+    create_mapping
+    assert_equal ["f_name", "l_name", "address_id"], ActiveRecord::Base.connection.columns('v_people').collect { |c| c.name }
+  end
+  
+  def test_view_select_statement
+    case ActiveRecord::Base.connection.adapter_name
+    when "MySQL"
+      assert_equal "select `people`.`first_name` AS `f_name`,`people`.`last_name` AS `l_name`,`people`.`ssn` AS `social_security` from `people`", ActiveRecord::Base.connection.view_select_statement('v_people')
+    when "PostgreSQL"
+      assert_equal "SELECT people.first_name AS f_name, people.last_name AS l_name, people.ssn AS social_security FROM people;", ActiveRecord::Base.connection.view_select_statement('v_people')
+    end
+  end
+  
+  def test_old_name_not_found_error_during_mapping
+    assert_raise ActiveRecord::ActiveRecordError do
+      ActiveRecord::Base.connection.create_mapping_view(:people, :v_people, :force => true) do |v|
+        v.map_column :foo, :bar
+      end
+    end
+  end
+ 
+### TODO
+#  def test_only_base_table_triggers_are_dropped_for_disabled_ref_integrity
+#    ActiveRecord::Base.connection.disable_referential_integrity do
+#    end
+#  end
+  
+  private
+  def create_view
+    ActiveRecord::Base.connection.create_view(:v_people, 'select first_name, last_name, ssn from people', :force => true) do |v|
+      v.column :f_name
+      v.column :l_name
+      v.column :social_security
+    end
+  end
+  
+  def create_mapping
+    ActiveRecord::Base.connection.create_mapping_view(:people, :v_people, :force => true) do |v|
+      v.map_column :id, nil
+      v.map_column :first_name, :f_name
+      v.map_column :last_name, :l_name
+      v.map_column :ssn, nil
+    end
+  end
+end

data/test/connection/native_mysql/connection.rb

@@ -0,0 +1,32 @@
+print "Using native MySQL\n"
+
+adapter_name = 'mysql'
+config = YAML.load_file(File.join(File.dirname(__FILE__), '/../../connection.yml'))[adapter_name]
+
+#require 'logger'
+#ActiveRecord::Base.logger = Logger.new("debug.log")
+
+ActiveRecord::Base.silence do
+  ActiveRecord::Base.configurations = {
+    config['database'] => {
+    :adapter  => adapter_name,
+    :username => config['username'],
+    :password => config['password'],
+    :host     => config['host'],
+    :database => config['database'],
+    :encoding => config['encoding'],
+    :schema_file => config['schema_file'],
+  }
+  }
+
+  ActiveRecord::Base.establish_connection config['database']
+  ActiveRecord::Migration.verbose = false
+
+  puts "Resetting database"
+  conn = ActiveRecord::Base.connection
+  conn.recreate_database(conn.current_database)
+  conn.reconnect!
+  lines = open(File.join(File.dirname(__FILE__), ActiveRecord::Base.configurations[config['database']][:schema_file])).readlines
+  lines.join.split(';').each { |line| conn.execute(line) }
+  conn.reconnect!
+end

data/test/connection/native_postgresql/connection.rb

@@ -0,0 +1,31 @@
+print "Using native PostgreSQL\n"
+
+adapter_name = 'postgresql'
+config = YAML.load_file(File.join(File.dirname(__FILE__), '/../../connection.yml'))[adapter_name]
+
+#require 'logger'
+#ActiveRecord::Base.logger = Logger.new("debug.log")
+
+ActiveRecord::Base.silence do
+  ActiveRecord::Base.configurations = {
+  'rails_sql_views_unittest' => {
+    :adapter  => adapter_name,
+    :username => config['username'],
+    :password => config['password'],
+    :host     => config['host'],
+    :database => config['database'],
+    :encoding => config['encoding'],
+    :schema_file => config['schema_file'],
+  }
+  }
+
+  ActiveRecord::Base.establish_connection config['database']
+
+  puts "Resetting database"
+  conn = ActiveRecord::Base.connection
+  #conn.recreate_database(conn.current_database)
+  conn.reconnect!
+  lines = open(File.join(File.dirname(__FILE__), ActiveRecord::Base.configurations[config['database']][:schema_file])).readlines
+  lines.join.split(';').each { |line| conn.execute(line) }
+  conn.reconnect!
+end

data/test/connection/oracle_enhanced/connection.rb

@@ -0,0 +1,29 @@
+print "Using Oracle Enhanced\n"
+
+#require 'logger'
+#ActiveRecord::Base.logger = Logger.new("debug.log")
+
+ActiveRecord::Base.configurations = {
+  'rails_sql_views_unittest' => {
+    :adapter  => :oracle_enhanced,
+    :username => 'rails_sql_views_unittest',
+    :password => 'rails',
+    :host     => 'localhost',
+    :database => 'mydev',
+    :encoding => 'utf8',
+    :procedures_file  => 'procedures.sql',
+    :schema_file      => 'schema.sql',
+  }
+}
+
+ActiveRecord::Base.establish_connection 'rails_sql_views_unittest'
+
+puts "Resetting database"
+conn = ActiveRecord::Base.connection
+#conn.recreate_database(conn.current_database)
+conn.reconnect!
+[:procedures_file, :schema_file].each do |file|
+  lines = open(File.join(File.dirname(__FILE__), ActiveRecord::Base.configurations['rails_sql_views_unittest'][file])).readlines
+  conn.execute(lines.to_s)
+end
+conn.reconnect!

data/test/models/item.rb

@@ -0,0 +1,4 @@
+class Item < ActiveRecord::Base
+  belongs_to :person
+  has_and_belongs_to_many :people
+end

data/test/models/person.rb

@@ -0,0 +1,5 @@
+class Person < ActiveRecord::Base
+  belongs_to :address, :class_name => 'Place', :foreign_key => :address_id
+  has_many :owned_items, :class_name => 'Item'
+  has_and_belongs_to_many :shared_items, :class_name => 'Item'
+end

data/test/models/person2.rb

@@ -0,0 +1,3 @@
+class Person2 < ActiveRecord::Base
+  set_table_name :people2
+end

data/test/models/place.rb

@@ -0,0 +1,2 @@
+class Place < ActiveRecord::Base
+end

data/test/models/v_person.rb

@@ -0,0 +1,4 @@
+require 'active_record/view'
+
+class VPerson < ActiveRecord::View
+end

data/test/models/v_profile.rb

@@ -0,0 +1,3 @@
+class VProfile < ActiveRecord::Base
+  
+end

data/test/schema.native_mysql.expected.rb

@@ -0,0 +1,51 @@
+# This file is auto-generated from the current state of the database. Instead of editing this file, 
+# please use the migrations feature of Active Record to incrementally modify your database, and
+# then regenerate this schema definition.
+#
+# Note that this schema.rb definition is the authoritative source for your database schema. If you need
+# to create the application database on another system, you should be using db:schema:load, not running
+# all the migrations from scratch. The latter is a flawed and unsustainable approach (the more migrations
+# you'll amass, the slower it'll run and the greater likelihood for issues).
+#
+# It's strongly recommended to check this file into your version control system.
+
+ActiveRecord::Schema.define(:version => 0) do
+
+  create_table "items", :force => true do |t|
+    t.integer "person_id"
+  end
+
+  create_table "items_people", :id => false, :force => true do |t|
+    t.integer "person_id"
+    t.integer "item_id"
+  end
+
+  create_table "people", :force => true do |t|
+    t.string  "first_name"
+    t.string  "last_name"
+    t.string  "ssn",        :limit => 64
+    t.integer "address_id"
+  end
+
+  create_table "people2", :force => true do |t|
+    t.string "first_name"
+    t.string "last_name"
+    t.string "ssn",        :limit => 64
+  end
+
+  create_table "places", :force => true do |t|
+    t.text   "address"
+    t.string "city"
+    t.string "cstate"
+    t.string "country", :limit => 2
+  end
+
+  create_view "v_people", "select `people`.`id` AS `id`,`people`.`first_name` AS `f_name`,`people`.`last_name` AS `l_name`,`people`.`ssn` AS `social_security`,`people`.`address_id` AS `address_id` from `people`", :force => true do |v|
+    v.column :id
+    v.column :f_name
+    v.column :l_name
+    v.column :social_security
+    v.column :address_id
+  end
+
+end

data/test/schema.native_postgresql.expected.rb

@@ -0,0 +1,51 @@
+# This file is auto-generated from the current state of the database. Instead of editing this file, 
+# please use the migrations feature of Active Record to incrementally modify your database, and
+# then regenerate this schema definition.
+#
+# Note that this schema.rb definition is the authoritative source for your database schema. If you need
+# to create the application database on another system, you should be using db:schema:load, not running
+# all the migrations from scratch. The latter is a flawed and unsustainable approach (the more migrations
+# you'll amass, the slower it'll run and the greater likelihood for issues).
+#
+# It's strongly recommended to check this file into your version control system.
+
+ActiveRecord::Schema.define(:version => 0) do
+
+  create_table "items", :force => true do |t|
+    t.integer "person_id"
+  end
+
+  create_table "items_people", :id => false, :force => true do |t|
+    t.integer "person_id"
+    t.integer "item_id"
+  end
+
+  create_table "people", :force => true do |t|
+    t.string  "first_name"
+    t.string  "last_name"
+    t.string  "ssn",        :limit => 64
+    t.integer "address_id"
+  end
+
+  create_table "people2", :force => true do |t|
+    t.string "first_name"
+    t.string "last_name"
+    t.string "ssn",        :limit => 64
+  end
+
+  create_table "places", :force => true do |t|
+    t.text   "address"
+    t.string "city"
+    t.string "cstate"
+    t.string "country", :limit => 2
+  end
+
+  create_view "v_people", "SELECT people.id, people.first_name AS f_name, people.last_name AS l_name, people.ssn AS social_security, people.address_id FROM people;", :force => true do |v|
+    v.column :id
+    v.column :f_name
+    v.column :l_name
+    v.column :social_security
+    v.column :address_id
+  end
+
+end

data/test/schema.oracle_enhanced.expected.rb

@@ -0,0 +1,51 @@
+# This file is auto-generated from the current state of the database. Instead of editing this file, 
+# please use the migrations feature of Active Record to incrementally modify your database, and
+# then regenerate this schema definition.
+#
+# Note that this schema.rb definition is the authoritative source for your database schema. If you need
+# to create the application database on another system, you should be using db:schema:load, not running
+# all the migrations from scratch. The latter is a flawed and unsustainable approach (the more migrations
+# you'll amass, the slower it'll run and the greater likelihood for issues).
+#
+# It's strongly recommended to check this file into your version control system.
+
+ActiveRecord::Schema.define(:version => 0) do
+
+  create_table "items", :force => true do |t|
+    t.integer "person_id", :precision => 38, :scale => 0
+  end
+
+  create_table "items_people", :id => false, :force => true do |t|
+    t.integer "person_id", :precision => 38, :scale => 0
+    t.integer "item_id",   :precision => 38, :scale => 0
+  end
+
+  create_table "people", :force => true do |t|
+    t.string  "first_name"
+    t.string  "last_name"
+    t.string  "ssn",        :limit => 64
+    t.integer "address_id",               :precision => 38, :scale => 0
+  end
+
+  create_table "people2", :force => true do |t|
+    t.string "first_name"
+    t.string "last_name"
+    t.string "ssn",        :limit => 64
+  end
+
+  create_table "places", :force => true do |t|
+    t.string "address", :limit => 2000
+    t.string "city"
+    t.string "cstate"
+    t.string "country", :limit => 2
+  end
+
+  create_view "v_people", "select id, first_name, last_name, ssn, address_id from people", :force => true do |v|
+    v.column :id
+    v.column :f_name
+    v.column :l_name
+    v.column :social_security
+    v.column :address_id
+  end
+
+end

data/test/schema_dumper_test.rb

@@ -0,0 +1,117 @@
+require "#{File.dirname(__FILE__)}/test_helper"
+require 'active_record/schema_dumper'
+
+class SchemaDumperTest < Test::Unit::TestCase
+  def setup
+    teardown
+  end
+  def teardown
+    ['V_PEOPLE', 'V_PROFILE'].each do |view|
+      if ActiveRecord::Base.connection.adapter_name == 'OracleEnhanced'
+        ActiveRecord::Base.connection.execute("
+          DECLARE
+            CURSOR C1 is SELECT view_name FROM user_views where view_name = '#{view}';
+          BEGIN
+            FOR I IN C1 LOOP
+              EXECUTE IMMEDIATE 'DROP VIEW '||I.view_name||'';
+            END LOOP;
+          END;
+        ");
+      else
+        ActiveRecord::Base.connection.execute("drop view if exists #{view}")
+      end
+    end
+  end
+  def test_view
+    create_people_view
+    stream = StringIO.new
+    dumper = ActiveRecord::SchemaDumper.dump(ActiveRecord::Base.connection, stream)
+    stream.rewind
+    assert_equal File.open(File.dirname(__FILE__) + "/schema.#{$connection}.expected.rb", 'r').readlines, stream.readlines
+  end
+  def test_dump_and_load
+    create_people_view
+    assert_dump_and_load_succeed
+  end
+  def test_union
+    Person.create(:first_name => 'Joe', :last_name => 'User', :ssn => '123456789')
+    Person2.create(:first_name => 'Jane', :last_name => 'Doe', :ssn => '222334444')
+    
+    select_stmt = <<-HERE
+      select first_name, last_name, ssn from people
+      UNION
+      select first_name, last_name, ssn from people2
+    HERE
+    
+    ActiveRecord::Base.connection.create_view(:v_profile, select_stmt, :force => true) do |v|
+      v.column :first_name
+      v.column :last_name
+      v.column :ssn
+    end
+    
+    assert_dump_and_load_succeed
+  end
+  def test_view_creation_order
+    ActiveRecord::SchemaDumper.view_creation_order << :v_people
+    create_people_view
+    assert_dump_and_load_succeed
+    ActiveRecord::SchemaDumper.view_creation_order.pop
+  end
+  def test_symbol_ignore
+    ActiveRecord::SchemaDumper.ignore_views << :v_people
+    create_people_view
+    assert_dump_and_load_succeed
+    ActiveRecord::SchemaDumper.ignore_views.pop
+  end
+  def test_regex_ignore
+    ActiveRecord::SchemaDumper.ignore_views << Regexp.new(/v_people/)
+    create_people_view
+    assert_dump_and_load_succeed
+    ActiveRecord::SchemaDumper.ignore_views.pop
+  end
+  def test_non_allowed_object_raises_error
+    create_people_view
+    ActiveRecord::SchemaDumper.ignore_views << 0
+    begin
+      schema_file = File.dirname(__FILE__) + "/schema.#{$connection}.out.rb"
+      File.open(schema_file, "w") do |file|
+        assert_raise(StandardError) do
+          ActiveRecord::SchemaDumper.dump(ActiveRecord::Base.connection, file)
+        end
+      end
+    ensure
+      ActiveRecord::SchemaDumper.ignore_views.pop
+    end
+  end
+  
+  def test_logging_error
+    ActiveRecord::SchemaDumper.ignore_views << 0
+    old_logger = ActiveRecord::Base.logger
+    
+    begin
+      mock_logger = flexmock('logger', :error => nil)
+      mock_logger.should_receive(:error)
+      ActiveRecord::Base.logger = mock_logger
+      schema_file = File.dirname(__FILE__) + "/schema.#{$connection}.out.rb"
+      File.open(schema_file, "w") do |file|
+        ActiveRecord::SchemaDumper.dump(ActiveRecord::Base.connection, file)
+      end
+    ensure
+      ActiveRecord::SchemaDumper.ignore_views.pop
+      ActiveRecord::Base.logger = old_logger
+    end
+  end
+  
+  def assert_dump_and_load_succeed
+    schema_file = File.dirname(__FILE__) + "/schema.#{$connection}.out.rb"
+    assert_nothing_raised do
+      File.open(schema_file, "w") do |file|
+        ActiveRecord::SchemaDumper.dump(ActiveRecord::Base.connection, file)
+      end
+    end
+
+    assert_nothing_raised do
+      load(schema_file)
+    end
+  end
+end