tire_async_index 0.1.0 → 0.2.0

checksums.yaml

@@ -1,15 +1,7 @@
 ---
-!binary "U0hBMQ==":
-  metadata.gz: !binary |-
-    Zjg2NTkzM2I5ZGE4MDE1YmRiYjY5NzhjOGU0NWFlMzZmNTI0OTcyZA==
-  data.tar.gz: !binary |-
-    NzBmOTM3YTgwZWUxYmI3NTVkMzM5YjFiMWYyNmRhMWQ2NGE1NjhlOQ==
-!binary "U0hBNTEy":
-  metadata.gz: !binary |-
-    MzEzZjc5YWFjY2NlZjBiMjE3NmI1YjcwMzRjMDExYTJlOTMzYjNjMjg3NWEz
-    NGJmNTc4MzNhMGQ5ZWNhNWE2N2E1ZWRiZjA4MjMxMjA5NDBjYWMwN2QxYWQ4
-    ODg0NWVkYTNmZGFhMGQzMmQ2MDM5Y2ZiZTk2MjVjMDQ1MWUxN2M=
-  data.tar.gz: !binary |-
-    ODgwZjgyNmYwMWNkYjY0ZGNiYWVhYzYxYzllYzc4Yjk1YmJmODc1N2UzMjZj
-    NGJjNTUyZDgyODc4ODZiODY2NWUyY2QzMTk5ZDk0MDM2NjU1ZmJlNWZhMzU1
-    NGQ3NWViOTFiOTU3M2FmMGQ4OTViOTc3MzIxNjBiYTNiODU0YmM=
+SHA1:
+  metadata.gz: e1afa186dedd7fe80f38bb488bf077e1ff2e85b5
+  data.tar.gz: 9432299f7e026815033fc47893283e3fee927236
+SHA512:
+  metadata.gz: 8a2647253801a63bcc58e39e73c31a2684da461f2f9f15b9191a11c6887bc3b831ad1ea2f024ef301bee17caf74e6f263410d0106d2ec66cf75b218201f9a8c9
+  data.tar.gz: fd6514f0d283ab6c9addd3286fc411755a69f6395bfe80b4a4b320010587a512a28d85f485f645d315b10aa8fb336026f63eafea0d666c6ed605bbd4ca3f2b74

data/Changelog.md

@@ -0,0 +1,4 @@
+## 0.2.0
+
+  * ActiveModel support
+

data/README.md

@@ -1,8 +1,8 @@
 # TireAsyncIndex
 
