looksist 0.3.11 → 0.3.12

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 69c773e23a25636e39c6a9fa5d509ede5915cc2e
-  data.tar.gz: d2dcfa3496a4dc19519bf7472ed0250948fe8870
+  metadata.gz: 9f63a1c36d1bac556df87d59b84e56d3cb912864
+  data.tar.gz: bda04de8e57710e3aeb2c7f48b8b75b53cf947a4
 SHA512:
-  metadata.gz: df141569de0cf50b2597916fa0d3a4f210c7e9480537c234355be7619fc0e00bbf5f09c35cde13362a77d0b5a872b44010aef21188c78001997f87672494ac4e
-  data.tar.gz: ef61fb2dc42865334b6ef5fe06efa762f4d40ee9f997f0b6573477624c785d2227f1a2a85b24c6e4e37a6af4be24bfd7126e197f8dea99decc624308bcd7ca7a
+  metadata.gz: 86f43387109482d62aec666accbb5b68bb10c25e7ba30d6eead747aa9970f4bf5f0d2e24f64126af2dec368834db791b7f590983c8b4ec88892526d1828f4018
+  data.tar.gz: 6a1c54940cf6592d5eac3358f755838a3b64222bb46a4a2bedb3151cd4c831926895916d94ef5f8800cb220ece2bd77ab3391d8cd1b373ffadf72b849cb66d57

data/lib/looksist/core.rb

@@ -19,7 +19,12 @@ module Looksist
           alias_what = find_alias(as, what)
           @lookup_attributes[alias_what] = opts[:using]
           define_method(alias_what) do
-            Looksist.redis_service.send("#{__entity__(bucket_name)}_for", self.send(using).try(:to_s))
+            result = Looksist.redis_service.send("#{__entity__(bucket_name)}_for", self.send(using).try(:to_s))
+            begin
+              JSON.parse(result)[what.to_s]
+            rescue
+              result
+            end
           end
         end
       end

data/lib/looksist/hashed.rb

@@ -161,7 +161,12 @@ module Looksist
           end
         else
           alias_method = find_alias(as, populate)
-          elt[alias_method] = values[elt.with_indifferent_access[using]]
+          begin
+            json = JSON.parse(values[elt.with_indifferent_access[using]])
+            elt[alias_method] = json.with_indifferent_access[populate]
+          rescue
+            elt[alias_method] = values[elt.with_indifferent_access[using]]
+          end
         end
       end
 

data/lib/looksist/version.rb

@@ -1,3 +1,3 @@
 module Lookist
-  VERSION = '0.3.11'
+  VERSION = '0.3.12'
 end

data/spec/looksist/hashed_spec.rb

@@ -507,6 +507,33 @@ describe Looksist::Hashed do
                                                                )
     end
 
+    it 'should work for multiple attributes but looking up only one attribute' do
+      class HashWithMultipleAttributesLookUpOne
+        include Looksist
+
+        def metrics
+          [
+              {
+                  hero_id: 1
+              },
+              {
+                  hero_id: 2
+              }
+          ]
+        end
+
+        inject after: :metrics, using: :hero_id, populate: :name, as: {name: 'hero_name'}
+      end
+      js1 = {name: 'Rajini', mnemonic: 'SuperStart'}.to_json
+      jsons = [js1, nil]
+      expect(@mock).to receive(:mget).once.with(*%w(heros/1 heros/2)).and_return(jsons)
+
+      expect(HashWithMultipleAttributesLookUpOne.new.metrics).to eq(
+                                                                     [{:hero_id => 1, :hero_name => 'Rajini'},
+                                                                      {:hero_id => 2, :hero_name => nil}]
+                                                                 )
+    end
+
     it 'should work for class methods' do
       class SelfHelp
         include Looksist

data/spec/looksist/looksist_spec.rb

@@ -52,6 +52,24 @@ describe Looksist do
         expect(e.nome).to eq('Rajini')
         expect(e.age).to eq(16)
         expect(e.to_json).to eq("{\"id\":1,\"nome\":\"Rajini\",\"age\":16}")
+        end
+
+      it 'should fetch attributes appropriately when plucking single attribute from a json key' do
+        module NonAliasSpecificLookup
+          class NoCacheEmployee
+            include Her::Model
+            use_api TEST_API
+            include Looksist
+            lookup :name, using: :id, as: {name: 'nome'}
+            def as_json(opts)
+              super(opts).merge(attributes)
+            end
+          end
+        end
+        expect(@mock).to receive(:get).twice.times.with('ids/1').and_return({name: 'Rajini', age: 16}.to_json)
+        e = NonAliasSpecificLookup::NoCacheEmployee.new(id:1)
+        expect(e.nome).to eq('Rajini')
+        expect(e.to_json).to eq("{\"id\":1,\"nome\":\"Rajini\"}")
       end
     end
 

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: looksist
 version: !ruby/object:Gem::Version
-  version: 0.3.11
+  version: 0.3.12
 platform: ruby
 authors:
 - RC
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2014-12-03 00:00:00.000000000 Z
+date: 2015-08-18 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler