active_schema 0.1.0

data/spec/active_schema/associations/by_foreign_key_spec.rb

@@ -0,0 +1,73 @@
+require File.expand_path(File.dirname(__FILE__) + '/../../spec_helper')
+include ActiveSchema::Associations
+
+describe ActiveSchema::Associations::ByForeignKey do
+  def given_foreign_key(from_table, to_table, key_name, unique = false)
+    @from_model = mock("FakeFromModel",
+      :columns => [ActiveRecord::ConnectionAdapters::Column.new(key_name, nil, 'int(11)')],
+      :name => from_table.singularize.camelize
+    )
+    @to_model = mock("FakeToModel",
+      :name => to_table.singularize.camelize
+    )
+    @key_name = key_name
+    @unique = unique
+  end
+  after {
+    described_class.new(@from_model, @to_model, @key_name, @unique).generate
+  }
+  
+  describe ActiveSchema::Associations::ByForwardForeignKey do 
+    it "sets belongs_to" do
+      given_foreign_key('prisoners', 'facilities', 'facility_id')
+      @from_model.should_receive(:belongs_to)\
+        .with("facility", { :class_name => "Facility", :foreign_key => 'facility_id'})
+    end
+  end
+
+  describe ActiveSchema::Associations::ByReverseForeignKey do
+    context "Key column not unique" do
+      it "sets has_many" do
+        given_foreign_key('prisoners', 'facilities', 'facility_id')
+        @to_model.should_receive(:has_many)\
+          .with("prisoners", { :class_name => "Prisoner", :foreign_key => 'facility_id'})
+      end
+    end
+    context "Key column unique" do
+      it "sets has_one" do
+        given_foreign_key('prisoners', 'facilities', 'facility_id', true)
+        @to_model.should_receive(:has_one)\
+          .with("prisoner", { :class_name => "Prisoner", :foreign_key => 'facility_id'})
+      end
+
+    end
+  end
+end
+
+describe ActiveSchema::Associations::Naming do
+  before {
+    @naming = Class.new.send(:include, Naming).new
+  }
+
+  def given_model(model_name)
+    @model = mock(model_name, :name => model_name)
+  end
+  context "name_for" do
+    it "underscores camelized names" do
+      given_model("ABumpyBackModel")
+      @naming.name_for(@model).should == "a_bumpy_back_model"
+    end
+
+    it "strips module names" do
+      given_model("Hideously::Complicated::And::Convoluted::ClassHierarchy")
+      @naming.name_for(@model).should == "class_hierarchy"
+    end
+  end
+
+  context "plural_name_for" do
+    it "pluralizes name_for" do
+      given_model("BumpyBackModel")
+      @naming.plural_name_for(@model).should == "bumpy_back_models"
+    end
+  end
+end

data/spec/active_schema/associations/generator_spec.rb

@@ -0,0 +1,5 @@
+require File.expand_path(File.dirname(__FILE__) + '/../../spec_helper')
+
+describe ActiveSchema::Associations::Generator do
+end
+

data/spec/active_schema/in_advance_feeder_spec.rb

@@ -0,0 +1,25 @@
+require File.expand_path(File.dirname(__FILE__) + '/../spec_helper')
+include ActiveSchema
+
+describe InAdvanceFeeder do
+  before(:each) do
+    @prisoner_model = prisoner_model
+    @in_advance_feeder = InAdvanceFeeder.new
+    @in_advance_feeder.stub!(:dispatch_attachments)
+  end
+
+  context "when a model is loaded" do
+    after { @in_advance_feeder.model_loaded(@prisoner_model) }
+
+    it "adds the model" do
+      @in_advance_feeder.table_hub.should_receive(:add_model).with(@prisoner_model)
+    end
+
+    it "attaches associations and validations" do
+      @in_advance_feeder.should_receive(:dispatch_attachments)
+    end
+
+  end
+
+end
+

data/spec/active_schema/on_the_fly_feeder_spec.rb

