bosh-verifyconnections 1.0.1 → 1.0.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 820df08ec6e4f8bb911adf3b77010afeaa429a7f
-  data.tar.gz: fa25cf1b90e232c949752801ff3c48f61763acc3
+  metadata.gz: 8f25f5d50e5ee314d28693c880349f5c4deaaef8
+  data.tar.gz: 71bef84a3153e2e46d7e769abddb6bbf0028b6ff
 SHA512:
-  metadata.gz: 7a3bdfe381d651ef8df6a21234cde13ea895dcc7c1b425db01e82d6915d0a559a73dbd583d8106ecf45624ee427e348ec7fcfb2575f97105ce2195a62a5106f7
-  data.tar.gz: c7e8be56bdf4e734b52651d41a388953d692b0925d9f1b32a669db97e4e33a7fe031049df27b97e30387acdcc4eeb68812b8af56118736971ec26089ce42c120
+  metadata.gz: 68cf9781479536c921fd33358508b445bc10302cbb5c658e82a6298f31a60d3b0ee895def00ef0370c62797c1a18942f9eee8d65896ee9b84a1bb142cad77858
+  data.tar.gz: 662baeee9ff26390bcc9796815a657c13dfecdbae52d328cf4512ed0ed930da055545ada4b5f242bd751519bdb5664771d840c05d0bdeb6fd4b565263f236408

data/ChangeLog.md

@@ -3,3 +3,4 @@
 ## v1.0
 
 * Released May 3rd 2014
+* Now tests properties that contain arrays of static_ips or hostnames [v1.0.2]

data/Guardfile

@@ -4,5 +4,6 @@
 guard :rspec do
   watch(%r{^spec/.+_spec\.rb$})
   watch(%r{^lib/bosh/verifyconnections/(.+)\.rb$})     { |m| "spec/#{m[1]}_spec.rb" }
+  watch(%r{^lib/core-ext/(.+)\.rb$})     { |m| "spec/core-ext/#{m[1]}_spec.rb" }
   watch('spec/spec_helper.rb')  { "spec" }
 end

data/lib/bosh/cli/commands/verifyconnections.rb

@@ -1,4 +1,4 @@
-require "core-ext/hash_to_dotted_notation"
+require "core-ext/hash/to_dotted_hash"
 
 module Bosh::Cli::Command
   # Performs job interconnection verifications upon the target deployment

data/lib/bosh/verifyconnections/models/deployment.rb

@@ -1,6 +1,6 @@
 require "common/deep_copy"
-require "core-ext/hash_deep_merge"
-require "core-ext/hash_to_dotted_notation"
+require "core-ext/hash/deep_merge"
+require "core-ext/hash/to_dotted_hash"
 
 module Bosh::VerifyConnections
   class Deployment

data/lib/bosh/verifyconnections/version.rb

@@ -1,5 +1,5 @@
 module Bosh
   module VerifyConnections
-    VERSION = "1.0.1"
+    VERSION = "1.0.2"
   end
 end

data/lib/core-ext/{hash_to_dotted_notation.rb → hash/to_dotted_hash.rb}

@@ -5,6 +5,16 @@ class Hash
       key = recursive_key + k.to_s
       if v.is_a? Hash
         ret.merge!(v.to_dotted_hash(key + "."))
+      elsif v.is_a?(Array)
+        if v.first.is_a?(Hash)
+          v.each_with_index do |item, index|
+            ret.merge!(item.to_dotted_hash(key + "[#{index}]."))
+          end
+        else
+          v.each_with_index do |item, index|
+            ret[key + "[#{index}]"] = item
+          end
+        end
       else
         ret[key] = v
       end

data/spec/core-ext/hash/to_dotted_hash_spec.rb

@@ -0,0 +1,12 @@
+describe "Hash#to_dotted_hash" do
+  it { expect({}.to_dotted_hash).to eq({}) }
+  it { expect({"foo" => "bar"}.to_dotted_hash).to eq({"foo" => "bar"}) }
+  it { expect({foo: "bar"}.to_dotted_hash).to eq({"foo" => "bar"}) }
+  it { expect({foo: {bar: "baz"}}.to_dotted_hash).to eq({"foo.bar" => "baz"}) }
+
+  context "arrays" do
+    it { expect({foo: {bar: []}}.to_dotted_hash).to eq({}) }
+    it { expect({foo: {bar: ["a", "b"]}}.to_dotted_hash).to eq({"foo.bar[0]" => "a", "foo.bar[1]" => "b"}) }
+    it { expect({foo: [{bar: "baz"}, {alpha: 123}]}.to_dotted_hash).to eq({"foo[0].bar" => "baz", "foo[1].alpha" => 123}) }
+  end
+end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: bosh-verifyconnections
 version: !ruby/object:Gem::Version
-  version: 1.0.1
+  version: 1.0.2
 platform: ruby
 authors:
 - Dr Nic Williams
@@ -105,8 +105,9 @@ files:
 - lib/bosh/verifyconnections/models/deployment_job.rb
 - lib/bosh/verifyconnections/models/director.rb
 - lib/bosh/verifyconnections/version.rb
-- lib/core-ext/hash_deep_merge.rb
-- lib/core-ext/hash_to_dotted_notation.rb
+- lib/core-ext/hash/deep_merge.rb
+- lib/core-ext/hash/to_dotted_hash.rb
+- spec/core-ext/hash/to_dotted_hash_spec.rb
 - spec/fixtures/combination_of_correct_dns_and_static_ip_connections.yml
 - spec/fixtures/job_with_static_ips_but_not_referenced.yml
 - spec/fixtures/property_is_bosh_dns_but_not_for_bosh_job.yml
@@ -140,6 +141,7 @@ specification_version: 4
 summary: Performs job interconnection verifications upon the target BOSH deployment
   manifest file.
 test_files:
+- spec/core-ext/hash/to_dotted_hash_spec.rb
 - spec/fixtures/combination_of_correct_dns_and_static_ip_connections.yml
 - spec/fixtures/job_with_static_ips_but_not_referenced.yml
 - spec/fixtures/property_is_bosh_dns_but_not_for_bosh_job.yml