dm-parse 0.3.7 → 0.3.8

data/VERSION

@@ -1 +1 @@
-0.3.7
+0.3.8

data/dm-parse.gemspec

@@ -5,11 +5,11 @@
 
 Gem::Specification.new do |s|
   s.name = "dm-parse"
-  s.version = "0.3.7"
+  s.version = "0.3.8"
 
   s.required_rubygems_version = Gem::Requirement.new(">= 0") if s.respond_to? :required_rubygems_version=
   s.authors = ["Zhi-Qiang Lei"]
-  s.date = "2012-07-04"
+  s.date = "2012-07-05"
   s.description = "An extension to make DataMapper working on Parse.com"
   s.email = "zhiqiang.lei@gmail.com"
   s.extra_rdoc_files = [

data/lib/adapters/parse_adapter.rb

@@ -36,6 +36,10 @@ module DataMapper
         response      = engine.read storage_name, params
 
         response["results"]
+      rescue NotImplementedError
+        log :error, "Unsupported Query:"
+        log :error, "  Model: #{model}"
+        log :error, "  Conditions: #{query.conditions}"
       end
 
       # Read the "count" from Parse
@@ -165,7 +169,13 @@ module DataMapper
 
         result = {}
         translate(conditions, result)
-        finalize result
+        result_for_parse = finalize result
+
+        log :info, "Translating query on #{query.model}:"
+        log :info, "  from: #{query.conditions}"
+        log :info, "  to: #{result_for_parse.to_json}"
+
+        result_for_parse
       end
 
       def translate(condition, result)
@@ -272,6 +282,10 @@ module DataMapper
         end
       end
 
+      def log(level, message)
+        DataMapper.logger.send level, "[dm-parse][#{level}] #{message}"
+      end
+
     end
 
     const_added(:ParseAdapter)

data/spec/parse_adapter_spec.rb

@@ -20,7 +20,13 @@ describe DataMapper::Adapters::ParseAdapter do
 
     context "when query assigns some exact values" do
       let(:query) { model.all(:id => "z", :title => "x", :body => "y").query }
+
       it { should eq("objectId" => "z", "title" => "x", "body" => "y") }
+
+      it "should log translation info" do
+        DataMapper.logger.should_receive(:info).with(an_instance_of(String)).exactly(3).times
+        adapter.send :parse_conditions_for, query
+      end
     end
 
     [:gt, :gte, :lt, :lte].each do |slug|
@@ -167,6 +173,7 @@ describe DataMapper::Adapters::ParseAdapter do
   describe "#parse_params_for" do
     subject { adapter.send :parse_params_for, query }
     let(:query) { model.all.query }
+
     it { should eq(:limit => 1000) }
 
     context "when limit is given" do
@@ -190,4 +197,13 @@ describe DataMapper::Adapters::ParseAdapter do
     end
   end # #parse_params_for
 
+  describe "#read" do
+    let(:query) { model.all.query }
+
+    it "should log error" do
+      DataMapper.logger.should_receive(:error).with(an_instance_of(String)).exactly(3).times
+      adapter.stub(:parse_params_for) { raise NotImplementedError  }
+      adapter.read query
+    end
+  end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: dm-parse
 version: !ruby/object:Gem::Version
-  version: 0.3.7
+  version: 0.3.8
   prerelease: 
 platform: ruby
 authors:
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2012-07-04 00:00:00.000000000 Z
+date: 2012-07-05 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: dm-core
@@ -242,7 +242,7 @@ required_ruby_version: !ruby/object:Gem::Requirement
       version: '0'
       segments:
       - 0
-      hash: -1860376599775651579
+      hash: 3602608103859692141
 required_rubygems_version: !ruby/object:Gem::Requirement
   none: false
   requirements: