dm-types 0.9.11 → 0.10.0

data/spec/fixtures/ticket.rb

@@ -0,0 +1,25 @@
+module DataMapper
+  module Types
+    module Fixtures
+      class Ticket
+        #
+        # Behaviors
+        #
+
+        include DataMapper::Resource
+        include DataMapper::Validate
+
+        #
+        # Properties
+        #
+
+        property :id,     Serial
+        property :title,  String, :length => 255
+        property :body,   Text
+        property :status, Enum[:unconfirmed, :confirmed, :assigned, :resolved, :not_applicable]
+
+        auto_migrate!
+      end # Ticket
+    end
+  end
+end

data/spec/fixtures/tshirt.rb

@@ -0,0 +1,28 @@
+module DataMapper
+  module Types
+    module Fixtures
+
+      class TShirt
+        #
+        # Behaviors
+        #
+
+        include ::DataMapper::Resource
+
+        #
+        # Properties
+        #
+
+        property :id,          Serial
+        property :writing,     String
+        property :has_picture, Boolean, :default => false
+        property :picture,     Enum[:octocat, :fork_you, :git_down]
+
+        property :color, Enum[:white, :black, :red, :orange, :yellow, :green, :cyan, :blue, :purple]
+        property :size,  Flag[:xs, :small, :medium, :large, :xl, :xxl]
+
+        auto_migrate!
+      end # Shirt
+    end # Fixtures
+  end # Types
+end # DataMapper

data/spec/integration/bcrypt_hash_spec.rb

@@ -1,59 +1,41 @@
-require 'pathname'
-require Pathname(__FILE__).dirname.parent.expand_path + 'spec_helper'
-
-begin
-  gem 'bcrypt-ruby', '~>2.0.5'
-  require 'bcrypt'
-rescue LoadError
-  skip_tests = true
-end
+require 'spec_helper'
+
+try_spec do
+
+  require 'spec/fixtures/person'
+
+  describe DataMapper::Types::Fixtures::Person do
+    before :all  do
+      @resource  = DataMapper::Types::Fixtures::Person.create(:password => 'DataMapper R0cks!')
+      DataMapper::Types::Fixtures::Person.create(:password => 'password1')
+
+      @people = DataMapper::Types::Fixtures::Person.all
+      @resource.reload
+    end
+
+    it 'persists the password on initial save' do
+      @resource.password.should       == 'DataMapper R0cks!'
+      @people.last.password.should == 'password1'
+    end
+
+    it 'recalculates password hash on attribute update' do
+      @resource.attribute_set(:password, 'bcryptic obscure')
+      @resource.save
+
+      @resource.reload
+      @resource.password.should     == 'bcryptic obscure'
+      @resource.password.should_not == 'DataMapper R0cks!'
+    end
+
+    it 'does not change password value on reload' do
+      resource = @people.last
+      original = resource.password.to_s
+      resource.reload
+      resource.password.to_s.should == original
+    end
 
-describe 'DataMapper::Types::BCryptHash' do
-  unless skip_tests
-    describe "with no options" do
-      before(:each) do
-        class User
-          include DataMapper::Resource
-
-          property :id, Serial
-          property :password, BCryptHash
-        end
-        User.auto_migrate!
-        User.create(:password => "DataMapper R0cks!")
-      end
-
-      it "should save a password to the DB on creation" do
-        repository(:default) do
-          User.create(:password => "password1")
-        end
-        user = User.all
-        user[0].password.should == "DataMapper R0cks!"
-        user[1].password.should == "password1"
-      end
-
-      it "should change the password on attribute update" do
-        @user = User.first
-        @user.attribute_set(:password, "D@t@Mapper R0cks!")
-        @user.save
-        @user.password.should_not == "DataMapper R0cks!"
-        @user.password.should == "D@t@Mapper R0cks!"
-      end
-
-      it "should not change the password on save and reload" do
-        @user = User.first
-        v1 = @user.password.to_s
-        @user.save
-        @user.reload
-        v2 = @user.password.to_s
-        v1.should == v2
-      end
-
-      it "should have a cost of BCrypt::Engine::DEFAULT_COST" do
-        @user = User.first
-        @user.password.cost.should == BCrypt::Engine::DEFAULT_COST
-      end
+    it 'uses cost of BCrypt::Engine::DEFAULT_COST' do
+      @resource.password.cost.should == BCrypt::Engine::DEFAULT_COST
     end
