sequel-rails 0.7.0 → 0.8.0

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

@@ -1,36 +1,36 @@
-require "spec_helper"
-require "generator_spec/test_case"
-require "generators/sequel/migration/migration_generator"
+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__)
+  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|
+  it 'generates different ids for simultaneously generated migrations' do
+    migrations = %w(create_authors create_users)
+    first_number, second_number = migrations.map do |migration_name|
       run_generator [migration_name]
       file_name = migration_file_name "db/migrate/#{migration_name}.rb"
-      File.basename(file_name).split("_").first
+      File.basename(file_name).split('_').first
     end
-    first_number.should_not == second_number
+    expect(first_number).to_not eq(second_number)
   end
 
-  it "refuses to generate migration with invalid filename" do
+  it 'refuses to generate migration with invalid filename' do
     expect do
-      run_generator ["add_something:datetime"]
+      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
+  context 'when name starts with create' do
+    before { run_generator ['create_authors'] }
+    it 'creates a new migration using change to create the table' do
+      expect(destination_root).to have_structure {
+        directory 'db' do
+          directory 'migrate' do
+            migration 'create_authors' do
               contains <<-CONTENT.strip_heredoc
               Sequel.migration do
                 change do
@@ -47,14 +47,14 @@ describe Sequel::Generators::MigrationGenerator do
     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
+  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
+        expect(destination_root).to have_structure {
+          directory 'db' do
+            directory 'migrate' do
+              migration 'drop_author' do
                 contains <<-CONTENT.strip_heredoc
                 Sequel.migration do
                   up do
@@ -73,13 +73,13 @@ describe Sequel::Generators::MigrationGenerator do
         }
       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
+    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
+        expect(destination_root).to have_structure {
+          directory 'db' do
+            directory 'migrate' do
+              migration 'drop_salary_from_authors' do
                 contains <<-CONTENT.strip_heredoc
                 Sequel.migration do
                   up do
@@ -101,14 +101,14 @@ describe Sequel::Generators::MigrationGenerator do
     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
+  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
+        expect(destination_root).to have_structure {
+          directory 'db' do
+            directory 'migrate' do
+              migration 'add_new_indexes' do
                 contains <<-CONTENT.strip_heredoc
                 Sequel.migration do
                   up do
@@ -128,13 +128,13 @@ describe Sequel::Generators::MigrationGenerator do
         }
       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
+    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
+        expect(destination_root).to have_structure {
+          directory 'db' do
+            directory 'migrate' do
+              migration 'add_salary_to_authors' do
                 contains <<-CONTENT.strip_heredoc
                 Sequel.migration do
                   change do
@@ -149,13 +149,13 @@ describe Sequel::Generators::MigrationGenerator do
         }
       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
+    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
+        expect(destination_root).to have_structure {
+          directory 'db' do
+            directory 'migrate' do
+              migration 'add_salary_from_authors' do
                 contains <<-CONTENT.strip_heredoc
                 Sequel.migration do
                   change do
@@ -172,14 +172,14 @@ describe Sequel::Generators::MigrationGenerator do
     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
+  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
+        expect(destination_root).to have_structure {
+          directory 'db' do
+            directory 'migrate' do
+              migration 'remove_new_indexes' do
                 contains <<-CONTENT.strip_heredoc
                 Sequel.migration do
                   up do
@@ -199,13 +199,13 @@ describe Sequel::Generators::MigrationGenerator do
         }
       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
+    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
+        expect(destination_root).to have_structure {
+          directory 'db' do
+            directory 'migrate' do
+              migration 'remove_salary_to_authors' do
                 contains <<-CONTENT.strip_heredoc
                 Sequel.migration do
                   up do
@@ -225,13 +225,13 @@ describe Sequel::Generators::MigrationGenerator do
         }
       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
+    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
+        expect(destination_root).to have_structure {
+          directory 'db' do
+            directory 'migrate' do
+              migration 'remove_salary_from_authors' do
                 contains <<-CONTENT.strip_heredoc
                 Sequel.migration do
                   up do

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

@@ -0,0 +1,41 @@
+require 'spec_helper'
+require 'generator_spec/test_case'
+require 'generators/sequel/session_migration/session_migration_generator'
+
+describe Sequel::Generators::SessionMigrationGenerator do
+  include GeneratorSpec::TestCase
+  destination File.expand_path('../../../../internal/tmp', __FILE__)
+
+  before { prepare_destination }
+
+  it 'refuses to generate migration with invalid filename' do
+    expect do
+      run_generator ['add:sessions']
+    end.to raise_error
+  end
+
+  it 'creates a new migration for sessions table' do
+    run_generator
+    expect(destination_root).to have_structure {
+      directory 'db' do
+        directory 'migrate' do
+          migration 'add_sessions_table' do
+            contains <<-CONTENT.strip_heredoc
+            Sequel.migration do
+              change do
+                create_table :sessions do
+                  primary_key :id
+                  String :session_id, :null => false, :unique => true, :index => true
+                  String :data, :text => true, :null => false
+                  DateTime :updated_at, :null => true, :index => true
+                end
+              end
+            end
+            CONTENT
+          end
+        end
+      end
+    }
+  end
+
+end