sequel_migration_builder 0.2.0 → 0.2.1

data/CHANGELOG

@@ -1,5 +1,9 @@
 CHANGELOG
 
+== 0.2.1
+
+* Bug fix - cast :unique => nil to false in DbIndex.
+
 == 0.2.0
 
 * Switched to using the new style 'change' block in the generated

data/VERSION

@@ -1 +1 @@
-0.2.0
+0.2.1

data/lib/sequel/schema/db_index.rb

@@ -19,12 +19,12 @@ module Sequel
       def initialize(name, columns, unique=false)
         @name = name.to_sym
         @columns = columns.kind_of?(Array) ? columns.clone : [columns]
-        @unique = unique
+        @unique = !! unique
       end
 
       # Returns true if this index is unique
       def unique?
-        !! @unique
+        @unique
       end
 
       # Returns true if this index has more than one column.

data/sequel_migration_builder.gemspec

@@ -5,7 +5,7 @@
 
 Gem::Specification.new do |s|
   s.name = %q{sequel_migration_builder}
-  s.version = "0.2.0"
+  s.version = "0.2.1"
 
   s.required_rubygems_version = Gem::Requirement.new(">= 0") if s.respond_to? :required_rubygems_version=
   s.authors = ["Roland Swingler"]

data/spec/alter_table_operations_spec.rb

@@ -127,4 +127,14 @@ describe "Sequel::Schema::AlterTableOperations.build" do
 
     ops.last.should be_nil
   end
+
+  it "should not output an add_index statement if there is nothing to be done" do
+    table_a = {:name => :example_table,
+      :indexes => {:foo_idx => {:columns => [:foo]}},
+      :columns => []}
+    table_b = {:name => :example_table, :indexes => {:foo_idx => {:columns => [:foo]}}, :columns => []}
+
+    ops = Sequel::Schema::AlterTableOperations.build(table_a,table_b)
+    ops.should == []
+  end
 end

data/spec/db_index_spec.rb

@@ -42,6 +42,10 @@ describe Sequel::Schema::DbIndex do
     i1.hash.should == Sequel::Schema::DbIndex.new('foo_idx', :foo, true).hash
     i1.should be_eql(Sequel::Schema::DbIndex.new('foo_idx', :foo, true))
   end
+
+  it "should ensure nil as unique is converted to false" do
+    ([Sequel::Schema::DbIndex.new(:foo_idx, :foo, :unique => nil)] - [Sequel::Schema::DbIndex.new(:foo_idx, :foo, :unique => false)]).should == []
+  end
   
   it "can be built from a hash returned by Sequel::Database#indexes" do
     hsh = {:foo_idx => {:columns => [:foo], :unique => true}}

metadata

@@ -1,13 +1,13 @@
 --- !ruby/object:Gem::Specification 
 name: sequel_migration_builder
 version: !ruby/object:Gem::Version 
-  hash: 23
+  hash: 21
   prerelease: false
   segments: 
   - 0
   - 2
-  - 0
-  version: 0.2.0
+  - 1
+  version: 0.2.1
 platform: ruby
 authors: 
 - Roland Swingler