sequel-rails 0.1.8 → 0.4.0.pre

data/lib/sequel_rails/version.rb

@@ -0,0 +1,3 @@
+module SequelRails
+  VERSION = "0.4.0.pre"
+end

data/sequel-rails.gemspec

@@ -1,93 +1,29 @@
-# Generated by jeweler
-# DO NOT EDIT THIS FILE DIRECTLY
-# Instead, edit Jeweler::Tasks in Rakefile, and run the gemspec command
 # -*- encoding: utf-8 -*-
+$:.push File.expand_path("../lib", __FILE__)
 
-Gem::Specification.new do |s|
-  s.name = %q{sequel-rails}
-  s.version = "0.1.8"
+require "sequel_rails/version"
 
-  s.required_rubygems_version = Gem::Requirement.new(">= 0") if s.respond_to? :required_rubygems_version=
-  s.authors = ["Brasten Sager (brasten)"]
-  s.date = %q{2010-08-31}
-  s.description = %q{Integrate Sequel with Rails 3}
-  s.email = %q{brasten@gmail.com}
-  s.extra_rdoc_files = [
-    "LICENSE",
-     "README.rdoc"
-  ]
-  s.files = [
-    ".document",
-     ".gitignore",
-     "CHANGELOG",
-     "Gemfile",
-     "LICENSE",
-     "README.rdoc",
-     "Rakefile",
-     "VERSION",
-     "autotest/discover.rb",
-     "lib/generators/sequel.rb",
-     "lib/generators/sequel/migration/migration_generator.rb",
-     "lib/generators/sequel/migration/templates/migration.rb",
-     "lib/generators/sequel/model/model_generator.rb",
-     "lib/generators/sequel/model/templates/model.rb",
-     "lib/generators/sequel/observer/observer_generator.rb",
-     "lib/generators/sequel/observer/templates/observer.rb",
-     "lib/sequel-rails.rb",
-     "lib/sequel-rails/configuration.rb",
-     "lib/sequel-rails/migrations.rb",
-     "lib/sequel-rails/railtie.rb",
-     "lib/sequel-rails/railties/benchmarking_mixin.rb",
-     "lib/sequel-rails/railties/controller_runtime.rb",
-     "lib/sequel-rails/railties/database.rake",
-     "lib/sequel-rails/railties/i18n_support.rb",
-     "lib/sequel-rails/railties/log_subscriber.rb",
-     "lib/sequel-rails/runtime.rb",
-     "lib/sequel-rails/session_store.rb",
-     "lib/sequel-rails/setup.rb",
-     "lib/sequel-rails/storage.rb",
-     "sequel-rails.gemspec",
-     "spec/rcov.opts",
-     "spec/setup_spec.rb",
-     "spec/spec.opts",
-     "spec/spec_helper.rb",
-     "tasks/ci.rake",
-     "tasks/clean.rake",
-     "tasks/metrics.rake",
-     "tasks/spec.rake",
-     "tasks/yard.rake",
-     "tasks/yardstick.rake"
-  ]
-  s.homepage = %q{http://github.com/brasten/sequel-rails}
-  s.rdoc_options = ["--charset=UTF-8"]
+Gem::Specification.new do |s|
+  s.name        = "sequel-rails"
+  s.version     = SequelRails::VERSION
+  s.platform    = Gem::Platform::RUBY
+  s.authors     = ["Brasten Sager (brasten)", "Jonathan TRON"]
+  s.email       = ["brasten@gmail.com", "jonathan.tron@metrilio.com"]
+  s.homepage    = "https://github.com/TalentBox/sequel-rails"
+  s.description = "Integrate Sequel with Rails 3"
+  s.summary     = "Use Sequel with Rails 3"
+  s.files         = `git ls-files`.split("\n")
+  s.test_files    = `git ls-files -- {test,spec,features}/*`.split("\n")
+  s.executables   = `git ls-files -- bin/*`.split("\n").map{ |f| File.basename(f) }
   s.require_paths = ["lib"]
-  s.rubygems_version = %q{1.3.6}
-  s.summary = %q{Use Sequel with Rails 3}
-  s.test_files = [
-    "spec/setup_spec.rb",
-     "spec/spec_helper.rb"
-  ]
+  s.extra_rdoc_files = ["LICENSE", "README.md"]
+  s.rdoc_options = ["--charset=UTF-8"]
 
-  if s.respond_to? :specification_version then
-    current_version = Gem::Specification::CURRENT_SPECIFICATION_VERSION
-    s.specification_version = 3
+  s.add_runtime_dependency("sequel", ["~> 3.28"])
+  s.add_runtime_dependency("railties", ["~> 3.2.0"])
 
-    if Gem::Version.new(Gem::RubyGemsVersion) >= Gem::Version.new('1.2.0') then
-      s.add_runtime_dependency(%q<sequel>, ["~> 3.13"])
-      s.add_runtime_dependency(%q<activesupport>, ["~> 3.0.0"])
-      s.add_runtime_dependency(%q<actionpack>, ["~> 3.0.0"])
-      s.add_runtime_dependency(%q<railties>, ["~> 3.0.0"])
-    else
-      s.add_dependency(%q<sequel>, ["~> 3.13"])
-      s.add_dependency(%q<activesupport>, ["~> 3.0.0"])
-      s.add_dependency(%q<actionpack>, ["~> 3.0.0"])
-      s.add_dependency(%q<railties>, ["~> 3.0.0"])
-    end
-  else
-    s.add_dependency(%q<sequel>, ["~> 3.13"])
-    s.add_dependency(%q<activesupport>, ["~> 3.0.0"])
-    s.add_dependency(%q<actionpack>, ["~> 3.0.0"])
-    s.add_dependency(%q<railties>, ["~> 3.0.0"])
-  end
+  s.add_development_dependency "rake", ">= 0.8.7"
+  s.add_development_dependency "rspec", "~> 2.7.0"
+  s.add_development_dependency "combustion", "~> 0.3.1"
+  s.add_development_dependency "generator_spec", "~> 0.8.7"
 end
-

data/spec/internal/app/models/user.rb

@@ -0,0 +1,2 @@
+class User < Sequel::Model
+end

data/spec/internal/config/database.yml

@@ -0,0 +1,7 @@
+test:
+  adapter: "<%= ENV["TEST_ADAPTER"] || "postgresql" %>"
+  database: "<%= ENV["TEST_DATABASE"] || "sequel_rails_test" %>"
+  owner: "<%= ENV["TEST_OWNER"] || ENV["USER"] %>"
+  username: "<%= ENV["TEST_USERNAME"] || ENV["USER"] %>"
+  password: "<%= ENV["TEST_PASSWORD"] %>"
+  encoding: "<%= ENV["TEST_ENCODING"] || "unicode" %>"

data/spec/internal/config/routes.rb

@@ -0,0 +1,3 @@
+Rails.application.routes.draw do
+  #
+end

data/spec/internal/db/schema.rb

@@ -0,0 +1,8 @@
+Sequel.migration do
+  change do
+    create_table :users do
+      primary_key :id
+      String      :email
+    end
+  end
+end

data/spec/lib/generators/sequel/migration_spec.rb

@@ -0,0 +1,256 @@
+require "spec_helper"
+require "generator_spec/test_case"
+require "generators/sequel/migration/migration_generator"
+
+describe Sequel::Generators::MigrationGenerator do
+  include GeneratorSpec::TestCase
+  destination File.expand_path("../../../../internal/tmp", __FILE__)
+
+  before { prepare_destination }
+
+  it "generates different ids for simultaneously generated migrations" do
+    migrations = ["create_authors", "create_users"]
+    first_number, second_number = migrations.collect do |migration_name|
+      run_generator [migration_name]
+      file_name = migration_file_name "db/migrate/#{migration_name}.rb"
+      File.basename(file_name).split("_").first
+    end
+    first_number.should_not == second_number
+  end
+
+  it "refuses to generate migration with invalid filename" do
+    expect do
+      run_generator ["add_something:datetime"]
+    end.to raise_error
+  end
+
+  context "when name starts with create" do
+    before { run_generator ["create_authors"] }
+    it "creates a new migration using change to create the table" do
+      destination_root.should have_structure {
+        directory "db" do
+          directory "migrate" do
+            migration "create_authors" do
+              contains <<-CONTENT.strip_heredoc
+              Sequel.migration do
+                change do
+                  create_table :authors do
+                    primary_key :id
+                  end
+                end
+              end
+              CONTENT
+            end
+          end
+        end
+      }
+    end
+  end
+
+  context "when name starts with drop" do
+    context "and does not contains _from_" do
+      before { run_generator ["drop_author"] }
+      it "creates a new migration using up/down to drop the table" do
+        destination_root.should have_structure {
+          directory "db" do
+            directory "migrate" do
+              migration "drop_author" do
+                contains <<-CONTENT.strip_heredoc
+                Sequel.migration do
+                  up do
+                    drop_table :authors
+                  end
+
+                  down do
+                    create_table :authors do
+                    end
+                  end
+                end
+                CONTENT
+              end
+            end
+          end
+        }
+      end
+    end
+    context "and contains _from_" do
+      before { run_generator ["drop_salary_from_authors"] }
+      it "creates a new migration using up/down to drop the column from table" do
+        destination_root.should have_structure {
+          directory "db" do
+            directory "migrate" do
+              migration "drop_salary_from_authors" do
+                contains <<-CONTENT.strip_heredoc
+                Sequel.migration do
+                  up do
+                    alter_table :authors do
+                    end
+                  end
+
+                  down do
+                    alter_table :authors do
+                    end
+                  end
+                end
+                CONTENT
+              end
+            end
+          end
+        }
+      end
+    end
+  end
+
+  context "when name starts with add" do
+    context "and does not contains _to_ nor _from_" do
+      before { run_generator ["add_new_indexes"] }
+      it "creates a new migration using up/down" do
+        destination_root.should have_structure {
+          directory "db" do
+            directory "migrate" do
+              migration "add_new_indexes" do
+                contains <<-CONTENT.strip_heredoc
+                Sequel.migration do
+                  up do
+                    alter_table :add_new_indexes do
+                    end
+                  end
+
+                  down do
+                    alter_table :add_new_indexes do
+                    end
+                  end
+                end
+                CONTENT
+              end
+            end
+          end
+        }
+      end
+    end
+    context "and contains _to_" do
+      before { run_generator ["add_salary_to_authors"] }
+      it "creates a new migration using change to add the column to the table" do
+        destination_root.should have_structure {
+          directory "db" do
+            directory "migrate" do
+              migration "add_salary_to_authors" do
+                contains <<-CONTENT.strip_heredoc
+                Sequel.migration do
+                  change do
+                    alter_table :authors do
+                    end
+                  end
+                end
+                CONTENT
+              end
+            end
+          end
+        }
+      end
+    end
+    context "and contains _from_" do
+      before { run_generator ["add_salary_from_authors"] }
+      it "creates a new migration using change to add the column to the table" do
+        destination_root.should have_structure {
+          directory "db" do
+            directory "migrate" do
+              migration "add_salary_from_authors" do
+                contains <<-CONTENT.strip_heredoc
+                Sequel.migration do
+                  change do
+                    alter_table :authors do
+                    end
+                  end
+                end
+                CONTENT
+              end
+            end
+          end
+        }
+      end
+    end
+  end
+
+  context "when name starts with remove" do
+    context "and does not contains _to_ nor _from_" do
+      before { run_generator ["remove_new_indexes"] }
+      it "creates a new migration using up/down" do
+        destination_root.should have_structure {
+          directory "db" do
+            directory "migrate" do
+              migration "remove_new_indexes" do
+                contains <<-CONTENT.strip_heredoc
+                Sequel.migration do
+                  up do
+                    alter_table :remove_new_indexes do
+                    end
+                  end
+
+                  down do
+                    alter_table :remove_new_indexes do
+                    end
+                  end
+                end
+                CONTENT
+              end
+            end
+          end
+        }
+      end
+    end
+    context "and contains _to_" do
+      before { run_generator ["remove_salary_to_authors"] }
+      it "creates a new migration using up/down to remove the column from the table" do
+        destination_root.should have_structure {
+          directory "db" do
+            directory "migrate" do
+              migration "remove_salary_to_authors" do
+                contains <<-CONTENT.strip_heredoc
+                Sequel.migration do
+                  up do
+                    alter_table :authors do
+                    end
+                  end
+
+                  down do
+                    alter_table :authors do
+                    end
+                  end
+                end
+                CONTENT
+              end
+            end
+          end
+        }
+      end
+    end
+    context "and contains _from_" do
+      before { run_generator ["remove_salary_from_authors"] }
+      it "creates a new migration using change to remove the column from the table" do
+        destination_root.should have_structure {
+          directory "db" do
+            directory "migrate" do
+              migration "remove_salary_from_authors" do
+                contains <<-CONTENT.strip_heredoc
+                Sequel.migration do
+                  up do
+                    alter_table :authors do
+                    end
+                  end
+
+                  down do
+                    alter_table :authors do
+                    end
+                  end
+                end
+                CONTENT
+              end
+            end
+          end
+        }
+      end
+    end
+  end
+
+end

data/spec/lib/sequel_rails/railtie_spec.rb

@@ -0,0 +1,85 @@
+require "spec_helper"
+
+describe SequelRails::Railtie do
+  it "registers ::Sequel::Railtie::LogSubscriber to receive :sequel notification" do
+    ActiveSupport::LogSubscriber.log_subscribers.select do |subscriber|
+      subscriber.is_a?(SequelRails::Railties::LogSubscriber)
+    end.should have(1).item
+  end
+
+  context "configures generator to use Sequel" do
+    it "as orm" do
+      Combustion::Application.config.generators.options[:rails][:orm].should == :sequel
+    end
+
+    it "for migrations" do
+      Combustion::Application.config.generators.options[:sequel][:migration].should be true
+    end
+  end
+
+  it "configures rails to use fancy pants logging" do
+    Combustion::Application.config.rails_fancy_pants_logging.should be true
+  end
+
+  context "configures action dispatch's rescue responses" do
+    let(:rescue_responses) do
+      Combustion::Application.config.action_dispatch.rescue_responses
+    end
+
+    it "to handle Sequel::Plugins::RailsExtensions::ModelNotFound with :not_found" do
+      rescue_responses["Sequel::Plugins::RailsExtensions::ModelNotFound"].should == :not_found
+    end
+
+    it "to handle Sequel::ValidationFailed with :unprocessable_entity" do
+      rescue_responses["Sequel::ValidationFailed"].should == :unprocessable_entity
+    end
+
+    it "to handle Sequel::NoExistingObject with :unprocessable_entity" do
+      rescue_responses["Sequel::NoExistingObject"].should == :unprocessable_entity
+    end
+  end
+
+  it "adds it's own database's rake tasks" do
+    pending "need to find a way to spec it"
+  end
+
+  it "stores it's own config in app.config.sequel" do
+    Combustion::Application.config.sequel.should be_instance_of SequelRails::Configuration
+  end
+
+  it "sets Rails.logger as default logger for its configuration" do
+    Combustion::Application.config.sequel.logger.should be Rails.logger
+  end
+
+  it "configures Sequel::Model instances for i18n" do
+    User.new.i18n_scope.should == :sequel
+  end
+
+  it "adds Sequel runtime to controller for logging" do
+    ActionController::Base.included_modules.should include(
+      SequelRails::Railties::ControllerRuntime
+    )
+  end
+
+  context "Sequel::Model is configured" do
+    let(:plugins) { Sequel::Model.plugins }
+    it "to use :active_model plugin" do
+      plugins.should include Sequel::Plugins::ActiveModel
+    end
+    it "to use :validation_helpers plugin" do
+      plugins.should include Sequel::Plugins::ValidationHelpers
+    end
+    it "to use :rails_extensions plugin" do
+      plugins.should include Sequel::Plugins::RailsExtensions
+    end
+    it "to not raise on save failure" do
+      Sequel::Model.raise_on_save_failure.should be false
+    end
+  end
+
+  it "configures database in Sequel" do
+    expect do
+      Sequel::Model.db.test_connection
+    end.to_not raise_error Sequel::Error
+  end
+end