dopi 0.17.0

data/spec/spec_helper.rb

@@ -0,0 +1,104 @@
+require 'simplecov'
+SimpleCov.start do
+  add_filter '/spec/'
+  add_filter '/.bundle/'
+  add_filter '/vendor/'
+end
+
+require 'dopi'
+require 'command_helper'
+require 'rspec_command'
+
+Dopi.log.level = ::Logger::FATAL
+
+# This file was generated by the `rspec --init` command. Conventionally, all
+# specs live under a `spec` directory, which RSpec adds to the `$LOAD_PATH`.
+# The generated `.rspec` file contains `--require spec_helper` which will cause this
+# file to always be loaded, without a need to explicitly require it in any files.
+#
+# Given that it is always loaded, you are encouraged to keep this file as
+# light-weight as possible. Requiring heavyweight dependencies from this file
+# will add to the boot time of your test suite on EVERY test run, even for an
+# individual file that may not need all of that loaded. Instead, consider making
+# a separate helper file that requires the additional dependencies and performs
+# the additional setup, and require it from the spec files that actually need it.
+#
+# The `.rspec` file also contains a few flags that are not defaults but that
+# users commonly want.
+#
+# See http://rubydoc.info/gems/rspec-core/RSpec/Core/Configuration
+RSpec.configure do |config|
+  config.include CommandHelper
+  config.include RSpecCommand
+  # rspec-expectations config goes here. You can use an alternate
+  # assertion/expectation library such as wrong or the stdlib/minitest
+  # assertions if you prefer.
+  config.expect_with :rspec do |expectations|
+    # This option will default to `true` in RSpec 4. It makes the `description`
+    # and `failure_message` of custom matchers include text for helper methods
+    # defined using `chain`, e.g.:
+    # be_bigger_than(2).and_smaller_than(4).description
+    #   # => "be bigger than 2 and smaller than 4"
+    # ...rather than:
+    #   # => "be bigger than 2"
+    expectations.include_chain_clauses_in_custom_matcher_descriptions = true
+  end
+
+  # rspec-mocks config goes here. You can use an alternate test double
+  # library (such as bogus or mocha) by changing the `mock_with` option here.
+  config.mock_with :rspec do |mocks|
+    # Prevents you from mocking or stubbing a method that does not exist on
+    # a real object. This is generally recommended, and will default to
+    # `true` in RSpec 4.
+    mocks.verify_partial_doubles = true
+  end
+
+# The settings below are suggested to provide a good initial experience
+# with RSpec, but feel free to customize to your heart's content.
+=begin
+  # These two settings work together to allow you to limit a spec run
+  # to individual examples or groups you care about by tagging them with
+  # `:focus` metadata. When nothing is tagged with `:focus`, all examples
+  # get run.
+  config.filter_run :focus
+  config.run_all_when_everything_filtered = true
+
+  # Limits the available syntax to the non-monkey patched syntax that is recommended.
+  # For more details, see:
+  #   - http://myronmars.to/n/dev-blog/2012/06/rspecs-new-expectation-syntax
+  #   - http://teaisaweso.me/blog/2013/05/27/rspecs-new-message-expectation-syntax/
+  #   - http://myronmars.to/n/dev-blog/2014/05/notable-changes-in-rspec-3#new__config_option_to_disable_rspeccore_monkey_patching
+  config.disable_monkey_patching!
+
+  # This setting enables warnings. It's recommended, but in some cases may
+  # be too noisy due to issues in dependencies.
+  config.warnings = true
+
+  # Many RSpec users commonly either run the entire suite or an individual
+  # file, and it's useful to allow more verbose output when running an
+  # individual spec file.
+  if config.files_to_run.one?
+    # Use the documentation formatter for detailed output,
+    # unless a formatter has already been configured
+    # (e.g. via a command-line flag).
+    config.default_formatter = 'doc'
+  end
+
+  # Print the 10 slowest examples and example groups at the
+  # end of the spec run, to help surface which specs are running
+  # particularly slow.
+  config.profile_examples = 10
+
+  # Run specs in random order to surface order dependencies. If you find an
+  # order dependency and want to debug it, you can fix the order by providing
+  # the seed, which is printed after each run.
+  #     --seed 1234
+  config.order = :random
+
+  # Seed global randomization in this process using the `--seed` CLI option.
+  # Setting this allows you to use `--seed` to deterministically reproduce
+  # test failures related to randomization by passing the same `--seed` value
+  # as the one that triggered the failure.
+  Kernel.srand config.seed
+=end
+end