@@ -0,0 +1,34 @@
+# To change this template, choose Tools | Templates
+# and open the template in the editor.
+
+require File.expand_path(File.dirname(__FILE__) + '/../spec_helper')
+
+describe ActiveSchema::OnTheFlyFeeder do
+  before(:each) do
+    @on_the_fly_feeder = ActiveSchema::OnTheFlyFeeder.new
+    @prisoner_model = prisoner_model
+    @table_hub = @on_the_fly_feeder.table_hub
+  end
+
+  context "when a model is loaded" do
+    after { @on_the_fly_feeder.model_loaded(@prisoner_model) }
+
+    it "adds indexes to TableHub" do
+      @table_hub.should_receive(:add_index)\
+        .with(@prisoner_model.table_name, instance_of(ActiveRecord::ConnectionAdapters::IndexDefinition))
+    end
+
+    it "adds foreign keys to TableHub" do
+      @table_hub.should_receive(:add_foreign_key).with('prisoners', 'facilities', 'facility_id')
+    end
+    #
+    it "adds model to TableHub" do
+      @on_the_fly_feeder.stub!(:dispatch_attachments)
+      @table_hub.should_receive(:add_model).with(@prisoner_model)
+    end
+
+    it "attaches associations and validations" do
+      @on_the_fly_feeder.should_receive(:dispatch_attachments)
+    end
+  end
+end

data/spec/active_schema/schema_feeder_spec.rb

@@ -0,0 +1,111 @@
+require File.expand_path(File.dirname(__FILE__) + '/../spec_helper')
+
+include ActiveSchema
+describe SchemaFeeder do
+  before(:each) do
+    @table_hub = ActiveSchema::TableHub.new
+    @schema_feeder = SchemaFeeder.new
+    @schema_feeder.stub!(:table_hub).and_return(@table_hub)
+  end
+
+  context "add_index" do
+    after do
+      @schema_feeder.add_index "facilities", ["warden_id"], :name => "warden_id", :unique => true
+    end
+
+    it "adds an IndexDefinition to TableHub" do
+      @table_hub.should_receive(:add_index).with('facilities', instance_of(ActiveRecord::ConnectionAdapters::IndexDefinition))
+    end
+  end
+
+  context "add_foreign_key" do
+    after do
+      @schema_feeder.add_foreign_key "facilities", "wardens", :name => "facilities_warden_id"
+    end
+
+    it "guesses the foreign key column name if not supplied" do
+      @table_hub.should_receive(:add_foreign_key).with(anything, anything, "warden_id")
+    end
+
+    it "adds a foreign key to TableHub" do
+      @table_hub.should_receive(:add_foreign_key).with('facilities', 'wardens', "warden_id")
+    end
+
+  end
+
+  context "reading the contents of a schema file" do 
+    it "receives appropriate callbacks" do
+      @schema_feeder.should_receive(:add_index).exactly(4).times
+      @schema_feeder.should_receive(:add_foreign_key).exactly(3).times
+      @schema_feeder.read(EXAMPLE_SCHEMA)
+    end
+
+  end
+end
+
+describe FilteredSchemaReader do
+  before do
+    @call_back = mock("SchemaFeeder")
+    @filtered_schema_reader = FilteredSchemaReader.new(@call_back, EXAMPLE_SCHEMA)
+  end
+
+  it "contains only lines matching filter" do
+    @filtered_schema_reader.filtered_lines.size.should == 7
+  end
+
+  it "makes callbacks to the supplied object" do
+    @call_back.should_receive(:add_index).exactly(4).times
+    @call_back.should_receive(:add_foreign_key).exactly(3).times
+    @filtered_schema_reader.evaluate
+  end
+
+end
+#ActiveRecord::Schema.define(:version => 0)
+
+EXAMPLE_SCHEMA = <<FILE
+# 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 "facilities", :force => true do |t|
+    t.text    "name",      :null => false
+    t.integer "warden_id", :null => false
+  end
+
+  add_index "facilities", ["warden_id"], :name => "warden_id", :unique => true
+
+  create_table "prisoners", :force => true do |t|
+    t.text    "name",        :null => false
+    t.integer "facility_id", :null => false
+  end
+
+  add_index "prisoners", ["facility_id"], :name => "facility_id"
+
+  create_table "wardens", :primary_key => "facility_id", :force => true do |t|
+    t.integer "id",   :null => false
+    t.text    "name", :null => false
+  end
+
+  add_index "wardens", ["facility_id"], :name => "facility_id", :unique => true
+  add_index "wardens", ["id"], :name => "id"
+
+  add_foreign_key "facilities", "wardens", :name => "facilities_warden_id"
+
+  add_foreign_key "prisoners", "facilities", :name => "prisoners_facility_id"
+
+  add_foreign_key "wardens", "facilities", :name => "wardens_facility_id"
+
+end
+FILE
+
+