-[![Build Status](https://secure.travis-ci.org/EvilFaeton/tire_async_index.png)](http://travis-ci.org/EvilFaeton/tire_async_index)
+[![Build Status](https://secure.travis-ci.org/EvilFaeton/tire_async_index.png)](http://travis-ci.org/EvilFaeton/tire_async_index) [![endorse](https://api.coderwall.com/evilfaeton/endorsecount.png)](https://coderwall.com/evilfaeton)
 
-It's extension for [Tire](https://github.com/karmi/tire/) (client for the Elasticsearch search engine), which allow to update index of ActiveRecord model using background job (based on [Sidekiq](https://github.com/mperham/sidekiq) or [Resque](https://github.com/resque/resque)).
+It's extension for [Tire](https://github.com/karmi/tire/) (client for the Elasticsearch search engine), which allow to update index of ActiveRecord/ActiveModel model using background job (based on [Sidekiq](https://github.com/mperham/sidekiq) or [Resque](https://github.com/resque/resque)).
 
 Requirements: Ruby 1.9, 2.0, Rails => 3.0
 
@@ -25,7 +25,7 @@ Or install it yourself as:
 ## Configuration
 
 You could configure TireAsyncIndex in initializer:
-    
+
     TireAsyncIndex.configure do |config|
         config.background_engine :sidekiq # or :resque
         config.use_queue         :high    # name of your queue
@@ -34,16 +34,21 @@ You could configure TireAsyncIndex in initializer:
 ## Usage
 
 Just add AsyncCallbacks to your model:
-    
+
     class User < ActiveRecord::Base
         include Tire::Model::Search
         include Tire::Model::AsyncCallbacks
-        
+
         ...
     end
-    
+
 That's all.
 
+## TODO
+
+* Add support for custom filter / custom finders
+* Test for workers
+
 ## Contributing
 
 1. Fork it

data/lib/tire/model/async_callbacks.rb

@@ -1,36 +1,64 @@
 module Tire
   module Model
     module AsyncCallbacks
+
+      ID_CONVERSION = {
+          'Moped::BSON::ObjectId' => :to_s
+      }
+
       def self.included(base)
-        if base.respond_to?(:after_commit)
-          index_update = lambda {
-            case TireAsyncIndex.engine
-            when :sidekiq
-              SidekiqUpdateIndexWorker.perform_async(base.name, id)
-            when :resque
-              Resque.enqueue(ResqueUpdateIndexJob, base.name, id)
-            else
-              tire.update_index
-            end
+        # Bind after save or create callback
+        if base.respond_to? :after_commit
+          base.send :after_commit, :async_tire_save_index
 
-            self
-          }
+        elsif base.respond_to? :after_save
+          base.send :after_save, :async_tire_save_index
+        end
 
-          base.send :after_commit, index_update, on: :create
-          base.send :after_commit, index_update, on: :update
-          base.send :after_commit, lambda { tire.update_index }, order: :first, on: :destroy
+        # Bind before destroy callback
+        if base.respond_to? :before_destroy
+          base.send :before_destroy, :async_tire_delete_index
         end
+      end
+
+      private
+
+      def async_tire_save_index
+        async_tire_callback :update
+      end
 
-        if base.respond_to?(:before_destroy) && !base.instance_methods.map(&:to_sym).include?(:destroyed?)
-          base.class_eval do
-            before_destroy  { @destroyed = true }
-            def destroyed?; !!@destroyed; end
-          end
+      def async_tire_delete_index
+        async_tire_callback :delete
+      end
+
+      def async_tire_callback(type)
+        case TireAsyncIndex.engine
+          when :sidekiq
+            TireAsyncIndex::Workers::SidekiqUpdateIndex.
+              perform_async type, self.class.name, async_tire_object_id
+
+          when :resque
+            Resque.enqueue TireAsyncIndex::Workers::ResqueUpdateIndex,
+              type, self.class.name, async_tire_object_id
+
+          else
+            case type
+              when :update
+                tire.update_index
+              when :delete
+                tire.index.remove self
+            end
         end
+      end
 
+      def async_tire_object_id
+        if (method = ID_CONVERSION[self.id.class.name])
+          self.id.send(method)
+        else
+          self.id
+        end
       end
 
     end
-
   end
 end

data/lib/tire_async_index.rb

@@ -21,12 +21,19 @@ module TireAsyncIndex
   end
 
   def reconfig_workers
-    SidekiqUpdateIndexWorker.sidekiq_options_hash["queue"] = TireAsyncIndex.queue if defined?(Sidekiq)
+    if defined?(Sidekiq)
+      Workers::SidekiqUpdateIndex.sidekiq_options_hash["queue"] = TireAsyncIndex.queue
+    end
+  end
+
+  module Workers
+    autoload :UpdateIndex, 'tire_async_index/workers/update_index'
   end
 
 end
 
-require 'tire_async_index/workers/sidekiq_update_index_worker' if defined?(Sidekiq)
-require 'tire_async_index/workers/resque_update_index_job'     if defined?(Resque)
+require 'tire_async_index/workers/sidekiq_update_index' if defined?(Sidekiq)
+require 'tire_async_index/workers/resque_update_index' if defined?(Resque)
 require 'tire/model/async_callbacks'
+
 TireAsyncIndex.configure {}

data/lib/tire_async_index/version.rb

@@ -1,3 +1,3 @@
 module TireAsyncIndex
-  VERSION = "0.1.0"
+  VERSION = "0.2.0"
 end

data/lib/tire_async_index/workers/resque_update_index.rb

@@ -0,0 +1,11 @@
+module TireAsyncIndex
+  module Workers
+    class ResqueUpdateIndex < UpdateIndex
+      def self.queue; TireAsyncIndex.queue end
+
+      def self.perform(*args)
+        self.new.perform *args
+      end
+    end
+  end
+end

data/lib/tire_async_index/workers/sidekiq_update_index.rb

@@ -0,0 +1,8 @@
+module TireAsyncIndex
+  module Workers
+    class SidekiqUpdateIndex < UpdateIndex
+      include Sidekiq::Worker
+      sidekiq_options queue: :normal
+    end
+  end
+end

data/lib/tire_async_index/workers/update_index.rb

@@ -0,0 +1,33 @@
+module TireAsyncIndex
+  module Workers
+
+    # Worker for updating ElasticSearch index
+    class UpdateIndex
+
+      # Update or delete ElasticSearch index based on the action_type parameter.
+      #
+      # Parameters:
+      #   action_type - Determine index direction. (allowed value - Update or Delete)
+      #   class_name - Model class name
+      #   id - Document id
+      #
+      def perform(action_type, class_name, id)
+        klass = Kernel.const_get(class_name)
+
+        case action_type.to_sym
+          when :update
+            object = klass.find(id)
+
+            if object.present? && object.respond_to?(:tire)
+              object.tire.update_index
+            end
+          when :delete
+
+            klass.new.tap do |inst|
+              inst.tire.index.remove(inst.tire.document_type, { _id: id })
+            end
+        end
+      end
+    end
+  end
+end

data/spec/active_model_tire_async_index_spec.rb

@@ -0,0 +1,142 @@
+require 'test_helper'
+require 'tire'
+require 'sidekiq'
+require 'resque'
+require 'tire_async_index'
+
+class AmUser
+  extend ActiveModel::Callbacks
+  define_model_callbacks :save, :destroy
+
+  include Tire::Model::AsyncCallbacks
+  attr_accessor :id
+
+  def save
+    run_callbacks :save do
+    end
+  end
+
+  def destroy
+    run_callbacks :destroy do
+    end
+  end
+end
+
+describe TireAsyncIndex do
+
+  before(:all) do
+    TireAsyncIndex.configuration = TireAsyncIndex::Configuration.new
+  end
+
+  context "configurable" do
+    it "valid default config settings" do
+      TireAsyncIndex.queue.should eql :normal
+      TireAsyncIndex.engine.should eql :none
+    end
+
+    it "set queue name" do
+      TireAsyncIndex.configure do |config|
+        config.use_queue :high
+      end
+
+      TireAsyncIndex.queue.should eql :high
+    end
+
+    it "should be able to set sidekiq as engine" do
+      TireAsyncIndex.configure do |config|
+        config.background_engine :sidekiq
+      end
+
+      TireAsyncIndex.engine.should eql :sidekiq
+    end
+
+    it "should be able to set resque as engine" do
+      TireAsyncIndex.configure do |config|
+        config.background_engine :resque
+      end
+
+      TireAsyncIndex.engine.should eql :resque
+    end
+
+    it "should not be able to set not supported engine" do
+      expect {
+        TireAsyncIndex.configure do |config|
+          config.background_engine :some_engine
+        end
+      }.to raise_error(TireAsyncIndex::EngineNotFound)
+    end
+  end
+
+  context "integration" do
+
+    describe '#after_save' do
+      it "should not start backroub on no engine" do
+        TireAsyncIndex.configure do |config|
+          config.background_engine :none
+        end
+        a    = AmUser.new.tap { |a| a.id = 23 }
+        tire = double(:Tire)
+        a.stub(:tire) { tire }
+        tire.should_receive(:update_index)
+        a.save
+      end
+
+      it "should start sidekiq" do
+        TireAsyncIndex.configure do |config|
+          config.background_engine :sidekiq
+        end
+
+        TireAsyncIndex::Workers::SidekiqUpdateIndex.should_receive(:perform_async).with(:update, "AmUser", instance_of(Fixnum))
+
+        AmUser.new.tap { |a| a.id = 23 }.save
+      end
+
+      it "should start resque" do
+        TireAsyncIndex.configure do |config|
+          config.background_engine :resque
+        end
+
+        Resque.should_receive(:enqueue).with(TireAsyncIndex::Workers::ResqueUpdateIndex, :update, "AmUser", instance_of(Fixnum))
+
+        AmUser.new.tap { |a| a.id = 23 }.save
+      end
+    end
+
+    describe '#after_destroy' do
+
+      it 'should directly invoke remove index' do
+        TireAsyncIndex.configure do |config|
+          config.background_engine :none
+        end
+
+        a    = AmUser.new.tap { |a| a.id = 23 }
+        tire = double(:Tire, index: double(:index))
+        a.stub(:tire) { tire }
+
+        tire.index.should_receive(:remove)
+        a.destroy
+      end
+
+      it 'should enqueue sidekiq job' do
+        TireAsyncIndex.configure do |config|
+          config.background_engine :sidekiq
+        end
+
+        TireAsyncIndex::Workers::SidekiqUpdateIndex.should_receive(:perform_async).with(:delete, "AmUser", instance_of(Fixnum))
+
+        AmUser.new.tap { |a| a.id = 23 }.destroy
+      end
+
+      it "should enqueue resque job" do
+        TireAsyncIndex.configure do |config|
+          config.background_engine :resque
+        end
+
+        Resque.should_receive(:enqueue).with(TireAsyncIndex::Workers::ResqueUpdateIndex, :delete, "AmUser", instance_of(Fixnum))
+
+        AmUser.new.tap { |a| a.id = 23 }.destroy
+      end
+    end
+
+  end
+end

data/spec/active_record_tire_async_index_spec.rb

@@ -0,0 +1,130 @@
+require 'test_helper'
+require 'tire'
+require 'sidekiq'
+require 'resque'
+require 'tire_async_index'
+
+class ArUser < ActiveRecord::Base
+  include Tire::Model::AsyncCallbacks
+end
+
+describe TireAsyncIndex do
+
+  before(:all) do
+    TireAsyncIndex.configuration = TireAsyncIndex::Configuration.new
+  end
+
+  context "configurable" do
+    it "valid default config settings" do
+      TireAsyncIndex.queue.should eql :normal
+      TireAsyncIndex.engine.should eql :none
+    end
+
+    it "set queue name" do
+      TireAsyncIndex.configure do |config|
+        config.use_queue :high
+      end
+
+      TireAsyncIndex.queue.should eql :high
+    end
+
+    it "should be able to set sidekiq as engine" do
+      TireAsyncIndex.configure do |config|
+        config.background_engine :sidekiq
+      end
+
+      TireAsyncIndex.engine.should eql :sidekiq
+    end
+
+    it "should be able to set resque as engine" do
+      TireAsyncIndex.configure do |config|
+        config.background_engine :resque
+      end
+
+      TireAsyncIndex.engine.should eql :resque
+    end
+
+    it "should not be able to set not supported engine" do
+      expect {
+        TireAsyncIndex.configure do |config|
+          config.background_engine :some_engine
+        end
+      }.to raise_error(TireAsyncIndex::EngineNotFound)
+    end
+  end
+
+  context "integration" do
+
+    describe '#after_save' do
+      it "should not start backroub on no engine" do
+        TireAsyncIndex.configure do |config|
+          config.background_engine :none
+        end
+
+        a = ArUser.new
+        a.stub(:tire) { a }
+        a.should_receive(:update_index)
+        a.save
+      end
+
+      it "should start sidekiq" do
+        TireAsyncIndex.configure do |config|
+          config.background_engine :sidekiq
+        end
+
+        TireAsyncIndex::Workers::SidekiqUpdateIndex.should_receive(:perform_async).with(:update, "ArUser", instance_of(Fixnum))
+
+        a = ArUser.new
+        a.save
+      end
+
+      it "should start resque" do
+        TireAsyncIndex.configure do |config|
+          config.background_engine :resque
+        end
+
+        Resque.should_receive(:enqueue).with(TireAsyncIndex::Workers::ResqueUpdateIndex, :update, "ArUser", instance_of(Fixnum))
+
+        a = ArUser.new
+        a.save
+      end
+    end
+
+    describe '#after_destroy' do
+
+      it 'should directly invoke remove index' do
+        TireAsyncIndex.configure do |config|
+          config.background_engine :none
+        end
+
+        a    = ArUser.new.tap { |a| a.id = 23 }
+        tire = double(:Tire, index: double(:index))
+        a.stub(:tire) { tire }
+
+        tire.index.should_receive(:remove)
+        a.destroy
+      end
+
+      it 'should enqueue sidekiq job' do
+        TireAsyncIndex.configure do |config|
+          config.background_engine :sidekiq
+        end
+
+        TireAsyncIndex::Workers::SidekiqUpdateIndex.should_receive(:perform_async).with(:delete, "ArUser", instance_of(Fixnum))
+
+        ArUser.new.tap { |a| a.id = 23 }.destroy
+      end
+
+      it "should enqueue resque job" do
+        TireAsyncIndex.configure do |config|
+          config.background_engine :resque
+        end
+
+        Resque.should_receive(:enqueue).with(TireAsyncIndex::Workers::ResqueUpdateIndex, :delete, "ArUser", instance_of(Fixnum))
+
+        ArUser.new.tap { |a| a.id = 23 }.destroy
+      end
+    end
+
+  end
+end

metadata

@@ -1,27 +1,27 @@
 --- !ruby/object:Gem::Specification
 name: tire_async_index
 version: !ruby/object:Gem::Version
-  version: 0.1.0
+  version: 0.2.0
 platform: ruby
 authors:
 - Sergey Efremov
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2013-04-23 00:00:00.000000000 Z
+date: 2013-06-26 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: tire
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ! '>='
+    - - '>='
       - !ruby/object:Gem::Version
         version: '0'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ! '>='
+    - - '>='
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
@@ -42,14 +42,14 @@ dependencies:
   name: rake
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ! '>='
+    - - '>='
       - !ruby/object:Gem::Version
         version: '0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ! '>='
+    - - '>='
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
@@ -70,56 +70,56 @@ dependencies:
   name: activerecord-nulldb-adapter
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ! '>='
+    - - '>='
       - !ruby/object:Gem::Version
         version: '0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ! '>='
+    - - '>='
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
   name: sidekiq
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ! '>='
+    - - '>='
       - !ruby/object:Gem::Version
         version: '0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ! '>='
+    - - '>='
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
   name: resque
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ! '>='
+    - - '>='
       - !ruby/object:Gem::Version
         version: '0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ! '>='
+    - - '>='
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
   name: rspec
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ! '>='
+    - - '>='
       - !ruby/object:Gem::Version
         version: '0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ! '>='
+    - - '>='
       - !ruby/object:Gem::Version
         version: '0'
 description: Update tire (elasticsearch) index async with Sidekiq or Resque
@@ -132,6 +132,7 @@ files:
 - .gitignore
 - .rspec
 - .travis.yml
+- Changelog.md
 - Gemfile
 - LICENSE.txt
 - README.md
@@ -141,11 +142,13 @@ files:
 - lib/tire_async_index/configuration.rb
 - lib/tire_async_index/exceptions.rb
 - lib/tire_async_index/version.rb
-- lib/tire_async_index/workers/resque_update_index_job.rb
-- lib/tire_async_index/workers/sidekiq_update_index_worker.rb
+- lib/tire_async_index/workers/resque_update_index.rb
+- lib/tire_async_index/workers/sidekiq_update_index.rb
+- lib/tire_async_index/workers/update_index.rb
+- spec/active_model_tire_async_index_spec.rb
+- spec/active_record_tire_async_index_spec.rb
 - spec/test_helper.rb
 - spec/test_schema.rb
-- spec/tire_async_index_spec.rb
 - tire_async_index.gemspec
 homepage: http://github.com/EvilFaeton/tire_async_index
 licenses:
@@ -157,12 +160,12 @@ require_paths:
 - lib
 required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
-  - - ! '>='
+  - - '>='
     - !ruby/object:Gem::Version
       version: '0'
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
-  - - ! '>='
+  - - '>='
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
@@ -172,6 +175,7 @@ signing_key:
 specification_version: 4
 summary: Update tire (elasticsearch) index async with Sidekiq or Resque
 test_files:
+- spec/active_model_tire_async_index_spec.rb
+- spec/active_record_tire_async_index_spec.rb
 - spec/test_helper.rb
 - spec/test_schema.rb
-- spec/tire_async_index_spec.rb

data/lib/tire_async_index/workers/resque_update_index_job.rb

@@ -1,11 +0,0 @@
-class ResqueUpdateIndexJob
-
-  def self.queue
-    TireAsyncIndex.queue
-  end
-
-  def self.perform class_name, id
-    class_name.constantize.find_by_id(id).try(:update_index)
-  end
-
-end

data/lib/tire_async_index/workers/sidekiq_update_index_worker.rb

@@ -1,10 +0,0 @@
-class SidekiqUpdateIndexWorker
-
-  include Sidekiq::Worker
-  sidekiq_options queue: :normal
-
-  def perform class_name, id
-    class_name.constantize.find_by_id(id).try(:update_index)
-  end
-
-end

data/spec/tire_async_index_spec.rb

@@ -1,86 +0,0 @@
-require 'test_helper'
-require 'tire'
-require 'sidekiq'
-require 'resque'
-require 'tire_async_index'
-
-class User < ActiveRecord::Base
-  include Tire::Model::AsyncCallbacks
-end
-
-describe TireAsyncIndex do
-  context "configurable" do
-    it "valid default config settings" do
-      TireAsyncIndex.queue.should eql :normal
-      TireAsyncIndex.engine.should eql :none
-    end
-
-    it "set queue name" do
-      TireAsyncIndex.configure do |config|
-        config.use_queue :high
-      end
-
-      TireAsyncIndex.queue.should eql :high
-    end    
-
-    it "should be able to set sidekiq as engine" do
-      TireAsyncIndex.configure do |config|
-        config.background_engine :sidekiq
-      end
-
-      TireAsyncIndex.engine.should eql :sidekiq
-    end
-
-    it "should be able to set resque as engine" do
-      TireAsyncIndex.configure do |config|
-        config.background_engine :resque
-      end
-
-      TireAsyncIndex.engine.should eql :resque
-    end
-
-    it "should not be able to set not supported engine" do
-      expect {
-        TireAsyncIndex.configure do |config|
-          config.background_engine :some_engine
-        end
-      }.to raise_error(TireAsyncIndex::EngineNotFound)
-    end
-  end
-
-  context "integration" do
-
-    it "should not start backroub on no engine" do
-      TireAsyncIndex.configure do |config|
-        config.background_engine :none
-      end
-      a = User.new
-      a.stub(:tire) { a }
-      a.should_receive(:update_index)
-      a.save
-    end
-
-    it "should start sidekiq" do
-      TireAsyncIndex.configure do |config|
-        config.background_engine :sidekiq
-      end
-
-      SidekiqUpdateIndexWorker.should_receive(:perform_async).with("User", instance_of(Fixnum))
-
-      a = User.new
-      a.save
-    end
-
-    it "should start resque" do
-      TireAsyncIndex.configure do |config|
-        config.background_engine :resque
-      end
-
-      Resque.should_receive(:enqueue).with(ResqueUpdateIndexJob, "User", instance_of(Fixnum))
-
-      a = User.new
-      a.save
-    end
-
-  end
-end