packer-config 1.3.1 → 1.4.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 0ebf455ba8b1a7020ca59d720b991ed9dddc4b80
-  data.tar.gz: a1a59a07dd64d5db0ae356d7b8a4c7f5e06c103c
+  metadata.gz: e4920f738fb0ed191d9670882f3336847462a085
+  data.tar.gz: c41a894f64a07e8e61c1f94dfc5160fdf700880d
 SHA512:
-  metadata.gz: 0179e5f9a51d35102209bd304ec4ad1c3ec4646d8b34ef7c33e2d0df238ea0e41d75c4c36a030cc61af472912921dd7fad4e3f25f0c950ae3faeb42dc9f0ddee
-  data.tar.gz: 0ec5863e8a71cd33156c8f975002251c97cb448db2cf868e1280b0f56eb59c4d154e65d1adb354819060f908b85b9c5fdb325de03a08f16f66a56e8fdb0d36ab
+  metadata.gz: 59da5cd9f6be7590e63b8210049d4db889d4339e7f64b6f18e20cded754a377dddc8531e211fad5c801fde8aea16750aa931a27d61823d114c58756c8ccff963
+  data.tar.gz: 304585133fb571ad92c8864f731f0454d5888cc9c85ef47879ad18ba44db52d2e82feb56535964f6430b9f73d0922ecf9a9175c3e6254315f32d327ac0aabbd0

data/.travis.yml

@@ -3,5 +3,7 @@ rvm:
 - 2.0.0
 - 2.1.1
 - 2.1.5
+- 2.1.6
 - 2.2.0
+- 2.2.3
 - ruby-head

data/COPYRIGHT

@@ -5,6 +5,7 @@ Including, but not limited to:
   Ian Chesal
   Fraser Cobb
   Greg Poirier
+  Greg Diamond
 
 For licensing information please see LICENSE. Copyright holders contributing to
 this project agree to have their contributions licensed under the terms of the

data/README.md

@@ -23,6 +23,10 @@ Bonus: you can really go to town with templates when it's all done it Ruby.
 
     require 'packer-config'
 
+## Requires
+
+* [Packer](http://packer.io) version 0.8.5 or higher
+
 ### Builders
 
 The following [Packer builders](http://www.packer.io/docs/templates/builders.html) are currently implemented:
@@ -35,6 +39,8 @@ The following [Packer builders](http://www.packer.io/docs/templates/builders.htm
 * [vmware-iso](https://www.packer.io/docs/builders/vmware-iso)
 * [null](https://www.packer.io/docs/builders/null.html)
 
+[Communicators](https://www.packer.io/docs/templates/communicator.html) are supported as options on Builders in `packer-config`. The `none`, `ssh`, and `winrm` communicators are all available as is the `docker` communicator on the Docker-type builders. `packer-config` will raise an error if you try to use a Communicator type that isn't valid for the Builder.
+
 ### Provisioners
 
 The following [Packer provisioners](http://www.packer.io/docs/templates/provisioners.html) are currently implemented:
@@ -79,15 +85,16 @@ This example is based on the integration test [spec/integration/centos_vagrant_s
     builder.guest_additions_path "VBoxGuestAdditions_#{pconfig.macro.Version}.iso"
     builder.guest_os_type "RedHat_64"
     builder.http_directory "scripts/kickstart"
-    builder.iso_checksum '08be09fd7276822bd3468af8f96198279ffc41f0'
+    builder.iso_checksum '7bb8c1c23a4fdef93e6f0a6347d570e5764d0b38'
     builder.iso_checksum_type 'sha1'
-    builder.iso_url "#{pconfig.variable 'mirror'}/6.6/isos/x86_64/CentOS-6.6-x86_64-bin-DVD1.iso"
+    builder.iso_url "#{pconfig.variable 'mirror'}/6.7/isos/x86_64/CentOS-6.7-x86_64-bin-DVD1.iso"
     builder.output_directory "#{OS}-x86_64-virtualbox"
     builder.shutdown_command "echo 'vagrant'|sudo -S /sbin/halt -h -p"
+    builder.communicator "ssh"
     builder.ssh_password "vagrant"
     builder.ssh_port 22
     builder.ssh_username "vagrant"
-    builder.ssh_wait_timeout "10000s"
+    builder.ssh_timeout "10000s"
     builder.vboxmanage [
       [
         "modifyvm",

data/RELEASENOTES.md

@@ -1,5 +1,11 @@
 # packer-config Release Notes
 
+## 1.4.0
+
+* Adds support for Communicators (with help from [diamond29][]).
+* Validates the minimum version of Packer is met before running any Packer command line calls. Minimum version is set to 0.8.5 to support Communicators.
+* Bumps integration spec and working example in the README to CentOS 6.7.
+
 ## 1.3.1
 
 * Adds a fix for [issue #8](https://github.com/ianchesal/packer-config/issues/8). Removes psuedo-terminals from the runner library which were causing the problem.
@@ -34,3 +40,4 @@
   [frasercobb]: https://github.com/frasercobb
   [grepory]: https://github.com/grepory
   [ianchesal]: https://github.com/ianchesal
+  [diamond29]: https://github.com/diamond29

data/TODO.md

@@ -1,6 +1,5 @@
 # TODO
 
-* Use [lowered-expectations](https://rubygems.org/gems/lowered-expectations) to enforce a minimum version of `packer` exists in order to use this gem
 * Add an option to Packer::Config#validate to run the configuration through packer's `validate` command
 * Add spec tests for every method on every sub-class. Found during integration testing that some methods on the sub-classes had typos in the `__*` method calls. Spec tests would have caught this.
 * Look in to something like VCR to drive the tests of the child classes -- there's a lot of repetitive testing that could be done on them.

data/lib/packer-config.rb

@@ -8,6 +8,7 @@ require 'packer/postprocessor'
 require 'packer/macro'
 require 'packer/envvar'
 require 'packer/version'
+require 'lowered/expectations'
 
 module Packer
   class Config < Packer::DataObject
@@ -20,6 +21,8 @@ module Packer
     attr_reader   :envvar
     attr_reader   :output_file
 
+    PACKER_VERSION = '0.8.5'
+
     def initialize(file)
       super()
       self.data['variables'] = {}
@@ -35,6 +38,7 @@ module Packer
 
     def validate
       super
+      verify_packer_version
       if self.builders.length == 0
         raise DataValidationError.new("At least one builder is required")
       end
@@ -117,10 +121,6 @@ module Packer
       self.__add_string('description', description)
     end
 
-    def min_packer_version(version)
-      self.__add_string('min_packer_version', version)
-    end
-
     def variables
       self.data['variables']
     end
@@ -160,8 +160,18 @@ module Packer
     end
 
     private
+
     attr_writer :output_file
     attr_writer :macro
     attr_writer :envvar
+
+    def min_packer_version(version)
+      self.__add_string('min_packer_version', version)
+    end
+
+    def verify_packer_version
+      min_packer_version PACKER_VERSION
+      LoweredExpectations.expect('packer', ">= #{PACKER_VERSION}", vopt: 'version')
+    end
   end
 end

data/lib/packer/builder.rb

@@ -39,6 +39,8 @@ module Packer
       }.fetch(type).new
     end
 
+    attr_reader :communicators
+
     def self.types
       VALID_BUILDER_TYPES
     end
@@ -46,13 +48,99 @@ module Packer
     def initialize
       super
       self.add_required('type')
+      self.communicators = []
     end
 
     def name(name)
       self.__add_string('name', name)
     end
 
+    # @ianchesal: Communicators are technically Templates in Packer land but
+    # they modify Builders. Weird. So we'll treat them as Builder attributes.
+    # See: https://packer.io/docs/templates/communicator.html
+    def communicator(comm)
+      raise(DataValidationError, "unknown communicator protocol #{comm}") unless communicators.include? comm
+      self.__add_string('communicator', comm)
+    end
+
+    # Technically these only apply if the communicator is ssh
+    def ssh_host(host)
+      self.__add_string('ssh_host', host)
+    end
+
+    def ssh_port(port)
+      self.__add_integer('ssh_port', port)
+    end
+
+    def ssh_username(username)
+      self.__add_string('ssh_username', username)
+    end
+
+    def ssh_password(password)
+      self.__add_string('ssh_password', password)
+    end
+
+    def ssh_private_key_file(filename)
+      self.__add_string('ssh_private_key_file', filename)
+    end
+
+    def ssh_pty(pty)
+      self.__add_boolean('ssh_pty', pty)
+    end
+
+    def ssh_timeout(timeout)
+      self.__add_string('ssh_timeout', timeout)
+    end
+
+    def ssh_handshake_attempts(attempts)
+      self.__add_integer('ssh_handshake_attempts', attempts)
+    end
+
+    def ssh_disable_agent(disable)
+      self.__add_boolean('ssh_disable_agent', disable)
+    end
+
+    def ssh_bastion_host(hostname)
+      self.__add_string('ssh_bastion_host', hostname)
+    end
+
+    def ssh_bastion_username(username)
+      self.__add_string('ssh_bastion_username', username)
+    end
+
+    def ssh_bastion_password(password)
+      self.__add_string('ssh_bastion_password', password)
+    end
+
+    def ssh_bastion_private_key_file(filename)
+      self.__add_string('ssh_bastion_private_key_file', filename)
+    end
+
+    # Technically these only apply if the communicator is winrm
+    def winrm_host(host)
+      self.__add_string('winrm_host', host)
+    end
+
+    def winrm_port(port)
+      self.__add_string('winrm_port', port)
+    end
+
+    def winrm_username(username)
+      self.__add_string('winrm_username', username)
+    end
+
+    def winrm_password(password)
+      self.__add_string('winrm_password', password)
+    end
+
+    def winrm_timeout(timeout)
+      self.__add_string('winrm_timeout', timeout)
+    end
+
     private
+
+    attr_writer :communicators
+
     def self.validate_type(type)
       VALID_BUILDER_TYPES.include? type
     end

data/lib/packer/builders/amazon.rb

@@ -12,8 +12,9 @@ module Packer
           'instance_type',
           'region',
           'source_ami',
-          'ssh_username'
+          'communicator'
         )
+        self.communicators = %w(none ssh winrm)
       end
 
       def access_key(key)
@@ -36,10 +37,6 @@ module Packer
         self.__add_string('source_ami', name)
       end
 
-      def ssh_username(username)
-        self.__add_string('ssh_username', username)
-      end
-
       def secret_key(key)
         self.__add_string('secret_key', key)
       end
@@ -104,18 +101,6 @@ module Packer
         self.__add_array_of_strings('security_group_ids', ids, %w[security_group_id])
       end
 
-      def ssh_port(port)
-        self.__add_integer('ssh_port', port)
-      end
-
-      def ssh_private_key_file(file)
-        self.__add_string('ssh_private_key_file', file)
-      end
-
-      def ssh_timeout(time)
-        self.__add_string('ssh_timeout', time)
-      end
-
       def subnet_id(id)
         self.__add_string('subnet_id', id)
       end

data/lib/packer/builders/null.rb

@@ -8,32 +8,7 @@ module Packer
       def initialize
         super
         self.data['type'] = NULL
-        self.add_required(
-          'host',
-          'ssh_password',
-          'ssh_private_key_file',
-          'ssh_username'
-        )
-      end
-
-      def host(name)
-        self.__add_string('host', name)
-      end
-
-      def ssh_password(passwd)
-        self.__add_string('ssh_password', passwd)
-      end
-
-      def ssh_private_key_file(filename)
-        self.__add_string('ssh_private_key_file', filename)
-      end
-
-      def ssh_username(name)
-        self.__add_string('ssh_username', name)
-      end
-
-      def port(number)
-        self.__add_integer('port', number)
+        self.communicators = %w(none ssh winrm)
       end
     end
   end

data/lib/packer/builders/virtualbox.rb

@@ -12,8 +12,9 @@ module Packer
           'iso_checksum',
           'iso_checksum_type',
           'iso_url',
-          'ssh_username'
+          'communicator'
         )
+        self.communicators = %w(none ssh winrm)
       end
 
       def iso_checksum(checksum)
@@ -32,10 +33,6 @@ module Packer
         self.__add_array_of_strings('iso_urls', urls, %[iso_url])
       end
 
-      def ssh_username(username)
-        self.__add_string('ssh_username', username)
-      end
-
       def boot_command(commands)
         self.__add_array_of_strings('boot_command', commands)
       end
@@ -112,30 +109,6 @@ module Packer
         self.__add_string('shutdown_timeout', time)
       end
 
-      def ssh_host_port_min(port_number)
-        self.__add_integer('ssh_host_port_min', port_number)
-      end
-
-      def ssh_host_port_max(port_number)
-        self.__add_integer('ssh_host_port_max', port_number)
-      end
-
-      def ssh_key_path(path)
-        self.__add_string('ssh_key_path', path)
-      end
-
-      def ssh_password(password)
-        self.__add_string('ssh_password', password)
-      end
-
-      def ssh_port(port_number)
-        self.__add_integer('ssh_port', port_number)
-      end
-
-      def ssh_wait_timeout(time)
-        self.__add_string('ssh_wait_timeout', time)
-      end
-
       def vboxmanage(array_of_commands)
         self.__add_array_of_array_of_strings('vboxmanage', array_of_commands)
       end

data/lib/packer/builders/vmware_iso.rb

@@ -12,8 +12,9 @@ module Packer
           'iso_checksum',
           'iso_checksum_type',
           'iso_url',
-          'ssh_username'
+          'communicator'
         )
+        self.communicators = %w(none ssh winrm)
       end
 
       def iso_checksum(checksum)
@@ -32,10 +33,6 @@ module Packer
         self.__add_array_of_strings('iso_urls', urls, %[iso_url])
       end
 
-      def ssh_username(username)
-        self.__add_string('ssh_username', username)
-      end
-
       def boot_command(commands)
         self.__add_array_of_strings('boot_command', commands)
       end
@@ -130,38 +127,6 @@ module Packer
         self.__add_boolean('skip_compaction', bool)
       end
 
-      def ssh_host(host)
-        self.__add_string('ssh_host', host)
-      end
-
-      def ssh_host_port_min(port_number)
-        self.__add_integer('ssh_host_port_min', port_number)
-      end
-
-      def ssh_host_port_max(port_number)
-        self.__add_integer('ssh_host_port_max', port_number)
-      end
-
-      def ssh_key_path(path)
-        self.__add_string('ssh_key_path', path)
-      end
-
-      def ssh_password(password)
-        self.__add_string('ssh_password', password)
-      end
-
-      def ssh_port(port_number)
-        self.__add_integer('ssh_port', port_number)
-      end
-
-      def ssh_skip_request_pty(bool)
-        self.__add_boolean('ssh_skip_request_pty', bool)
-      end
-
-      def ssh_wait_timeout(time)
-        self.__add_string('ssh_wait_timeout', time)
-      end
-
       def tools_upload_flavor(flavor)
         self.__add_string('tools_upload_flavor', flavor)
       end

data/lib/packer/builders/vmware_vmx.rb

@@ -6,22 +6,15 @@ module Packer
         data['type'] = VMWARE_VMX
         add_required(
           'source_path',
-          'ssh_username'
+          'communicator'
         )
+        self.communicators = %w(none ssh winrm)
       end
 
       def source_path(path)
         __add_string('source_path', path)
       end
 
-      def ssh_username(username)
-        __add_string('ssh_username', username)
-      end
-
-      def ssh_password(password)
-        __add_string('ssh_password', password)
-      end
-
       def boot_command(commands)
         __add_array_of_strings('boot_command', commands)
       end
@@ -70,22 +63,6 @@ module Packer
         __add_boolean('skip_compaction', bool)
       end
 
-      def ssh_key_path(path)
-        __add_string('ssh_key_path', path)
-      end
-
-      def ssh_port(port)
-        __add_integer('ssh_port', port)
-      end
-
-      def ssh_skip_request_pty(bool)
-        __add_boolean('ssh_skip_request_pty', bool)
-      end
-
-      def ssh_wait_timeout(timeout)
-        __add_string('ssh_wait_timeout', timeout)
-      end
-
       def vm_name(name)
         __add_string('vm_name', name)
       end

data/lib/packer/version.rb

@@ -1,3 +1,3 @@
 module Packer
-    VERSION = "1.3.1"
+    VERSION = "1.4.0"
 end

data/packer-config.gemspec

@@ -6,7 +6,7 @@ require 'packer/version'
 Gem::Specification.new do |spec|
   spec.name          = "packer-config"
   spec.version       = Packer::VERSION
-  spec.authors       = ["Ian Chesal", "Fraser Cobb", "Greg Poirier", "Matasano Security"]
+  spec.authors       = ["Ian Chesal", "Fraser Cobb", "Greg Poirier", "Matasano Security", "Greg Diamond"]
   spec.email         = ["ian.chesal@gmail.com"]
   spec.summary       = 'An object model to build packer.io configurations in Ruby.'
   spec.description   = <<-END
@@ -26,6 +26,8 @@ END
   spec.test_files    = spec.files.grep(%r{^(test|spec|features)/})
   spec.require_paths = ["lib"]
 
+  spec.add_runtime_dependency "lowered-expectations"
+
   spec.add_development_dependency "bundler", "~> 1.7"
   spec.add_development_dependency "rake", "~> 10.3"
   spec.add_development_dependency "rspec", "~> 3.0"

data/spec/integration/centos_vagrant_spec.rb

@@ -18,15 +18,16 @@ RSpec.describe Packer::Config do
     builder.guest_additions_path "VBoxGuestAdditions_#{pconfig.macro.Version}.iso"
     builder.guest_os_type "RedHat_64"
     builder.http_directory "scripts/kickstart"
-    builder.iso_checksum '08be09fd7276822bd3468af8f96198279ffc41f0'
+    builder.iso_checksum '7bb8c1c23a4fdef93e6f0a6347d570e5764d0b38'
     builder.iso_checksum_type 'sha1'
-    builder.iso_url "#{pconfig.variable 'mirror'}/6.6/isos/x86_64/CentOS-6.6-x86_64-bin-DVD1.iso"
+    builder.iso_url "#{pconfig.variable 'mirror'}/6.7/isos/x86_64/CentOS-6.7-x86_64-bin-DVD1.iso"
     builder.output_directory "#{OS}-x86_64-virtualbox"
     builder.shutdown_command "echo 'vagrant'|sudo -S /sbin/halt -h -p"
+    builder.communicator "ssh"
     builder.ssh_password "vagrant"
     builder.ssh_port 22
     builder.ssh_username "vagrant"
-    builder.ssh_wait_timeout "10000s"
+    builder.ssh_timeout "10000s"
     builder.vboxmanage [
       [
         "modifyvm",

data/spec/packer/builder_spec.rb

@@ -23,4 +23,10 @@ RSpec.describe Packer::Builder do
       builder.data.delete('name')
     end
   end
+
+  describe '#communicator' do
+    it 'raises an error if you try to set an invalid communicator' do
+      expect { builder.communicator 'foo' }.to raise_error Packer::DataObject::DataValidationError
+    end
+  end
 end

data/spec/packer/builders/amazon_spec.rb

@@ -57,19 +57,39 @@ end
 RSpec.describe Packer::Builder::Amazon::EBS do
   let(:builder) { Packer::Builder.get_builder(Packer::Builder::AMAZON_EBS) }
 
-  describe '#initialize' do
-    it 'has a type of amazon-ebs' do
-      expect(builder.data['type']).to eq(Packer::Builder::AMAZON_EBS)
-    end
+  it 'has a type of amazon-ebs' do
+    expect(builder.data['type']).to eq(Packer::Builder::AMAZON_EBS)
+  end
+
+  it 'requires ami_name, instance_type, region, source_ami, and communicator' do
+    expect { builder.validate }.to raise_error(Packer::DataObject::DataValidationError)
+    builder.ami_name 'foo'
+    builder.instance_type 'foo'
+    builder.region 'foo'
+    builder.source_ami 'foo'
+    builder.communicator 'ssh'
+    expect { builder.validate }.not_to raise_error
   end
 end
 
 RSpec.describe Packer::Builder::Amazon::Instance do
   let(:builder) { Packer::Builder.get_builder(Packer::Builder::AMAZON_INSTANCE) }
 
-  describe '#initialize' do
-    it 'has a type of amazon-instance' do
-      expect(builder.data['type']).to eq(Packer::Builder::AMAZON_INSTANCE)
-    end
+  it 'has a type of amazon-instance' do
+    expect(builder.data['type']).to eq(Packer::Builder::AMAZON_INSTANCE)
+  end
+
+  it 'requires ami_name, instance_type, region, source_ami, account_id, s3_bucket, x509_cert_path, x509_key_path, and communicator' do
+    expect { builder.validate }.to raise_error(Packer::DataObject::DataValidationError)
+    builder.ami_name 'foo'
+    builder.instance_type 'foo'
+    builder.region 'foo'
+    builder.source_ami 'foo'
+    builder.account_id 'foo'
+    builder.s3_bucket 'foo'
+    builder.x509_cert_path 'foo'
+    builder.x509_key_path 'foo'
+    builder.communicator 'ssh'
+    expect { builder.validate }.not_to raise_error
   end
 end

data/spec/packer/builders/null_spec.rb

@@ -5,21 +5,7 @@ RSpec.describe Packer::Builder::Null do
   let(:builder) { Packer::Builder.get_builder(Packer::Builder::NULL) }
   let(:some_string) { 'some string' }
 
-  it 'requires a number of parameters to be valid' do
-    expect{ builder.validate }.to raise_error
-    builder.host :some_string
-    expect{ builder.validate }.to raise_error
-    builder.ssh_password :some_string
-    expect{ builder.validate }.to raise_error
-    builder.ssh_private_key_file :some_string
-    expect{ builder.validate }.to raise_error
-    builder.ssh_username :some_string
-    expect(builder.validate).to be_truthy
-  end
-
-  describe '#initialize' do
-    it 'has a type of null' do
-      expect(builder.data['type']).to eq(Packer::Builder::NULL)
-    end
+  it 'has a type of null' do
+    expect(builder.data['type']).to eq(Packer::Builder::NULL)
   end
 end

data/spec/packer/builders/virtualbox_spec.rb

@@ -7,10 +7,17 @@ RSpec.describe Packer::Builder::VirtualBoxISO do
   let(:in_commands_mixed)    { [["command1",  1 ], ["command2",  2 ]] }
   let(:out_commands_strings) { [["command1", "1"], ["command2", "2"]] }
 
-  describe '#initialize' do
-    it 'has a type of virtualbox-iso' do
-      expect(builder.data['type']).to eq(Packer::Builder::VIRTUALBOX_ISO)
-    end
+  it 'has a type of virtualbox-iso' do
+    expect(builder.data['type']).to eq(Packer::Builder::VIRTUALBOX_ISO)
+  end
+
+  it 'requires iso_checksum, iso_checksum_type, iso_url and communicator' do
+    expect { builder.validate }.to raise_error(Packer::DataObject::DataValidationError)
+    builder.iso_checksum '88197272b2a442402820fcc788a8cc7a'
+    builder.iso_checksum_type "MD5"
+    builder.iso_url 'path'
+    builder.communicator 'ssh'
+    expect { builder.validate }.not_to raise_error
   end
 
   describe '#vboxmanage' do

data/spec/packer/builders/vmware_iso_spec.rb

@@ -4,19 +4,17 @@ require 'spec_helper'
 RSpec.describe Packer::Builder::VMWareISO do
   let(:builder) { Packer::Builder.get_builder(Packer::Builder::VMWARE_ISO) }
 
-  describe '#initialize' do
-    it 'has a type of VMWare ISO' do
-      expect(builder.data['type']).to eq(Packer::Builder::VMWARE_ISO)
-    end
+  it 'has a type of VMWare ISO' do
+    expect(builder.data['type']).to eq(Packer::Builder::VMWARE_ISO)
+  end
 
-    it 'requires iso_checksum' do
-      expect { builder.validate }.to raise_error(Packer::DataObject::DataValidationError)
-      builder.iso_checksum '88197272b2a442402820fcc788a8cc7a'
-      builder.iso_checksum_type "MD5"
-      builder.iso_url 'path'
-      builder.ssh_username 'user'
-      expect { builder.validate }.not_to raise_error
-    end
+  it 'requires iso_checksum, iso_checksum_type, iso_url and communicator' do
+    expect { builder.validate }.to raise_error(Packer::DataObject::DataValidationError)
+    builder.iso_checksum '88197272b2a442402820fcc788a8cc7a'
+    builder.iso_checksum_type "MD5"
+    builder.iso_url 'path'
+    builder.communicator 'ssh'
+    expect { builder.validate }.not_to raise_error
   end
 
   describe '#vmx_data' do

data/spec/packer/builders/vmware_vmx_spec.rb

@@ -4,17 +4,15 @@ require 'spec_helper'
 RSpec.describe Packer::Builder::VMWareVMX do
   let(:builder) { Packer::Builder.get_builder(Packer::Builder::VMWARE_VMX) }
 
-  describe '#initialize' do
-    it 'has a type of VMWare VMX' do
-      expect(builder.data['type']).to eq(Packer::Builder::VMWARE_VMX)
-    end
+  it 'has a type of VMWare VMX' do
+    expect(builder.data['type']).to eq(Packer::Builder::VMWARE_VMX)
+  end
 
-    it 'requires source_path' do
-      expect { builder.validate }.to raise_error(Packer::DataObject::DataValidationError)
-      builder.source_path 'path'
-      builder.ssh_username 'user'
-      expect { builder.validate }.not_to raise_error
-    end
+  it 'requires source_path and communicator' do
+    expect { builder.validate }.to raise_error(Packer::DataObject::DataValidationError)
+    builder.source_path 'path'
+    builder.communicator 'ssh'
+    expect { builder.validate }.not_to raise_error
   end
 
   describe '#vmx_data' do

data/spec/packer_config_spec.rb

@@ -22,6 +22,7 @@ RSpec.describe Packer::Config do
     it 'returns true for a valid instance' do
       expect(packer.builders).to receive(:length).and_return(1)
       expect(Packer::Runner).to receive(:run!).and_return('')
+      expect(LoweredExpectations).to receive(:expect).and_return(true)
       FakeFS do
         expect(packer.validate).to be_truthy
       end
@@ -29,10 +30,22 @@ RSpec.describe Packer::Config do
 
     it 'raises an error for an invalid instance' do
       expect(packer.builders).to receive(:length).and_return(0)
+      expect(LoweredExpectations).to receive(:expect).and_return(true)
       FakeFS do
         expect { packer.validate }.to raise_error
       end
     end
+
+    it 'sets the minimum Packer version' do
+      expect(packer.builders).to receive(:length).and_return(1)
+      expect(Packer::Runner).to receive(:run!).and_return('')
+      expect(LoweredExpectations).to receive(:expect).and_return(true)
+      expect(packer.data['min_packer_version']).to be_falsey
+      FakeFS do
+        expect(packer.validate).to be_truthy
+      end
+      expect(packer.data['min_packer_version']).to eq(Packer::Config::PACKER_VERSION)
+    end
   end
 
   describe '#dump' do

metadata

@@ -1,18 +1,33 @@
 --- !ruby/object:Gem::Specification
 name: packer-config
 version: !ruby/object:Gem::Version
-  version: 1.3.1
+  version: 1.4.0
 platform: ruby
 authors:
 - Ian Chesal
 - Fraser Cobb
 - Greg Poirier
 - Matasano Security
+- Greg Diamond
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2015-07-15 00:00:00.000000000 Z
+date: 2015-09-14 00:00:00.000000000 Z
 dependencies:
+- !ruby/object:Gem::Dependency
+  name: lowered-expectations
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
 - !ruby/object:Gem::Dependency
   name: bundler
   requirement: !ruby/object:Gem::Requirement
@@ -229,7 +244,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.4.6
+rubygems_version: 2.4.8
 signing_key: 
 specification_version: 4
 summary: An object model to build packer.io configurations in Ruby.