-  else
-    it "Needs the bcrypt-ruby gem installed"
   end
 end

data/spec/integration/comma_separated_list_spec.rb

@@ -0,0 +1,89 @@
+require 'spec_helper'
+
+try_spec do
+
+  require 'spec/fixtures/person'
+
+  describe DataMapper::Types::Fixtures::Person do
+    before :all do
+      @resource = DataMapper::Types::Fixtures::Person.new(:name => '')
+    end
+
+    describe 'with no interests information' do
+      before :all do
+        @resource.interests = nil
+      end
+
+      describe 'when dumped and loaded again' do
+        before :all do
+          @resource.save.should be_true
+          @resource.reload
+        end
+
+        it 'has no interests' do
+          @resource.interests.should == nil
+        end
+      end
+    end
+
+    describe 'with no interests information' do
+      before :all do
+        @resource.interests = []
+      end
+
+      describe 'when dumped and loaded again' do
+        before :all do
+          @resource.save.should be_true
+          @resource.reload
+        end
+
+        it 'has empty interests list' do
+          @resource.interests.should == []
+        end
+      end
+    end
+
+    describe 'with interests information given as a Hash' do
+      it 'raises ArgumentError' do
+        lambda do
+          @resource.interests = { :hash => 'value' }
+          @resource.save
+        end.should raise_error(ArgumentError, /must be a string, an array or nil/)
+      end
+    end
+
+    describe 'with a few items on the interests list' do
+      before :all do
+        @input = 'fire, water, fire, a whole lot of other interesting things, ,,,'
+        @resource.interests = @input
+      end
+
+      describe 'when dumped and loaded again' do
+        before :all do
+          @resource.save.should be_true
+          @resource.reload
+        end
+
+        it 'includes "fire" in interests' do
+          @resource.interests.should include('fire')
+        end
+
+        it 'includes "water" in interests' do
+          @resource.interests.should include('water')
+        end
+
+        it 'includes "a whole lot of other interesting things" in interests' do
+          @resource.interests.should include('a whole lot of other interesting things')
+        end
+
+        it 'has blank entries removed' do
+          @resource.interests.any? { |i| i.blank? }.should be_false
+        end
+
+        it 'has duplicates removed' do
+          @resource.interests.select { |i| i == 'fire' }.size.should == 1
+        end
+      end
+    end
+  end
+end

data/spec/integration/enum_spec.rb

@@ -1,57 +1,77 @@
-require 'pathname'
-require Pathname(__FILE__).dirname.parent.expand_path + 'spec_helper'
+require 'spec_helper'
 
-describe DataMapper::Types::Enum do
-  before(:all) do
-    class ::Bug
-      include DataMapper::Resource
+try_spec do
 
-      property :id, Integer, :serial => true
-      property :status, Enum[:crit, :warn, :info, :unknown]
+  require 'spec/fixtures/ticket'
+
+  describe DataMapper::Types::Fixtures::Ticket do
+    describe 'that is dumped and then loaded' do
+      before :all do
+        @resource = DataMapper::Types::Fixtures::Ticket.new(
+          :title  => "Can't order by aggregated fields",
+          :id     => 789,
+          :body   => "I'm trying to use the aggregate method and sort the results by a summed field, but it doesn't work.",
+          :status => 'confirmed'
+        )
+
+        @resource.save.should be_true
+        @resource.reload
+      end
+
+      it 'preserves property value' do
+        @resource.status.should == :confirmed
+      end
     end
-    Bug.auto_migrate!
-  end
 
