kitchen-terraform 4.5.0 → 4.6.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 151779b97d4bc65896d6ed19b7f21fd62c9d28c3de359070acebe8c6b9fb1c93
-  data.tar.gz: 4459af4b43c7b85098cd87ec86739004e74ef42a40126c58c1b6d4eb8df5d5ea
+  metadata.gz: 2e916889997cb402f0784bca7fd292359454ccebd072cf4a83d2859f54c689c7
+  data.tar.gz: 40837369dda96179987ea46fb225dc72f857ed4a47cee224f60eaa52f1bcb3f1
 SHA512:
-  metadata.gz: fb16cd67f5f1113eba6f48ec7e3584fd8d97ca1fb75348f9c5f87bd0cd597f8f7256de83eb1f889a71b936b2c27bc186fa06cdda8abe634a31b8581027a952d8
-  data.tar.gz: 7e85036cae0a55cadfa38d5bb43281717a70d0ef7ce5077bc03ba27f201b94fc407e5b148123c0067cbe9b0f4c85092c52f647d39d6ee82563bcb6c5dc0974dc
+  metadata.gz: 6f3a09a6ed6392d19f8a78502a3dc1a16db82eebeae3b4e2f30e527cbebdbc4b50acc1e5b542b876620c02653d034548d20dad0e6e3568a3bf73785721cffbb5
+  data.tar.gz: d9ee18045c6070dd4b500e7c8208af3adc8662ecb756bcafdfe32279c49a0598d471caf7909f99c751e71b68f18307bd1dc0246f1f164553d02bef03fb896aee

data/lib/kitchen/terraform/config_attribute/fail_fast.rb

@@ -0,0 +1,38 @@
+# frozen_string_literal: true
+
+# Copyright 2016 New Context Services, Inc.
+#
+# Licensed under the Apache License, Version 2.0 (the "License");
+# you may not use this file except in compliance with the License.
+# You may obtain a copy of the License at
+#
+#     http://www.apache.org/licenses/LICENSE-2.0
+#
+# Unless required by applicable law or agreed to in writing, software
+# distributed under the License is distributed on an "AS IS" BASIS,
+# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+# See the License for the specific language governing permissions and
+# limitations under the License.
+
+require "kitchen"
+require "kitchen/terraform/config_attribute"
+require "kitchen/terraform/config_schemas/boolean"
+
+# This attribute toggles fail fast behaviour when verifying systems.
+#
+# If fail fast behaviour is enabled then Kitchen will halt on the first error raised by a system during verification;
+# else errors raised by systems will be queued until all systems have attempted verification.
+#
+# Type:: {http://www.yaml.org/spec/1.2/spec.html#id2803629 Boolean}
+# Required:: False
+# Default:: <code>true</code>
+# Example:: <code>fail_fast: false</code>
+module ::Kitchen::Terraform::ConfigAttribute::FailFast
+  ::Kitchen::Terraform::ConfigAttribute.new(
+      attribute: :fail_fast,
+      default_value: lambda do
+        true
+      end,
+      schema: ::Kitchen::Terraform::ConfigSchemas::Boolean,
+    ).apply config_attribute: self
+end

data/lib/kitchen/terraform/inspec.rb

@@ -40,6 +40,7 @@ module Kitchen
 
       # #exec executes InSpec.
       #
+      # @raise [::Kitchen::Terraform::Error] if executing InSpec fails.
       # @return [self]
       def exec
         @runner.run.tap do |exit_code|

data/lib/kitchen/terraform/system.rb

@@ -14,8 +14,11 @@
 # See the License for the specific language governing permissions and
 # limitations under the License.
 
+require "kitchen/terraform/error"
 require "kitchen/terraform/inspec_with_hosts"
 require "kitchen/terraform/inspec_without_hosts"
+require "kitchen/terraform/system_attrs_resolver"
+require "kitchen/terraform/system_hosts_resolver"
 
 module Kitchen
   module Terraform
@@ -53,32 +56,6 @@ module Kitchen
         self
       end
 
-      # #resolve_attrs resolves the attributes of the system which are contained in Terraform outputs.
-      #
-      # @param system_attrs_resolver [::Kitchen::Terraform::SystemAttrsResolver] the resolver.
-      # @return [self]
-      def resolve_attrs(system_attrs_resolver:)
-        system_attrs_resolver.resolve attrs_outputs_keys: @attrs_outputs.keys,
-                                      attrs_outputs_values: @attrs_outputs.values, system: self
-
-        self
-      end
-
-      # #resolve_hosts resolves the hosts of the system which are contained a Terraform output.
-      #
-      # @param system_hosts_resolver [::Kitchen::Terraform::SystemHostsResolver] the resolver.
-      # @return [self]
-      def resolve_hosts(system_hosts_resolver:)
-        system_hosts_resolver.resolve(
-          hosts_output: @mapping.fetch(:hosts_output) do
-            return self
-          end,
-          system: self,
-        )
-
-        self
-      end
-
       # #to_s returns a string representation of the system.
       #
       # @return [::String] the name of the system.
@@ -91,20 +68,23 @@ module Kitchen
       # @param inspec_options [::Hash] the options to be passed to InSpec.
       # @param inspec_profile_path [::String] the path to the profile which InSpec will execute.
       # @return [self]
-      def verify(inspec_options:, inspec_profile_path:)
-        if @hosts.empty?
-          ::Kitchen::Terraform::InSpecWithoutHosts
-        else
-          ::Kitchen::Terraform::InSpecWithHosts
-        end
-          .new(options: inspec_options.merge(attributes: @attributes), profile_path: inspec_profile_path)
-          .exec(system: self)
+      def verify(inspec_options:, inspec_profile_path:, outputs:)
+        resolve outputs: outputs
+        execute_inspec options: inspec_options, profile_path: inspec_profile_path
 
         self
+      rescue => error
+        raise ::Kitchen::Terraform::Error, "#{self}: #{error.message}"
       end
 
       private
 
+      def execute_inspec(options:, profile_path:)
+        inspec.new(options: options_with_attributes(options: options), profile_path: profile_path).exec(
+          system: self
+        )
+      end
+
       def initialize(mapping:)
         @attributes = {}
         @attrs_outputs = mapping.fetch :attrs_outputs do
@@ -115,6 +95,46 @@ module Kitchen
         end
         @mapping = mapping
       end
+
+      def inspec
+        if @hosts.empty?
+          ::Kitchen::Terraform::InSpecWithoutHosts
+        else
+          ::Kitchen::Terraform::InSpecWithHosts
+        end
+      end
+
+      def options_with_attributes(options:)
+        options.merge attributes: @attributes
+      end
+
+      def resolve(outputs:)
+        resolve_attrs outputs: outputs
+        resolve_hosts outputs: outputs
+      end
+
+      def resolve_attrs(outputs:)
+        ::Kitchen::Terraform::SystemAttrsResolver.new(outputs: outputs).resolve(
+          attrs_outputs_keys: @attrs_outputs.keys,
+          attrs_outputs_values: @attrs_outputs.values,
+          system: self
+        )
+
+        self
+      end
+
+      def resolve_hosts(outputs:)
+        if !@mapping.key? :hosts_output
+          return self
+        end
+
+        ::Kitchen::Terraform::SystemHostsResolver.new(outputs: outputs).resolve(
+          hosts_output: @mapping.fetch(:hosts_output),
+          system: self,
+        )
+
+        self
+      end
     end
   end
 end

data/lib/kitchen/terraform/system_attrs_resolver.rb

@@ -34,8 +34,7 @@ module Kitchen
 
         self
       rescue ::KeyError => key_error
-        raise ::Kitchen::Terraform::Error,
-              "Resolving the attrs of system #{system} failed\n#{key_error}"
+        raise ::Kitchen::Terraform::Error, "Resolving attrs failed\n#{key_error}"
       end
 
       private

data/lib/kitchen/terraform/system_hosts_resolver.rb

@@ -29,7 +29,7 @@ module Kitchen
       def resolve(hosts_output:, system:)
         system.add_hosts hosts: @outputs.fetch(hosts_output).fetch("value")
       rescue ::KeyError => key_error
-        raise ::Kitchen::Terraform::Error, "Resolving the hosts of system #{system} failed\n#{key_error}"
+        raise ::Kitchen::Terraform::Error, "Resolving hosts failed\n#{key_error}"
       end
 
       private

data/lib/kitchen/terraform/version.rb

@@ -72,7 +72,7 @@ module ::Kitchen::Terraform::Version
 
     # @api private
     def value
-      self.value = ::Gem::Version.new "4.5.0" if not @value
+      self.value = ::Gem::Version.new "4.6.0" if not @value
       @value
     end
 

data/lib/kitchen/verifier/terraform.rb

@@ -16,12 +16,11 @@
 
 require "kitchen"
 require "kitchen/terraform/config_attribute/color"
+require "kitchen/terraform/config_attribute/fail_fast"
 require "kitchen/terraform/config_attribute/systems"
 require "kitchen/terraform/configurable"
 require "kitchen/terraform/error"
 require "kitchen/terraform/inspec_options_mapper"
-require "kitchen/terraform/system_attrs_resolver"
-require "kitchen/terraform/system_hosts_resolver"
 
 module Kitchen
   # This namespace is defined by Kitchen.
@@ -59,6 +58,10 @@ module Kitchen
     #
     # {include:Kitchen::Terraform::ConfigAttribute::Color}
     #
+    # ==== fail_fast
+    #
+    # {include:Kitchen::Terraform::ConfigAttribute::FailFast}
+    #
     # ==== systems
     #
     # {include:Kitchen::Terraform::ConfigAttribute::Systems}
@@ -71,6 +74,7 @@ module Kitchen
       include ::Kitchen::Configurable
       include ::Kitchen::Logging
       include ::Kitchen::Terraform::ConfigAttribute::Color
+      include ::Kitchen::Terraform::ConfigAttribute::FailFast
       include ::Kitchen::Terraform::ConfigAttribute::Systems
       include ::Kitchen::Terraform::Configurable
       @api_version = 2
@@ -84,8 +88,9 @@ module Kitchen
       # @return [void]
       def call(_kitchen_state)
         load_outputs
-        config_systems.each do |system|
-          verify system: system
+        verify_systems
+        if !@error_messages.empty?
+          raise ::Kitchen::ActionFailed, @error_messages.join("\n\n")
         end
       rescue ::Kitchen::Terraform::Error => error
         raise ::Kitchen::ActionFailed, error.message
@@ -113,6 +118,14 @@ module Kitchen
 
       private
 
+      def handle_error(message:)
+        if config_fail_fast
+          raise ::Kitchen::Terraform::Error, message
+        else
+          @error_messages.push message
+        end
+      end
+
       def load_outputs
         instance.driver.retrieve_outputs do |outputs:|
           @outputs.replace ::Kitchen::Util.stringified_hash outputs
@@ -121,6 +134,7 @@ module Kitchen
 
       def initialize(configuration = {})
         init_config configuration
+        @error_messages = []
         @inspec_options = { "distinct_exit" => false }
         @outputs = {}
       end
@@ -141,24 +155,24 @@ module Kitchen
         raise ::Kitchen::ClientError, load_error.message
       end
 
-      def system_attrs_resolver
-        @system_attrs_resolver ||= ::Kitchen::Terraform::SystemAttrsResolver.new outputs: @outputs
-      end
-
-      def system_hosts_resolver
-        @system_hosts_resolver ||= ::Kitchen::Terraform::SystemHostsResolver.new outputs: @outputs
-      end
-
       def system_inspec_options(system:)
         ::Kitchen::Terraform::InSpecOptionsMapper.new(system: system).map options: @inspec_options.dup
       end
 
       def verify(system:)
-        ::Kitchen::Terraform::System
-          .new(mapping: system)
-          .resolve_attrs(system_attrs_resolver: system_attrs_resolver)
-          .resolve_hosts(system_hosts_resolver: system_hosts_resolver)
-          .verify(inspec_options: system_inspec_options(system: system), inspec_profile_path: inspec_profile_path)
+        ::Kitchen::Terraform::System.new(mapping: system).verify(
+          inspec_options: system_inspec_options(system: system),
+          inspec_profile_path: inspec_profile_path,
+          outputs: @outputs
+        )
+      rescue => error
+        handle_error message: error.message
+      end
+
+      def verify_systems
+        config_systems.each do |system|
+          verify system: system
+        end
       end
     end
   end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: kitchen-terraform
 version: !ruby/object:Gem::Version
-  version: 4.5.0
+  version: 4.6.0
 platform: ruby
 authors:
 - Aaron Lane
@@ -61,7 +61,7 @@ cert_chain:
   XAjP/LERlXAwM+NCUycJw7BZMn7L2wIdnq6Le1rL6VJS17CZt2heuRtsF+3g0egR
   x97GmUumeM67PGeVodlIV65tdXrdlqKlHcE=
   -----END CERTIFICATE-----
-date: 2019-04-10 00:00:00.000000000 Z
+date: 2019-04-11 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: guard-bundler
@@ -406,6 +406,7 @@ files:
 - lib/kitchen/terraform/config_attribute/client.rb
 - lib/kitchen/terraform/config_attribute/color.rb
 - lib/kitchen/terraform/config_attribute/command_timeout.rb
+- lib/kitchen/terraform/config_attribute/fail_fast.rb
 - lib/kitchen/terraform/config_attribute/lock.rb
 - lib/kitchen/terraform/config_attribute/lock_timeout.rb
 - lib/kitchen/terraform/config_attribute/parallelism.rb