data/spec/unit/dopi/command/custom_spec.rb

@@ -0,0 +1,58 @@
+require 'spec_helper'
+
+describe Dopi::Command do
+
+  describe '#exec' do
+    it 'should return the exec value if specified' do
+      command = create_command({:plugin => 'custom', :exec => 'echo'})
+      expect(command.exec).to eq('echo')
+    end
+    it 'will raise and error if exec is not specified' do
+      command = create_command({:plugin => 'custom'})
+      expect{command.exec}.to raise_error Dopi::CommandParsingError
+    end
+    it 'will raise and error if exec is not a String' do
+      command = create_command({:plugin => 'custom', :exec => 2})
+      expect{command.exec}.to raise_error Dopi::CommandParsingError
+    end
+  end
+
+  describe '#env' do
+    it 'should return the defaults hash if nothing is specified' do
+      command = create_command({:plugin => 'custom'})
+      expect(command.env).to eq({'DOP_NODE_FQDN' => 'test.example.com'})
+    end
+    it 'should return the specified hash merged with the defaults' do
+      command = create_command({:plugin => 'custom', :env => {'MYVAR' => 'MYVALUE'}})
+      expect(command.env).to eq({'DOP_NODE_FQDN' => 'test.example.com', 'MYVAR' => 'MYVALUE'})
+    end
+    it 'will raise and error if env is not a hash' do
+      command = create_command({:plugin => 'custom', :env => 2})
+      expect{command.env}.to raise_error Dopi::CommandParsingError
+    end
+  end
+
+  describe '#arguments' do
+    it 'should return an empty hash if nothing is specified' do
+      command = create_command({:plugin => 'custom'})
+      expect(command.arguments).to eq("")
+    end
+    it 'should return a correct string if arguments are specified as a String' do
+      command = create_command({:plugin => 'custom', :arguments => "my custom arguments"})
+      expect(command.arguments).to eq("my custom arguments")
+    end
+    it 'should return a correct string if arguments are specified as an Array' do
+      command = create_command({:plugin => 'custom', :arguments => ['my', 'custom', 'arguments']})
+      expect(command.arguments).to eq("my custom arguments")
+    end
+    it 'should return a correct string if arguments are specified as a Hash' do
+      command = create_command({:plugin => 'custom', :arguments => {'my' =>  'custom', 'arguments' => ''}})
+      expect(command.arguments).to eq("my custom arguments ")
+    end
+    it 'will raise and error if arguments is not a String, Array or Hash' do
+      command = create_command({:plugin => 'custom', :arguments => 2})
+      expect{command.arguments}.to raise_error Dopi::CommandParsingError
+    end
+  end
+
+end

data/spec/unit/dopi/command/mco/rpc_spec.rb

@@ -0,0 +1,157 @@
+require 'spec_helper'
+require 'mcollective'
+
+describe Dopi::Command::Mco::Rpc do
+
+  describe '#agent' do
+    it 'should return the name of the agent if it is specified' do
+      command = create_command({:plugin => 'mco/rpc', :agent => 'rpcutil'})
+      expect(command.agent).to eq('rpcutil')
+    end
+    it 'will raise and error if agent is not specified and valid' do
+      command = create_command({:plugin => 'mco/rpc'})
+      expect{command.agent}.to raise_error Dopi::CommandParsingError
+    end
+    it 'will raise and error if agent is not a String' do
+      command = create_command({:plugin => 'mco/rpc', :agent => 2})
+      expect{command.agent}.to raise_error Dopi::CommandParsingError
+    end
+    it 'will raise and error if agent does not exists' do
+      command = create_command({:plugin => 'mco/rpc', :agent => 'nonexistingagent'})
+      expect{command.agent}.to raise_error Dopi::CommandParsingError
+    end
+  end
+
+  describe '#options' do
+    it 'should return the merged options hash if the hash is valid' do
+      command = create_command({:plugin => 'mco/rpc', :options => {:ttl => 300}})
+      expect(command.options[:ttl]).to eq(300)
+    end
+    it 'should return the default options hash if the hash is not defined' do
+      command = create_command({:plugin => 'mco/rpc'})
+      expect(command.options).to eq(command.send(:options_defaults))
+    end
+    it 'will raise and error if options is not a hash' do
+      command = create_command({:plugin => 'mco/rpc', :options => 2})
+      expect{command.options}.to raise_error Dopi::CommandParsingError
+    end
+  end
+
+  describe '#action' do
+    it 'should return the name of the action if it is specified and valid' do
+      command = create_command({
+        :plugin => 'mco/rpc',
+        :agent  => 'rpcutil',
+        :action => 'inventory'
+      })
+      expect(command.action).to eq('inventory')
+    end
+    it 'will raise an error if action is not defined' do
+      command = create_command({
+        :plugin => 'mco/rpc',
+        :agent  => 'nonexistingagent'
+      })
+      expect{command.action}.to raise_error Dopi::CommandParsingError
+    end
+    it 'will raise an error if agent is not valid' do
+      command = create_command({
+        :plugin => 'mco/rpc',
+        :agent  => 'nonexistingagent',
+        :action => 'inventory'
+      })
+      expect{command.action}.to raise_error Dopi::CommandParsingError
+    end
+    it 'will raise an error if action is not valid' do
+      command = create_command({
+        :plugin => 'mco/rpc',
+        :agent  => 'rpcutil',
+        :action => 'nonexixtingaction'
+      })
+      expect{command.action}.to raise_error Dopi::CommandParsingError
+    end
+    it 'will raise and error if action is not a String' do
+      command = create_command({
+        :plugin => 'mco/rpc',
+        :agent  => 'rpcutil',
+        :action => 2
+      })
+      expect{command.action}.to raise_error Dopi::CommandParsingError
+    end
+  end
+
+  describe '#arguments' do
+    it 'should return the arguments hash if it is specified and valid' do
+      command = create_command({
+        :plugin    => 'mco/rpc',
+        :agent     => 'rpcutil',
+        :action    => 'get_fact',
+        :arguments => { :fact => 'osfamily' }
+      })
+      expect(command.arguments).to eq({ :fact => 'osfamily' })
+    end
+    it 'should return an empty hash if "arguments" is not specified and all arguments are optional' do
+      command = create_command({
+        :plugin    => 'mco/rpc',
+        :agent     => 'rpcutil',
+        :action    => 'inventory'
+      })
+      expect(command.arguments).to eq({})
+    end
+    it 'will raise an error if "arguments" is not specified and there are required arguments' do
+      command = create_command({
+        :plugin    => 'mco/rpc',
+        :agent     => 'rpcutil',
+        :action    => 'get_fact'
+      })
+      expect{command.arguments}.to raise_error Dopi::CommandParsingError
+    end
+    it 'will raise an error if an argument key is not valid' do
+      command = create_command({
+        :plugin    => 'mco/rpc',
+        :agent     => 'rpcutil',
+        :action    => 'get_fact',
+        :arguments => { :foo => 'osfamily' }
+      })
+      expect{command.arguments}.to raise_error Dopi::CommandParsingError
+    end
+    it 'will raise an error if an argument value is not valid' do
+      command = create_command({
+        :plugin    => 'mco/rpc',
+        :agent     => 'rpcutil',
+        :action    => 'get_fact',
+        :arguments => { :fact => 'osfamily&&' }
+      })
+      expect{command.arguments}.to raise_error Dopi::CommandParsingError
+    end
+    it 'will raise an error if agent is not valid' do
+      command = create_command({
+        :plugin => 'mco/rpc',
+        :agent  => 'nonexistingagent',
+        :action => 'get_fact'
+      })
+      expect{command.arguments}.to raise_error Dopi::CommandParsingError
+    end
+    it 'will raise an error if action is not valid' do
+      command = create_command({
+        :plugin => 'mco/rpc',
+        :agent  => 'rpcutil',
+        :action => 'nonexistingaction'
+      })
+      expect{command.arguments}.to raise_error Dopi::CommandParsingError
+    end
+    it 'will raise and error if arguments is not a Hash' do
+      command = create_command({
+        :plugin    => 'mco/rpc',
+        :agent     => 'rpcutil',
+        :action    => 'get_fact',
+        :arguments => 'foo'
+      })
+      expect{command.arguments}.to raise_error Dopi::CommandParsingError
+    end
+  end
+
+  describe '#run' do
+    pending
+  end
+
+end

data/spec/unit/dopi/command/ssh/custom_spec.rb

