myjohndeere 0.1.7 → 0.1.8

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: b526cc75a6720846322dc5e9c59edb027d43098d
-  data.tar.gz: f9441bbe922c914a55e7a425fd7595cfe42a9cb3
+  metadata.gz: f6f1396f290f034c7d9bc96100a798d5e3a653e7
+  data.tar.gz: f0815526e2742dae974c1ed89745265e60a5a433
 SHA512:
-  metadata.gz: 8d57d5d6887707b4d6a2abb31f8213b6cad412f7b2512b54a2b2c5a9f53d1b1805f172fe325a5f692d0f3b31f3cb177115c675102326f012fc367c82a36c3088
-  data.tar.gz: dd0775f0d88d5ae46970cd5c3849758d9a52c0685a4b4fb7b88bcbac01dc054135bfe93fb1650acff6ee008dc65adcfd97e16f61046971bfe165bb45e8f3ce8a
+  metadata.gz: 6e09bf7d4e04d3a9d2b2093fdff043d5653c86d246fe5db098b6fc117db90f111cbf240b99a56f63ef41b81decae090338689843ec953a08ac66b34e283c4d05
+  data.tar.gz: ae920d40bdcb973f0fa8ecfe7a5bf3c8185f212c80578a0ccb975f6e1e6925183a63501d4c16a01a71e512766fdd2a28223c744fdead6df105cfbf80dd5fba37

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    myjohndeere (0.1.7)
+    myjohndeere (0.1.8)
       oauth (>= 0.5.3)
 
 GEM

data/lib/myjohndeere/field.rb

@@ -36,9 +36,10 @@ module MyJohnDeere
 
     private
       def find_first_active_boundary(possible_boundaries)
-        active_boundaries = possible_boundaries.select { |b| b.active && !b.deleted }
+        active_boundaries = possible_boundaries.select { |b| b.active && !b.deleted }.
+          uniq { |b| b.id }
         if active_boundaries.count > 1 then
-          raise MyJohnDeereError.new("There was more than one boundary in the field, this is currently unexpected")
+          raise MyJohnDeereError.new("There was more than one boundary in the field ID: #{self.id}, this is currently unexpected")
         elsif active_boundaries.count == 1 then
           return active_boundaries.first
         else

data/lib/myjohndeere/file_resource.rb

@@ -46,7 +46,7 @@ module MyJohnDeere
             "#{self.base_jd_resource}/#{file_resource_id}",
             body: body,
             headers: { 
-              'accept'=> 'application/vnd.deere.axiom.v3+json',
+              'accept'=> JSON_CONTENT_HEADER_VALUE,
               "Content-Type"=>'application/octet-stream' ,
               "Content-Length" => body.bytesize.to_s
             })

data/lib/myjohndeere/version.rb

@@ -1,3 +1,3 @@
 module MyJohnDeere
-  VERSION = '0.1.7'
+  VERSION = '0.1.8'
 end

data/spec/fixtures.json

@@ -658,7 +658,7 @@
               "lon":-105.023052692
             }
           },
-          "id":"f4828b25-2a92-12a4-bf3e-a107c39ecb85",
+          "id":"f4828b25-2a92-12a4-bf3e-a107c39ecb84",
           "links":[  
             {  
               "@type":"Link",

data/test/test_field.rb

@@ -63,11 +63,12 @@ class TestField < Minitest::Test
       with(query: {embed: "boundaries"}).
       to_return(status: 200, body: fixture.to_json)
 
-    assert_raises MyJohnDeere::MyJohnDeereError, "At the moment I don't have a use case where this would happen" do
-      field = MyJohnDeere::Field.retrieve(default_access_token, 
-        fixture["id"], organization_id: ORGANIZATION_FIXTURE["id"],
-        body: {embed: "boundaries"})
-    end
+    field = MyJohnDeere::Field.retrieve(default_access_token, 
+      fixture["id"], organization_id: ORGANIZATION_FIXTURE["id"],
+      body: {embed: "boundaries"})
+
+    assert field.boundary.is_a?(MyJohnDeere::Boundary), 
+      "There should be only one boundary since the request returned 2 boundaries with the same id"
   end
 
   def test_list()
@@ -83,6 +84,6 @@ class TestField < Minitest::Test
 
     fields.next_page!
 
-    assert_equal 1, fields.data
+    assert_equal 1, fields.data.count
   end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: myjohndeere
 version: !ruby/object:Gem::Version
-  version: 0.1.7
+  version: 0.1.8
 platform: ruby
 authors:
 - Paul Susmarski