data/spec/active_schema/table_hub_spec.rb

@@ -0,0 +1,70 @@
+require File.expand_path(File.dirname(__FILE__) + '/../spec_helper')
+
+describe ActiveSchema::TableHub do
+  before(:each) do
+    @table_hub = ActiveSchema::TableHub.new
+  end
+
+  context "when adding a model" do
+    it "should have a table by the model's table name" do
+      @table_hub.add_model(prisoner_model)
+      @table_hub.tables.should have_key("prisoners")
+    end
+
+    context "and a table already exists" do
+      it "should not change the identity of the existing table object" do
+        @table_hub.add_model(prisoner_model)
+        @table_hub.add_foreign_key('prisoners', 'facilities', 'facility_id') # Makes a Table object for the 'facilities' table
+        facility_table = @table_hub.tables['facilities']
+        @table_hub.add_model(facility_model)
+        @table_hub.tables['facilities'].should equal(facility_table)
+      end
+
+      it "links the model with the existing table" do
+        @table_hub.add_model(prisoner_model)
+        @table_hub.add_foreign_key('prisoners', 'facilities', 'facility_id') # Makes a Table object for the 'facilities' table
+        facility = facility_model
+        @table_hub.add_model(facility)
+        @table_hub.tables['facilities'].model.should == facility
+      end
+    end
+  end
+
+  context "when adding a foreign key" do
+    it "should contain an entry for the destination table" do
+      @table_hub.add_model(prisoner_model)
+      @table_hub.add_foreign_key('prisoners', 'facilities', 'facility_id')
+      @table_hub.tables.should have_key("facilities")
+    end
+
+    it "adds forward and reverse relations between tables" do
+      @table_hub.add_foreign_key('prisoners', 'facilities', 'facility_id')
+      @table_hub.relations['prisoners'].should include(@table_hub.tables['facilities'])
+      @table_hub.relations['facilities'].should include(@table_hub.tables['prisoners'])
+    end
+
+    it "adds a foreign key to the source table" do
+      @table_hub.add_model(prisoner_model)
+      prisoner_table = @table_hub.tables['prisoners']
+      prisoner_table.should_receive(:add_foreign_key)
+      @table_hub.add_foreign_key('prisoners', 'facilities', 'facility_id')
+    end
+
+  end
+
+  context "when adding an index" do
+    it "should contain an entry for the indexed table" do
+      @table_hub.add_index('prisoners', mock("IndexObject"))
+      @table_hub.tables.should have_key("prisoners")
+    end
+
+    it "adds an index entry to the indexed table" do
+      @table_hub.add_model(prisoner_model)
+      index_obj = mock("IndexObject")
+      @table_hub.tables['prisoners'].should_receive(:add_index).with(index_obj)
+      @table_hub.add_index('prisoners', index_obj)
+    end
+  end
+
+end
+

data/spec/active_schema/table_spec.rb

@@ -0,0 +1,13 @@
+# To change this template, choose Tools | Templates
+# and open the template in the editor.
+
+require File.expand_path(File.dirname(__FILE__) + '/../spec_helper')
+
+describe ActiveSchema::Table do
+  describe "unique_index_on?" do
+    it "should be true if unique index" do
+      Warden.table_with_indexes.unique_index_on?('facility_id').should be_true
+    end
+  end
+end
+

data/spec/active_schema/validations/by_column_spec.rb

@@ -0,0 +1,47 @@
+require File.expand_path(File.dirname(__FILE__) + '/../../spec_helper')
+
+describe ActiveSchema::Validations::ByColumn do
+  after { described_class.new(@model, @column).generate }
+
+  def given_column(name, sql_type, null = true)
+    @model = double("FakeModel")
+    @column = ActiveRecord::ConnectionAdapters::Column.new(name, nil, sql_type, null)
+  end
+
+  describe ActiveSchema::Validations::ByDataType do
+
+    it "accepts only integers when int(11)" do
+      given_column('digit', 'int(11)')
+      @model.should_receive(:validates_numericality_of)\
+        .with(:digit, {:allow_nil => true, :only_integer => true})
+    end
+
+    it "accepts only numbers when decimal(8,2)" do
+      given_column('number', 'decimal(8,2)')
+      @model.should_receive(:validates_numericality_of)\
+        .with(:number, {:allow_nil => true})
+    end
+
+    it "validates text length when varchar(255)" do
+      given_column('text', 'varchar(255)')
+      @model.should_receive(:validates_length_of)\
+        .with(:text, {:allow_nil => true, :maximum => 255})
+    end
+  end
+
+  describe ActiveSchema::Validations::ByNullability do
+
+    it "validates presence when varchar(255)" do
+      given_column('text', 'varchar(255)', false)
+      @model.should_receive(:validates_presence_of)\
+        .with(:text, {})
+    end
+
+    it "validates inclusion of when boolean" do
+      given_column('yes_or_no', 'boolean', false)
+      @model.should_receive(:validates_inclusion_of)\
+        .with(:yes_or_no, {:in => [true, false]})
+    end
+  end
+end
+

data/spec/active_schema/validations/by_index_spec.rb

@@ -0,0 +1,15 @@
+# To change this template, choose Tools | Templates
+# and open the template in the editor.
+
+require File.expand_path(File.dirname(__FILE__) + '/../../spec_helper')
+
+describe ActiveSchema::Validations::ByIndex do
+  before(:each) do
+    @by_index = ActiveSchema::Validations::ByIndex
+  end
+
+  it "should desc" do
+    # TODO
+  end
+end
+

data/spec/active_schema/validations/generator_spec.rb

@@ -0,0 +1,23 @@
+require File.expand_path(File.dirname(__FILE__) + '/../../spec_helper')
+
+describe ActiveSchema::Validations::Generator do
+  before {
+    @prisoner_table = prisoner_table
+    @prisoner_model = @prisoner_table.model
+  }
+
+  it "is a healthy model" do
+    @prisoner_model.columns.should_not be_nil
+  end
+
+  it "skips validation for columns in given proc" do
+    @prisoner_model.should_not be_nil
+    skip_proc = proc{|column| column.name == "id" }
+    generator = ActiveSchema::Validations::Generator.new(@prisoner_table, skip_proc)
+    generator.generate
+    @prisoner_model.validators_on(:id).should be_empty
+    @prisoner_model.validators_on(:name).should_not be_empty
+
+  end
+end
+

data/spec/active_schema_spec.rb

@@ -0,0 +1,14 @@
+require File.expand_path(File.dirname(__FILE__) + '/spec_helper')
+
+describe "ActiveSchema" do
+  context "configuration" do
+    it "has configuration method" do
+      ActiveSchema.should respond_to :configuration
+    end
+    
+    it "defaults to OnTheFlyFeeder" do 
+      ActiveSchema.configuration.feeder.should be_instance_of(ActiveSchema::OnTheFlyFeeder)
+    end
+  end
+
+end

data/spec/spec_helper.rb

@@ -0,0 +1,31 @@
+require 'rubygems'
+require 'spork'
+
+Spork.prefork do
+  # Loading more in this block will cause your tests to run faster. However, 
+  # if you change any configuration or code from libraries loaded here, you'll
+  # need to restart spork for it take effect.
+  require 'bundler'
+  begin
+    Bundler.setup(:default, :development)
+  rescue Bundler::BundlerError => e
+    $stderr.puts e.message
+    $stderr.puts "Run `bundle install` to install missing gems"
+    exit e.status_code
+  end
+  $LOAD_PATH.unshift(File.dirname(__FILE__))
+  $LOAD_PATH.unshift(File.join(File.dirname(__FILE__), '..', 'lib'))
+  require 'rspec'
+end
+
+Spork.each_run do
+  require 'active_schema'
+end
+
+# Requires supporting files with custom matchers and macros, etc,
+# in ./support/ and its subdirectories.
+Dir["#{File.dirname(__FILE__)}/support/**/*.rb"].each {|f| require f}
+
+RSpec.configure do |c|
+  c.include(ModelMacros)
+end