-  it "should work" do
-    repository(:default) do
-      Bug.create(:status => :crit)
-      Bug.create(:status => :warn)
+    describe 'that is supplied a matching enumeration value' do
+      before :all do
+        @resource = DataMapper::Types::Fixtures::Ticket.new(:status => :assigned)
+      end
+
+      it 'typecasts it for outside reader' do
+        @resource.status.should == :assigned
+      end
     end
 
-    bugs = Bug.all
-    bugs[0].status.should == :crit
-    bugs[1].status.should == :warn
-  end
+    describe '#get' do
+      before :all do
+        @resource = DataMapper::Types::Fixtures::Ticket.new(
+          :title  => '"sudo make install" of drizzle fails because it tries to chown mysql',
+          :id     => 257497,
+          :body   => "Note that at the very least, there should be a check to see whether or not the user is created before chown'ing a file to the user.",
+          :status => 'confirmed'
+        )
+        @resource.save.should be_true
+      end
 
-  it 'should immediately typecast supplied values' do
-    Bug.new(:status => :crit).status.should == :crit
-  end
+      it 'supports queries with equality operator on enumeration property' do
+        DataMapper::Types::Fixtures::Ticket.all(:status => :confirmed).
+          should include(@resource)
+      end
 
-  describe "with finders" do
-    before(:all) do
-      @info = Bug.create(:status => :info)
-    end
-    it "should work with equality opeand" do
-      Bug.all(:status => [:info, :unknown]).entries.should == [@info]
-    end
-    it "should work with inequality operand" do
-      Bug.all(:status.not => [:crit, :warn]).entries.should == [@info]
+      it 'supports queries with inequality operator on enumeration property' do
+        DataMapper::Types::Fixtures::Ticket.all(:status.not => :confirmed).
+          should_not include(@resource)
+      end
     end
-  end
 
-  if defined?(Validate)
-    describe 'with validation' do
-      it "should accept crit status" do
-        bug = Bug.new
-        bug.status = :crit
-        bug.should be_valid
+    describe 'with value unknown to enumeration property' do
+      before :all do
+        @resource = DataMapper::Types::Fixtures::Ticket.new(:status => :undecided)
+      end
+
+      # TODO: consider sharing shared spec exampels with dm-validations,
+      #       which has 'invalid model' shared group
+      it 'is invalid (auto validation for :within kicks in)' do
+        @resource.should_not be_valid
+      end
+
+      it 'has errors' do
+        @resource.errors.should_not be_empty
       end
 
-      it "should not accept blah status" do
-        bug = Bug.new
-        bug.status = :blah
-        bug.should_not be_valid
-        bug.errors.on(:status).should_not be_empty
+      it 'has a meaningful error message on invalid property' do
+        @resource.errors.on(:status).should include('Status must be one of unconfirmed, confirmed, assigned, resolved, not_applicable')
       end
     end
   end

data/spec/integration/file_path_spec.rb

@@ -1,26 +1,158 @@
-require 'pathname'
-require Pathname(__FILE__).dirname.parent.expand_path + 'spec_helper'
+require 'spec_helper'
 
-describe DataMapper::Types::FilePath do
-  before(:all) do
-    class ::FilePathTest
-      include DataMapper::Resource
+try_spec do
 
-      property :id, Integer, :serial => true
-      property :path, FilePath
+  require 'spec/fixtures/software_package'
+
+  describe DataMapper::Types::Fixtures::SoftwarePackage do
+    describe 'with source path at /var/cache/apt/archives/linux-libc-dev_2.6.28-11.40_i386.deb' do
+      before :all do
+        @source_path = '/var/cache/apt/archives/linux-libc-dev_2.6.28-11.40_i386.deb'
+        @resource    = DataMapper::Types::Fixtures::SoftwarePackage.new(:source_path => @source_path)
+      end
+
+      describe 'when is a new record' do
+        before :all do
+        end
+
+        it 'points to original path' do
+          @resource.source_path.to_s.should == @source_path
+        end
+
+        it 'responds to :directory?' do
+          @resource.source_path.should respond_to(:directory?)
+        end
+
+        it 'responds to :file?' do
+          @resource.source_path.should respond_to(:file?)
+        end
+
+        it 'responds to :dirname' do
+          @resource.source_path.should respond_to(:dirname)
+        end
+
+        it 'responds to :absolute?' do
+          @resource.source_path.should respond_to(:absolute?)
+        end
+
+        it 'responds to :readable?' do
+          @resource.source_path.should respond_to(:readable?)
+        end
+
+        it 'responds to :size' do
+          @resource.source_path.should respond_to(:size)
+        end
+      end
     end
-    FilePathTest.auto_migrate!
-  end
 
-  it "should work" do
-    repository(:default) do
-      FilePathTest.create(:path => '/usr')
+    describe 'with destination path at /usr/local' do
+      before :all do
+        @destination_path = '/usr/local'
+        @resource         = DataMapper::Types::Fixtures::SoftwarePackage.new(:destination_path => @destination_path)
+      end
+
+      describe 'when saved and reloaded' do
+        before :all do
+          @resource.save.should be_true
+          @resource.reload
+        end
+
+        it 'points to original path' do
+          @resource.destination_path.to_s.should == @destination_path
+        end
+
+        it 'responds to :directory?' do
+          @resource.destination_path.should respond_to(:directory?)
+        end
+
+        it 'responds to :file?' do
+          @resource.destination_path.should respond_to(:file?)
+        end
+
+        it 'responds to :dirname' do
+          @resource.destination_path.should respond_to(:dirname)
+        end
+
+        it 'responds to :absolute?' do
+          @resource.destination_path.should respond_to(:absolute?)
+        end
+
+        it 'responds to :readable?' do
+          @resource.destination_path.should respond_to(:readable?)
+        end
+
+        it 'responds to :size' do
+          @resource.destination_path.should respond_to(:size)
+        end
+      end
     end
 
-    FilePathTest.first.path.should == Pathname.new('/usr')
-  end
+    describe 'with no (nil) source path' do
+      before :all do
+        @source_path = nil
+        @resource    = DataMapper::Types::Fixtures::SoftwarePackage.new(:source_path => @source_path)
+      end
+
+      describe 'when saved and reloaded' do
+        before :all do
+          @resource.save.should be_true
+          @resource.reload
+        end
+
+        it 'has nil source path' do
+          @resource.source_path.should be_nil
+        end
+      end
+    end
+
+    describe 'with a blank source path' do
+      before :all do
+        @source_path = ''
+        @resource    = DataMapper::Types::Fixtures::SoftwarePackage.new(:source_path => @source_path)
+      end
+
+      describe 'when saved and reloaded' do
+        before :all do
+          @resource.save.should be_true
+          @resource.reload
+        end
 
-  it 'should immediately typecast supplied values' do
-    FilePathTest.new(:path => '/usr').path.should == Pathname.new('/usr')
+        it 'has nil source path' do
+          @resource.source_path.should be_nil
+        end
+      end
+    end
+
+    describe 'with a source path assigned to an empty array' do
+      before :all do
+        @source_path = []
+        @resource    = DataMapper::Types::Fixtures::SoftwarePackage.new(:source_path => @source_path)
+      end
+
+      describe 'when saved and reloaded' do
+        before :all do
+          @resource.save.should be_true
+          @resource.reload
+        end
+
+        it 'has nil source path' do
+          @resource.source_path.should be_nil
+        end
+      end
+    end
+
+    describe 'with a source path assigned to a Hash' do
+      before :all do
+        @source_path = { :guitar => 'Joe Satriani' }
+      end
+
+      describe 'when instantiated' do
+        it 'raises an exception' do
+          lambda do
+            DataMapper::Types::Fixtures::SoftwarePackage.new(:source_path => @source_path)
+          end.should raise_error(TypeError)
+        end
+      end
+    end
   end
 end