active_record_uuid 0.0.1 → 0.1.0

data/spec/lib/serializer_spec.rb

@@ -0,0 +1,11 @@
+require 'spec_helper'
+
+describe "Serializer" do
+  it "should raise exception when loading invalid uuid" do
+    @serializer = ActiveRecordUuid::Serializer.new(:binary)
+
+    lambda {
+      @serializer.load("abkc")
+    }.should raise_error ActiveRecord::SerializationTypeMismatch
+  end
+end

data/spec/lib/string_uuid_spec.rb

@@ -0,0 +1,57 @@
+require 'spec_helper'
+
+describe "String Uuid" do
+  context "configure: Post model" do 
+    it "should have uuid as primary key based config" do
+      Post.primary_key.should eq('uuid')
+    end
+    
+    it "should store uuid as String" do
+      post = Post.create(:text => "String uuid1")
+      post.reload
+      
+      post.attributes_before_type_cast["uuid"]["value"].bytesize.should eq(36)
+    end
+    
+    it "should retreive back as uuid string from String" do
+      post = Post.create(:text => "String uuid2")
+      post.reload
+      
+      post.uuid.should be_present
+      post.uuid.should be_instance_of(String)
+      post.uuid.length.should eq(36)
+    end
+    
+    it "should retreive uuid back with the same value that was assigned" do
+      post = Post.new(:text => "String uuid2")
+      post.uuid = "b360c78e-b62e-11e1-9870-0026b90faf3c"
+      post.save
+      post.reload
+      
+      post.uuid.should eq("b360c78e-b62e-11e1-9870-0026b90faf3c")
+      post.uuid.should be_instance_of(String)
+      post.uuid.length.should eq(36)
+    end
+    
+    it "should find by uuid column" do
+      post = Post.create(:text => "String uuid3")
+
+      Post.find_by_uuid(post.uuid).should   eq(post)
+      Post.where(:uuid => post.uuid).should eq([post])
+    end
+    
+    it "should find by primary key" do
+      post = Post.create(:text => "String uuid4")
+
+      Post.find(post).should eq(post)
+      Post.find(post.uuid).should eq(post)
+    end
+    
+    it "should find by array of primary keys" do
+      post1 = Post.create(:text => "String uuid5")
+      post2 = Post.create(:text => "String uuid6")
+
+      Post.find([post1.uuid, post2.uuid]).should eq([post1, post2])
+    end
+  end
+end

data/spec/lib/uuid_config_spec.rb

@@ -0,0 +1,100 @@
+require 'spec_helper'
+
+describe "UuidConfig" do  
+  context "uuid base configuration - default option" do
+    it "should not use primary_key" do
+      Article.primary_key.should eq("id")
+    end
+    
+    it "should not set association" do
+      Article.reflections[:comments].options[:foreign_key].should be(nil)
+    end
+    
+    it "should generate uuid string on uuid column" do
+      article = Article.create(:title => "Hello World")
+      
+      article.uuid_valid?.should be_true
+    end
+    
+    it "should generate uuid on before_validation" do
+      article = Article.new(:title => "Hello World")
+      
+      article.valid?.should be_true
+    end
+  end
+  
+  context "generate and validate uuid" do
+    it "should validate duplicate uuid" do
+      article1 = Article.create(:uuid => '8df4689b-b580-11e1-9d31-0026b90faf3c', :title => "Hello World1")
+      article2 = Article.new(:uuid => '8df4689b-b580-11e1-9d31-0026b90faf3c', :title => "Hello World2")
+      
+      article2.valid?.should be_false
+      article2.errors[:uuid].first.should be_include("been taken")
+    end
+    
+    it "should assign uuid manually" do
+      article = Article.new(:title => "Manual uuid")
+      article.uuid = "79f8a42e-ae60-11e1-9aa9-0026b90faf3c"
+      article.save
+      article.reload
+
+      article.uuid.should eq("79f8a42e-ae60-11e1-9aa9-0026b90faf3c")
+    end
+    
+    it "should assign uuid if blank" do
+      article = Article.new(:title => "Manual uuid 2")
+      article.assign_uuid
+
+      article.uuid.should be_present
+      article.uuid_valid?.should be_true
+    end
+
+    it "should assign uuid if blank and save immediately" do
+      article = Article.new(:title => "Manual uuid 3")
+      article.assign_uuid!
+
+      article.uuid.should be_present
+      article.uuid_valid?.should be_true
+    end
+
+    it "should have valid uuid" do
+      article = Article.new(:title => "Invalid uuid")
+      article.uuid = "invalid"
+
+      article.valid?.should eq(false)
+      article.errors[:uuid].first.should be_include("is invalid")
+    end
+    
+    it "should generate uuid" do
+      uuid = Article.generate_uuid
+      
+      uuid.should be_present
+      uuid.should be_instance_of(String)
+      uuid.length.should eq(36)
+    end
+    
+    it "should generate uuid on :after_initialize" do
+      class AfterInitializeArticle < ActiveRecord::Base
+        self.table_name = "articles"
+        uuid_config do
+          hook :after_initialize
+        end
+      end
+      article = AfterInitializeArticle.new(:title => "After initialize uuid")
+      
+      article.uuid_valid?.should be_true
+    end
+
+    it "should generate uuid on :before_create" do
+      class BeforeCreateArticle < ActiveRecord::Base
+        self.table_name = "articles"
+        uuid_config do
+          hook :before_create
+        end
+      end
+      article = BeforeCreateArticle.create(:title => "Before create uuid")
+      
+      article.uuid_valid?.should be_true
+    end
+  end
+end

