fetcher-microdata 0.0.11 → 0.0.12

data/lib/fetcher-microdata/version.rb

@@ -1,5 +1,5 @@
 module Fetcher
   class Microdata
-    VERSION = "0.0.11"
+    VERSION = "0.0.12"
   end
 end

data/lib/writer/fetcher-microdata/person_user.rb

@@ -10,7 +10,8 @@ module Writer
 
         def hash
           @attributes = @source.attributes
-          {
+
+          resp = {
             "type" => [
               @source._type
             ],
@@ -35,6 +36,13 @@ module Writer
               ]
             }
           }
+
+          resp["properties"].keys.each do |att|
+            if resp["properties"]["#{att}"] == [nil]
+              resp["properties"].delete "#{att}"
+            end
+          end
+          resp
         end
       end
     end

data/spec/writer/fetcher/microdata/person_user_spec.rb

@@ -89,5 +89,65 @@ describe Writer::Fetcher::Microdata::PersonUser do
         writer.hash["properties"].should include @properties_hash        
       end
     end
+  end
+
+  describe "#hash innecesary data" do
+
+    it 'should not put dateRegistered on the data if data registered is nil' do
+      @properties_hash = {
+        "User#dateRegistered" => [nil]
+      }
+      aux_stub = stub "aux stub"
+      aux_stub.stub(:[]) do |symbol|
+        if symbol == :dateRegistered
+          nil
+        else
+          symbol.to_s
+        end
+      end
+      @nil_user_stub.should_receive(:_type).and_return "some type"
+      @nil_user_stub.should_receive(:attributes).and_return aux_stub
+
+      writer = Writer::Fetcher::Microdata::PersonUser.new @nil_user_stub
+      writer.hash["properties"].should_not include @properties_hash
+    end
+
+    it 'should not put description on the data if data registered is nil' do
+      @properties_hash = {
+        "description" => [nil]
+      }
+      aux_stub = stub "aux stub"
+      aux_stub.stub(:[]) do |symbol|
+        if symbol == :description
+          nil
+        else
+          symbol.to_s
+        end
+      end
+      @nil_user_stub.should_receive(:_type).and_return "some type"
+      @nil_user_stub.should_receive(:attributes).and_return aux_stub
+
+      writer = Writer::Fetcher::Microdata::PersonUser.new @nil_user_stub
+      writer.hash["properties"].should_not include @properties_hash
+    end
+
+    it 'should not put url on the data if data registered is nil' do
+      @properties_hash = {
+        "url" => [nil]
+      }
+      aux_stub = stub "aux stub"
+      aux_stub.stub(:[]) do |symbol|
+        if symbol == :url
+          nil
+        else
+          symbol.to_s
+        end
+      end
+      @nil_user_stub.should_receive(:_type).and_return "some type"
+      @nil_user_stub.should_receive(:attributes).and_return aux_stub
+
+      writer = Writer::Fetcher::Microdata::PersonUser.new @nil_user_stub
+      writer.hash["properties"].should_not include @properties_hash
+    end
   end  
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: fetcher-microdata
 version: !ruby/object:Gem::Version
-  version: 0.0.11
+  version: 0.0.12
   prerelease: 
 platform: ruby
 authors: