vagrant_spec 0.0.5 → 0.0.6

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: fb07a53bcd6698b0ad2af73417b1fb2235489274
-  data.tar.gz: 776a3dfe81a097b90e4c2a6ab68bf7803ec7c1a8
+  metadata.gz: 7222a82ae26538dbe990ae8edbe2ed8f903595de
+  data.tar.gz: 31a633030d6cafc40d9802e9caec5e7e715e6e1a
 SHA512:
-  metadata.gz: bed91f539ff20534a965e36b6300dc7c90c2ef10d0e2e189031cae58bbaa66cfc56068e3b9b815850b38bdf4b69d2a23b2f29413cee16313581cdac084b737cb
-  data.tar.gz: b44fb0bbe52e6223e5bfb97a0c6af652a8106fd0285f5c41e84f4b2644449d55e711958e0410fd4b513e30cab71a7a8277f50695c44eb189c57524bb87049ecf
+  metadata.gz: b30568dd3353610a7d26836866a5042de4282bf601fec5e653e419405872b30b55597a6d94e21c0efcb1e7626efff8fac8336f647b71379809b1826eaf10ef80
+  data.tar.gz: 913b13f14fd5f8893d32cc6618d8e7a86704f34e4fd3d83090cec458ed03d118f8cb40b94af48824704bbf58452e42cdc5014fb7bfaae08c422c8d526da0838a

data/CHANGELOG.md

@@ -1,11 +1,29 @@
 # Change Log
 