data/spec/spec_helper.rb

@@ -1,9 +1,28 @@
 require 'active_record_uuid'
 
-ActiveRecord::Base.establish_connection(
-  :adapter => "sqlite3", 
-  :database => File.dirname(__FILE__) + "/active_record_uuid.sqlite3"
-)
+db_config = {
+  :adapter => "mysql2", 
+  :database => "active_record_uuid",
+  :user => "root",
+  :password => "Q1p2m3g4"
+}
 
+ActiveRecord::Base.establish_connection(db_config) rescue nil
+ActiveRecord::Base.connection.drop_database(db_config[:database]) rescue nil
+ActiveRecord::Base.establish_connection(db_config.merge(:database => nil))
+ActiveRecord::Base.connection.create_database(db_config[:database], { :charset => 'utf8', :collation => 'utf8_unicode_ci' })
+ActiveRecord::Base.establish_connection(db_config)
+
+# load extension
+require 'active_record_uuid/extensions/quoting_extension'
+::ActiveRecord::Base.connection.class.send :include, ActiveRecordUuid::QuotingExtension
+::ActiveRecord::Base.send(:include, ActiveRecordUuid::Model)
+
+# load support
 load File.dirname(__FILE__) + '/support/schema.rb'
-load File.dirname(__FILE__) + '/support/models.rb'
+load File.dirname(__FILE__) + '/support/models.rb'
+
+RSpec.configure do |config|
+  config.filter_run :focus => true
+  config.run_all_when_everything_filtered = true
+end

data/spec/support/models.rb

@@ -1,22 +1,74 @@
-class PostInherit < ActiveRecord::UuidBase
-  self.table_name = "posts"
+class Blog < ActiveRecord::Base
+  uuid_config do
+    association true
+    primary_key true
+    store_as    :binary
+  end
+
+  has_many :articles
+end
+
+class Article < ActiveRecord::Base
+  uuid_config
+  
+  has_many :comments
+end
+
+class ArticleUuidBase < ActiveRecord::UuidBase
+  self.table_name = "articles"
+
+  has_many :comments
+end
+
+class PostBinary < ActiveRecord::Base
+  uuid_config do
+    association true
+    primary_key true
+    store_as    :binary
+  end
+  
+  has_many :comments
 end
 
-class PostInclude < ActiveRecord::Base
-  include UuidBaseHelper
-  self.table_name = "posts"
+class PostBase64 < ActiveRecord::Base
+  uuid_config do
+    primary_key true
+    store_as    :base64
+  end
 end
 
-class Author < ActiveRecord::UuidBase
+class PostHexDigest < ActiveRecord::Base
+  uuid_config do
+    primary_key true
+    store_as    :hexdigest
+  end
+end
+
+class Author < ActiveRecord::Base
+  uuid_config do
+    association true
+    primary_key true
+  end
+
   has_and_belongs_to_many :posts
 end
 
-class Post < ActiveRecord::UuidBase
+class Post < ActiveRecord::Base
+  uuid_config do
+    primary_key true
+    association true
+  end
+  
   has_many :comments
   has_one  :comment
   has_and_belongs_to_many :authors
 end
 
-class Comment < ActiveRecord::UuidBase
+class Comment < ActiveRecord::Base
+  uuid_config do
+    association true
+    primary_key true
+  end
+  
   belongs_to :post
 end

data/spec/support/schema.rb

@@ -1,6 +1,36 @@
 ActiveRecord::Schema.define do
   self.verbose = false
 
+  create_table :blogs, :force => true, :id => false do |t|
+    t.binary :uuid, :limit => 16
+    t.string :name
+    t.timestamps
+  end
+
+  create_table :articles, :force => true do |t|
+    t.string :title
+    t.string :uuid, :limit => 36
+    t.binary :blog_uuid, :limit => 16
+  end
+  
+  create_table :post_binaries, :force => true, :id => false do |t|
+    t.binary :uuid, :limit => 16
+    t.string :text
+    t.timestamps
+  end
+  
+  create_table :post_base64s, :force => true, :id => false do |t|
+    t.string :uuid, :limit => 24
+    t.string :text
+    t.timestamps
+  end
+  
+  create_table :post_hex_digests, :force => true, :id => false do |t|
+    t.string :uuid, :limit => 32
+    t.string :text
+    t.timestamps
+  end
+  
   create_table :posts, :force => true, :id => false do |t|
     t.string :uuid, :limit => 36
     t.string :text

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: active_record_uuid
 version: !ruby/object:Gem::Version