@@ -0,0 +1,30 @@
+require 'spec_helper'
+
+describe Dopi::Command::Ssh do
+
+  describe '#quiet' do
+    pending
+  end
+
+  describe '#port' do
+    it 'should return the default port if nothing is specified' do
+      command = create_command({:plugin => 'ssh/custom'})
+      expect(command.port).to eq('22')
+    end
+    it 'should return the correct port if specified correctly' do
+      command = create_command({:plugin => 'ssh/custom', :port => 42})
+      expect(command.port).to eq('42')
+    end
+    it 'will raise an error if port is not a number' do
+      command = create_command({:plugin => 'ssh/custom', :port => "2"})
+      expect{command.port}.to raise_error Dopi::CommandParsingError
+    end
+    it 'will raise an error if port is not in the valid range' do
+      command = create_command({:plugin => 'ssh/custom', :port => -1})
+      expect{command.port}.to raise_error Dopi::CommandParsingError
+      command = create_command({:plugin => 'ssh/custom', :port => 70000})
+      expect{command.port}.to raise_error Dopi::CommandParsingError
+    end
+  end
+
+end

data/spec/unit/dopi/command/ssh/file_deploy_spec.rb

@@ -0,0 +1,42 @@
+require 'spec_helper'
+
+describe Dopi::Command::Ssh::FileDeploy do
+
+  describe '#file' do
+    it 'should return the file path if specified correctly' do
+      command = create_command({:plugin => 'ssh/file_deploy', :file => '/tmp/foo'})
+      expect(command.file).to eq('/tmp/foo')
+    end
+    it 'will raise an error if the file path is not pecified' do
+      command = create_command({:plugin => 'ssh/file_deploy'})
+      expect{command.file}.to raise_error Dopi::CommandParsingError
+    end
+    it 'will raise an error if file path is not a String' do
+      command = create_command({:plugin => 'ssh/file_deploy', :file => 2})
+      expect{command.file}.to raise_error Dopi::CommandParsingError
+    end
+  end
+
+  describe '#content' do
+    it 'should return an error if nothing is specified' do
+      command = create_command({:plugin => 'ssh/file_deploy'})
+      expect{command.content}.to raise_error Dopi::CommandParsingError
+    end
+    it 'should return the content if specified correctly as a string' do
+      command = create_command({:plugin => 'ssh/file_deploy', :content => 'hello world'})
+      expect(command.content).to eq('hello world')
+    end
+    it 'should return the content if specified correctly as a file' do
+      file = Tempfile.new('secret_file', ENV['HOME'])
+      file.write('hello world')
+      file.close
+      command = create_command({:plugin => 'ssh/file_deploy', :content => {'file' => file.path} })
+      expect(command.content).to eq('hello world')
+      file.delete
+    end
+    it 'will raise an error if content is not a string' do
+      command = create_command({:plugin => 'ssh/file_deploy', :content => 2})
+    end
+  end
+
+end

data/spec/unit/dopi/command/ssh/file_replace_spec.rb

@@ -0,0 +1,35 @@
+require 'spec_helper'
+
+describe Dopi::Command::Ssh::FileReplace do
+
+  describe '#replacement' do
+    it 'should return the replacement value if specified' do
+      command = create_command({:plugin => 'ssh/file_replace', :replacement => 'foo'})
+      expect(command.replacement).to eq('foo')
+    end
+    it 'will raise and error if replacement is not specified' do
+      command = create_command({:plugin => 'ssh/file_replace'})
+      expect{command.replacement}.to raise_error Dopi::CommandParsingError
+    end
+    it 'will raise and error if replacement is not a String' do
+      command = create_command({:plugin => 'ssh/file_replace', :replacement => 2})
+      expect{command.replacement}.to raise_error Dopi::CommandParsingError
+    end
+  end
+
+  describe '#global' do
+    it 'should return true if nothing is specified' do
+      command = create_command({:plugin => 'ssh/file_replace'})
+      expect(command.global).to be true
+    end
+    it 'should return the correct value if specified' do
+      command = create_command({:plugin => 'ssh/file_replace', :global => false})
+      expect(command.global).to be false
+    end
+    it 'will raise and error if global is not a boolean' do
+      command = create_command({:plugin => 'ssh/file_replace', :global => 2})
+      expect{command.global}.to raise_error Dopi::CommandParsingError
+    end
+  end
+
+end

data/spec/unit/dopi/command_parser/credentials_spec.rb

@@ -0,0 +1,53 @@
+require 'spec_helper'
+
+class CredentialsTestKlass
+  include Dopi::CommandParser::Credentials
+  attr_accessor :hash, :step
+  def supported_credential_types() [:username_password] end
+end
+
+describe Dopi::CommandParser::Credentials do
+
+  before :each do
+    @credentials = CredentialsTestKlass.new
+    @fake_credentials  = instance_double('DopCommon::Credentials', :type => :username_password)
+    @fake_credentials4 = instance_double('DopCommon::Credentials', :type => :non_existing_type)
+    plan = instance_double('Dopi::Plan', :credentials => {
+      'test_credentials_01' => @fake_credentials,
+      'test_credentials_02' => @fake_credentials,
+      'test_credentials_04' => @fake_credentials4
+    })
+    @credentials.step = instance_double('Dopi::Step', :plan => plan)
+  end
+
+  describe '#credentials' do
+    it 'returns an empty array if nothing is specified' do
+      @credentials.hash = {}
+      expect(@credentials.credentials).to eq([])
+    end
+    it 'returns an array with one credential if only one is specified' do
+      @credentials.hash = {:credentials => 'test_credentials_01'}
+      expect(@credentials.credentials.length).to eq(1)
+      expect(@credentials.credentials.first).to be @fake_credentials
+    end
+    it 'returns an array with two credentials if two are specified' do
+      @credentials.hash = {:credentials => ['test_credentials_01', 'test_credentials_02']}
+      expect(@credentials.credentials.length).to eq(2)
+      expect(@credentials.credentials.last).to be @fake_credentials
+    end
+    it 'raises an error if the credentials are not valid' do
+      @credentials.hash = {:credentials => ['test_credentials_01', 2]}
+      expect{@credentials.credentials}.to raise_error Dopi::CommandParsingError
+    end
+    it 'raises an error if the credentials do not exist' do
+      @credentials.hash = {:credentials => ['test_credentials_01', 'test_credentials_03']}
+      expect{@credentials.credentials}.to raise_error Dopi::CommandParsingError
+    end
+    it 'raises an error if one of the credential types is not supported' do
+      @credentials.hash = {:credentials => ['test_credentials_01', 'test_credentials_04']}
+      expect{@credentials.credentials}.to raise_error Dopi::CommandParsingError
+    end
+  end
+
+end
+

data/spec/unit/dopi/command_parser/exit_code_spec.rb

@@ -0,0 +1,63 @@
+
+require 'spec_helper'
+
+class ExitCodeTestKlass
+  include Dopi::CommandParser::ExitCode
+  attr_accessor :hash
+end
+
+module ExitCodeDefaults
+  def expect_exit_codes_defaults
+    [1,2,3]
+  end
+end
+
+describe Dopi::CommandParser::ExitCode do
+
+  describe '#expect_exit_codes' do
+    subject do
+      exit_code_parser = ExitCodeTestKlass.new
+      exit_code_parser.hash = hash
+      exit_code_parser.expect_exit_codes
+    end
+
+    context 'Nothing is specified' do
+      let(:hash){nil}
+      it{ is_expected.to eq(0) }
+    end
+    context 'A number is specified' do
+      let(:hash){{:expect_exit_codes => 3}}
+      it{ is_expected.to eq(3) }
+    end
+    context 'An Array is specified' do
+      let(:hash){{:expect_exit_codes => [0, 1, 3]}}
+      it{ is_expected.to eq([0, 1, 3]) }
+    end
+    context 'The :all keyword is specified' do
+      let(:hash){{:expect_exit_codes => 'all'}}
+      it{ is_expected.to eq('all') }
+    end
+    context 'The value is an invalid string' do
+      let(:hash){{:expect_exit_codes => 'foo'}}
+      it{ expect{subject}.to raise_error Dopi::CommandParsingError }
+    end
+    context 'The Array contains a String' do
+      let(:hash){{:expect_exit_codes => [1, 2, 'foo']}}
+      it{ expect{subject}.to raise_error Dopi::CommandParsingError }
+    end
+    context 'A plugin defaults method is specified' do
+      subject do
+        exit_code_parser = ExitCodeTestKlass.new
+        exit_code_parser.extend(ExitCodeDefaults)
+        exit_code_parser.expect_exit_codes
+      end
+      it{ is_expected.to eq([1,2,3])}
+    end
+  end
+
+  describe 'check_exit_code' do
+    pending
+  end
+
+end
+

data/spec/unit/dopi/command_parser/output_spec.rb

