reuters 0.9.0 → 0.9.2

checksums.yaml

@@ -1,15 +1,15 @@
 ---
 !binary "U0hBMQ==":
   metadata.gz: !binary |-
-    YTY0MTNhN2U5NjI2OTFlY2Y5YjgwN2ViMGRkMzU3MDIyYTk4ZmNlMw==
+    ODdhMWQ4MGJjOWY1OTQ3ZjMwOTFjYWMxZDg3ZGQzZmY0OGQ2ZWY1MA==
   data.tar.gz: !binary |-
-    ZmQzODllYzhjZDlkYTczODdiYmY2YTgyNDMzOGQ3Y2ZhMTA1MTQyMQ==
+    MmY2ZTU4NTZiNmI1ODZlMDk5NmI4YmUwZmFlYzkyN2JjZmIzMzEzYw==
 SHA512:
   metadata.gz: !binary |-
-    NjA5NGQ0NDViMjIzNDAyMTNiOTkwNWQ0MTg0OGU2OTE3NDZkOTgzMmU1M2Rl
-    MGJmZjE2NTkxOWEwM2ZjOTA1MTBmZTA0NmIwZjNkZmE3MTIzODUyZmI5ZDQ5
-    YzM3NmM4NzUxNjY4OWFlMTk1ZGU4N2UwYjY3YjFiODgzMGU0MDI=
+    ZjA2YmM0Yjc1NWFkMmIzZDFkYTU1MTljOGFmMjVkNGYzNzUyY2JjMjBhM2M3
+    MThiNjIxNjk2ODcyNzAzNGFmYmQyNjIwYmEzMTg5OGZhZDM2OTZlYzUxNDI3
+    MTVkMTc2MmQ4YTIzYmJkNzBmNDkyODIzZDU2Y2Y1NWYzOTA0NWE=
   data.tar.gz: !binary |-
-    MjZiZThhNWZlNGJkNmVkODIwZjM1YmFjN2QyZWEwNDUwOTgyZmJmMzFlOTEy
-    YjhlZWJhZjNhYzhkNmQ1MWRhYWNhZjM1MTAxNjFkODkwZmQyMGQyNzEwZDA2
-    NDE0MzM4ZjFiM2EyZThhYzM5ZWY1NjJhY2JkNzQxNmQ2MzhkMDI=
+    OTdlM2U4MDA5OTg2NjY4NmQyMWJhOTgzNmNiN2NmNjY4MTdhNTllMTIzNDg4
+    MjNkNjFkYmYxZjk5OTRjNGU2N2JhNGM4MTNlNDQwNGNlMWE2ZDIwZTVjMzgx
+    YjgyNWVkYmQ2NmMwMWU4ZjA2YWRlMGQ3OGU5M2ZlOWUyZDBhMTI=

data/lib/reuters/builder.rb

@@ -77,6 +77,24 @@ module Reuters
       super || super(camelize(key))
     end
 
+    # Attempts to find a key that exists within the attributes hash.
+    # If the key cannot be found in its current format,
+    # the method attempts to camelcase the key and
+    # search again.
+    #
+    # @return [Boolean] True if the key exists, false otherwise
+    def attribute_key?(key, attr_key)
+      attrs = attribute_keys(key)
+      attrs.include?(camelize(attr_key) || attrs.include?(attr_key)) 
+    end
+
+    # Returns all keys inside the attributes hash.
+    #
+    # @return [Array] All attribute keys.
+    def attribute_keys(key)
+      (self[:attributes!][camelize(key)] || {}).keys
+    end
+
     private
 
     def assign_val(key, body)
@@ -114,4 +132,4 @@ module Reuters
     end
 
   end
-end
+end

data/lib/reuters/client/search/equity.rb

@@ -26,9 +26,15 @@ module Reuters
                 sec = req.send(section)
 
                 sec.keys.each do |key|
-                  sec.attributes({ key => { 'xmlns' => query_spec } }, false)
+                  # injext xmlns property only if it does not already exist
+                  unless sec.attribute_key? key, 'xmlns'
+                    sec.attributes({ key => { 'xmlns' => query_spec } }, false)
+                  end
                   sec[key].keys.each do |type|
-                    sec[key].attributes({ type => { 'xmlns' => data_type } }, false)
+                    # injext xmlns property only if it does not already exist
+                    unless sec[key].attribute_key? type, 'xmlns'
+                      sec[key].attributes({ type => { 'xmlns' => data_type } }, false)
+                    end
                   end
                 end
 

data/lib/reuters/version.rb

@@ -2,5 +2,5 @@
 # Knowledge Direct API.
 module Reuters
   # Current version of the Reuters gem.
-  VERSION = '0.9.0'
+  VERSION = '0.9.2'
 end

data/spec/reuters/builder_spec.rb

@@ -26,7 +26,45 @@ describe Reuters::Builder do
     expect(b).to include("TestMethod")
   end
 
+  describe "return value of #attribute_key?" do
+
+    before do
+      @builder.hold_key
+      @builder.attributes(hold_key: { "test_build" => "Type" })
+    end
+
+    it "should allow camelcase keys" do
+      expect(@builder.attribute_key?(:hold_key, "TestBuild")).to be_true
+    end
+
+    it "should allow underscore keys" do
+      expect(@builder.attribute_key?(:hold_key, :test_build)).to be_true
+    end
+
+  end
+
+  describe "return value of #attribute_keys" do
+
+    before do
+      @builder.hold_key
+      @builder.attributes(hold_key: { "test_build" => "Type" })
+      @builder.attributes({ hold_key: { "xml_ns" => "Type" } }, false)
+    end
+
+    subject { @builder.attribute_keys(:hold_key) }
+
+    it { should include "TestBuild", "xml_ns" }
+
+    it { should_not include :attributes! }
+
+    it { should_not be_empty }
+
+    specify { expect { @builder.attributes(hold_key: { "t" => "a" }) }.to change { @builder.attribute_keys(:hold_key) } }
+
+  end
+
   describe "return value of #keys" do
+
     before do
       @builder.foo_test
       @builder.bar_test
@@ -186,4 +224,4 @@ describe Reuters::Builder do
 
   end
 
-end
+end

data/spec/reuters/client/search/all_spec.rb

@@ -30,7 +30,6 @@ describe Reuters::Client::Search::All do
   describe "return value of #query_spec" do
 
     it "should be a string" do
-      puts subject.operations.inspect
       expect(subject.query_spec).to be_a(String)
     end
 

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: reuters
 version: !ruby/object:Gem::Version
-  version: 0.9.0
+  version: 0.9.2
 platform: ruby
 authors:
 - Stockflare