+## [v0.0.6](https://github.com/miroswan/vagrant_spec/tree/v0.0.6) (2016-08-13)
+[Full Changelog](https://github.com/miroswan/vagrant_spec/compare/v0.0.5...v0.0.6)
+
+**Closed issues:**
+
+- Test command spews stacktrace when no nodes are found [\#17](https://github.com/miroswan/vagrant_spec/issues/17)
+- Wiki should contain a walkthrough of how to get started with system config testing [\#8](https://github.com/miroswan/vagrant_spec/issues/8)
+
+**Merged pull requests:**
+
+- Develop [\#19](https://github.com/miroswan/vagrant_spec/pull/19) ([miroswan](https://github.com/miroswan))
+- Fix/handle no machines [\#18](https://github.com/miroswan/vagrant_spec/pull/18) ([miroswan](https://github.com/miroswan))
+- Fix/directory [\#16](https://github.com/miroswan/vagrant_spec/pull/16) ([miroswan](https://github.com/miroswan))
+- Update README.md [\#15](https://github.com/miroswan/vagrant_spec/pull/15) ([miroswan](https://github.com/miroswan))
+
 ## [v0.0.5](https://github.com/miroswan/vagrant_spec/tree/v0.0.5) (2016-08-10)
 [Full Changelog](https://github.com/miroswan/vagrant_spec/compare/v0.0.4...v0.0.5)
 
-**Closed issues:**
+**Fixed bugs:**
 
 - SSH Issue [\#14](https://github.com/miroswan/vagrant_spec/issues/14)
+
+**Closed issues:**
+
 - Fix spelling and grammatical issues in the documentation [\#7](https://github.com/miroswan/vagrant_spec/issues/7)
 
 **Merged pull requests:**

data/README.md

@@ -9,6 +9,8 @@ Vagrant Spec is a Vagrant plugin that makes integration testing for deployments
 to clustered systems a breeze. It also separates the build and deployment steps
 to clearly separate pipeline tasks. 
 
+For a complete tutorial, reference the [Welcome to VagrantSpec](https://github.com/miroswan/vagrant_spec/wiki/Welcome-to-VagrantSpec)
+
 ## Installation
 
 ```vagrant plugin install vagrant_spec```

data/lib/vagrant_spec/command/init.rb

@@ -1,5 +1,7 @@
 # encoding: UTF-8
 
+require 'fileutils'
+
 require 'vagrant_spec/ansible_inventory'
 require 'vagrant_spec/machine_data'
 require 'vagrant_spec/config'
@@ -30,6 +32,7 @@ module VagrantSpec
 
       def execute
         return unless parse_opts
+        FileUtils.mkdir @directory unless Dir.exist? @directory
         unless @ansible_inventory == DEFAULTS['ansible_inventory']
           VagrantSpec::AnsibleInventory.new(@env).generate
         end

data/lib/vagrant_spec/test_plan.rb

@@ -27,14 +27,21 @@ module VagrantSpec
     # This will fail if any of the tests fail, but it will allow all tests to
     # run
     def run
+      print_banner
+      @test_plan.each do |plan|
+        found_nodes = nodes(plan)
+        if found_nodes
+          found_nodes.each { |node| execute_plan_tests(node, plan) }
+        end
+      end
+      exit @ret_code
+    end
+
+    def print_banner
       @env.ui.info('*******************************************************')
       @env.ui.info('***************** ServerSpec Test Run *****************')
       @env.ui.info('*******************************************************')
       @env.ui.info('')
-      @test_plan.each do |plan|
-        nodes(plan).each { |node| execute_plan_tests(node, plan) }
-      end
-      exit @ret_code
     end
 
     # Return array of active Vagrant machines

data/lib/vagrant_spec/version.rb

@@ -2,5 +2,5 @@
 
 # Requisite version info
 module VagrantSpec
-  VERSION = '0.0.5'.freeze
+  VERSION = '0.0.6'.freeze
 end

data/scripts/poor_mans_smoke_test.sh

@@ -16,6 +16,7 @@ bundle exec vagrant spec test -h
 bundle exec vagrant spec no_command -h
 rm -f "serverspec/spec_helper.rb"
 rm -f ".vagrantspec_machine_data"
+rm -f "vagrantspec_inventory"
 bundle exec vagrant up
 bundle exec vagrant spec init
 ansible-playbook site.yml -i vagrantspec_inventory

data/spec/unit/vagrant_spec_test/command_spec/init_spec.rb

@@ -32,6 +32,8 @@ describe VagrantSpec::Command::Init do
       allow_any_instance_of(VagrantSpec::MachineData).to      receive(:generate)
       allow(mock_ansible_inventory).to receive(:generate)
       allow(mock_machine_data).to      receive(:generate)
+      allow(Dir).to                    receive(:exist?) { false }
+      allow(FileUtils).to              receive(:mkdir)
     end
   end
 
@@ -55,6 +57,14 @@ describe VagrantSpec::Command::Init do
       end
     end
 
+    context 'when the @directory does not exist' do
+      it '#execute creates the @directory' do
+        execute_protection_proc.call
+        expect(FileUtils).to receive(:mkdir).with(subject.directory)
+        subject.execute
+      end
+    end
+
     context 'and when @ansible_inventory eq empty hash,' do
       it '#execute creates an instance of VagrantSpec::SpecHelper' do
         execute_protection_proc.call

data/spec/unit/vagrant_spec_test/test_plan_spec.rb

@@ -10,10 +10,6 @@ require 'rspec/support/spec/stderr_splitter'
 describe VagrantSpec::TestPlan do
   include RSpec::Support::InSubProcess
   include_context 'unit'
-
-  ##############################################################################
-  # mocks
-
   include_examples 'shared_mocks'
 
   let(:mock_plan) do
@@ -40,9 +36,6 @@ describe VagrantSpec::TestPlan do
 
   let(:mock_ssh_backend) { double(Specinfra::Backend::Ssh) }
 
-  ##############################################################################
-  # Stubs
-
   before do
     allow_any_instance_of(VagrantSpec::MachineFinder)
       .to receive(:match_nodes) { [double(Vagrant::Machine)] }
@@ -63,6 +56,11 @@ describe VagrantSpec::TestPlan do
 
   subject { VagrantSpec::TestPlan.new(iso_env) }
 
+  it '#print_banner prints the testing initialization banner' do
+    expect(mock_ui).to receive(:info)
+    subject.print_banner
+  end
+
   context 'when passing a Regexp object' do
     it '#nodes calls match_nodes on a machine_finder instance' do
       expect(subject.m_finder).to receive(:match_nodes)
@@ -93,13 +91,6 @@ describe VagrantSpec::TestPlan do
     end
   end
 
-  ##############################################################################
-  # Testing #execute_plan_tests
-  #
-  # These tests must be executed in a sub process because execute_plan_tests
-  # executes clear_examples. clear_examples modifies global state, so we must
-  # contain it.
-
   def execute_plan_tests_proc
     proc do
       allow(subject).to receive(:close_ssh)
@@ -129,4 +120,14 @@ describe VagrantSpec::TestPlan do
       expect(subject).to receive(:configure_serverspec)
     end
   end
+
+  context 'when nodes returns nil' do
+    it '#run does not fail with NoMethodError' do
+      allow(subject).to receive(:execute_plan_tests)
+      allow(subject).to receive(:exit)
+      allow(subject).to receive(:nodes) { nil }
+
+      expect { subject.run }.not_to raise_error
+    end
+  end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: vagrant_spec
 version: !ruby/object:Gem::Version
-  version: 0.0.5
+  version: 0.0.6
 platform: ruby
 authors:
 - Demitri
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2016-08-10 00:00:00.000000000 Z
+date: 2016-08-13 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: serverspec