-  version: 0.0.1
+  version: 0.1.0
   prerelease: 
 platform: ruby
 authors:
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2012-06-07 00:00:00.000000000 Z
+date: 2012-06-20 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler
@@ -44,7 +44,7 @@ dependencies:
       - !ruby/object:Gem::Version
         version: 2.8.0
 - !ruby/object:Gem::Dependency
-  name: sqlite3
+  name: mysql2
   requirement: !ruby/object:Gem::Requirement
     none: false
     requirements:
@@ -91,7 +91,26 @@ dependencies:
     - - ~>
       - !ruby/object:Gem::Version
         version: 2.1.2
-description: A nice gem to add uuids to your models, associations, and schema.rb
+- !ruby/object:Gem::Dependency
+  name: mysql2
+  requirement: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: '0'
+description: ! 'active_record_uuid is a nice gem that add uuid supports to your activerecord
+  models (MySQL). It allows you to store uuid in various formats: binary (16 bytes),
+  base64 (24 bytes), hexdigest (32 bytes), or string (36 bytes), and query back with
+  uuid string.'
 email:
 - chamnapchhorn@gmail.com
 executables: []
@@ -107,14 +126,25 @@ files:
 - Rakefile
 - active_record_uuid.gemspec
 - lib/active_record_uuid.rb
-- lib/active_record_uuid/railtie.rb
-- lib/active_record_uuid/tasks/db.rake
+- lib/active_record_uuid/association_methods.rb
+- lib/active_record_uuid/config.rb
+- lib/active_record_uuid/extensions/quoting_extension.rb
+- lib/active_record_uuid/model.rb
+- lib/active_record_uuid/rails/db.rake
+- lib/active_record_uuid/rails/railtie.rb
+- lib/active_record_uuid/serializer.rb
 - lib/active_record_uuid/uuid_base.rb
 - lib/active_record_uuid/uuid_base_helper.rb
 - lib/active_record_uuid/version.rb
 - spec/lib/association_spec.rb
-- spec/lib/uuid_base_helper_spec.rb
-- spec/lib/uuid_base_spec.rb
+- spec/lib/base64_uuid_spec.rb
+- spec/lib/binary_uuid_spec.rb
+- spec/lib/config_spec.rb
+- spec/lib/deprecation_spec.rb
+- spec/lib/hexdigest_uuid_spec.rb
+- spec/lib/serializer_spec.rb
+- spec/lib/string_uuid_spec.rb
+- spec/lib/uuid_config_spec.rb
 - spec/spec_helper.rb
 - spec/support/models.rb
 - spec/support/schema.rb
@@ -138,14 +168,20 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 1.8.21
+rubygems_version: 1.8.24
 signing_key: 
 specification_version: 3
-summary: A gem for adding uuids to your active record models
+summary: A full-featured gem for adding uuid support to your active record models
 test_files:
 - spec/lib/association_spec.rb
-- spec/lib/uuid_base_helper_spec.rb
-- spec/lib/uuid_base_spec.rb
+- spec/lib/base64_uuid_spec.rb
+- spec/lib/binary_uuid_spec.rb
+- spec/lib/config_spec.rb
+- spec/lib/deprecation_spec.rb
+- spec/lib/hexdigest_uuid_spec.rb
+- spec/lib/serializer_spec.rb
+- spec/lib/string_uuid_spec.rb
+- spec/lib/uuid_config_spec.rb
 - spec/spec_helper.rb
 - spec/support/models.rb
 - spec/support/schema.rb

data/lib/active_record_uuid/railtie.rb

@@ -1,7 +0,0 @@
-module ActiveRecordUuid
-  class Railtie < Rails::Railtie
-  	rake_tasks do
-      load "active_record_uuid/tasks/db.rake"
-    end
-  end
-end

data/lib/active_record_uuid/tasks/db.rake

@@ -1,26 +0,0 @@
-namespace :db do
-  task :update_schema do
-    system "sed -i '/:primary_key => \"uuid\"/ a\
-    t.string   \"uuid\",         :limit => 36,  :default => \"\",   :null => false
-' db/schema.rb"
-    system "sed -i 's/:primary_key => \"uuid\"/:id => false/' db/schema.rb"
-  end
-  
-  # Append behavior to rails rake task to correct db/schema.rb, so that the test would work correctly
-  Rake::Task["db:schema:dump"].enhance do
-    Rake::Task["db:update_schema"].invoke
-  end
-  
-  task :add_primary_keys => :environment do
-    Dir["#{Rails.root}/app/models/**/*.rb"].each { |path| require path }
-    ActiveRecord::Base.descendants.each do |model|
-      next if model.abstract_class
-      
-      begin
-        ActiveRecord::Base.connection.execute "ALTER TABLE #{model.table_name} ADD PRIMARY KEY (#{model.primary_key});"
-      rescue => e
-        p e
-      end
-    end
-  end
-end