@@ -0,0 +1,129 @@
+require 'spec_helper'
+
+class OutputTestKlass
+  include Dopi::CommandParser::Output
+  attr_accessor :hash
+  def log(s,m); end
+end
+
+describe Dopi::CommandParser::Output do
+
+  before :each do
+    @output_parser = OutputTestKlass.new
+  end
+
+  describe 'parse_output' do
+    it 'Returns an empty hash if nothing is defined' do
+      expect(@output_parser.parse_output).to eq({})
+    end
+    it 'Returns the correct hash if one is defined' do
+      @output_parser.hash = {:parse_output => {:error => [], :warning => []}}
+      expect(@output_parser.parse_output).to eq({:error => [], :warning => []})
+    end
+    it 'Raises an exeption if the value is not a hash' do
+      @output_parser.hash = {:parse_output => 2}
+      expect{@output_parser.parse_output}.to raise_error Dopi::CommandParsingError
+    end
+  end
+
+  describe 'error_patterns' do
+     it 'Returns an empty array if nothing is defined' do
+      expect(@output_parser.error_patterns).to eq([])
+    end
+    it 'Returns the correct array if it is specified' do
+      @output_parser.hash = {:parse_output => {:error => [ 'foo', '^bar$' ]}}
+      expect(@output_parser.error_patterns).to eq([ 'foo', '^bar$' ])
+    end
+    it 'Raises an exeption if the value is not an array' do
+      @output_parser.hash = {:parse_output => {:error => 2}}
+      expect{@output_parser.error_patterns}.to raise_error Dopi::CommandParsingError
+    end
+    it 'Raises an exception if the pattern is not a Regular Expression' do
+      @output_parser.hash = {:parse_output => {:error => ['][']}}
+      expect{@output_parser.error_patterns}.to raise_error Dopi::CommandParsingError
+    end
+  end
+
+  describe 'warning_patterns' do
+     it 'Returns an empty array if nothing is defined' do
+      expect(@output_parser.warning_patterns).to eq([])
+    end
+    it 'Returns the correct array if it is specified' do
+      @output_parser.hash = {:parse_output => {:warning => [ 'foo', '^bar$' ]}}
+      expect(@output_parser.warning_patterns).to eq([ 'foo', '^bar$' ])
+    end
+    it 'Raises an exeption if the value is not an array' do
+      @output_parser.hash = {:parse_output => {:warning => 2}}
+      expect{@output_parser.warning_patterns}.to raise_error Dopi::CommandParsingError
+    end
+  end
+
+  describe 'fail_on_warning' do
+     it 'Returns false if nothing is defined' do
+      expect(@output_parser.fail_on_warning).to be false
+    end
+    it 'Returns true if specified' do
+      @output_parser.hash = {:fail_on_warning => true}
+      expect(@output_parser.fail_on_warning).to be true
+    end
+    it 'Returns false if specified' do
+      @output_parser.hash = {:fail_on_warning => false}
+      expect(@output_parser.fail_on_warning).to be false
+    end
+    it 'Raises an exeption if the value is not a boolean' do
+      @output_parser.hash = {:fail_on_warning => 2}
+      expect{@output_parser.fail_on_warning}.to raise_error Dopi::CommandParsingError
+    end
+  end
+
+  describe 'check_output' do
+    it 'Returns true if no error is detected' do
+      raw_output = <<-OUTPUT
+        Foo
+        Bar
+      OUTPUT
+      @output_parser.hash = {:parse_output => {:error => [ 'Error:' ]}}
+      expect(@output_parser.check_output(raw_output)).to be true
+    end
+    it 'Returns false if an error is detected' do
+      raw_output = <<-OUTPUT
+        Foo
+        Error: this is a fake error
+        Bar
+      OUTPUT
+      @output_parser.hash = {:parse_output => {:error => [ 'Error:' ]}}
+      expect(@output_parser.check_output(raw_output)).to be false
+    end
+    it 'Returns true if no warning is detected' do
+      raw_output = <<-OUTPUT
+        Foo
+        Bar
+      OUTPUT
+      @output_parser.hash = {:parse_output => {:warning => [ 'Warning:' ]}}
+      expect(@output_parser.check_output(raw_output)).to be true
+    end
+    it 'Returns true if an warning is detected but fail_on_warning is not true' do
+      raw_output = <<-OUTPUT
+        Foo
+        Warning: this is a fake error
+        Bar
+      OUTPUT
+      @output_parser.hash = {:parse_output => {:warning => [ 'Warning:' ]}}
+      expect(@output_parser.check_output(raw_output)).to be true
+    end
+    it 'Returns true if an warning is detected but fail_on_warning is not true' do
+      raw_output = <<-OUTPUT
+        Foo
+        Warning: this is a fake error
+        Bar
+      OUTPUT
+      @output_parser.hash = {
+        :fail_on_warning => true,
+        :parse_output => {:warning => [ 'Warning:' ]}
+      }
+      expect(@output_parser.check_output(raw_output)).to be false
+    end
+  end
+
+end
+