mirror of
https://github.com/inspec/inspec
synced 2024-11-27 07:00:39 +00:00
2bbcdbde9b
* Remove deprecated yumrepo. (#3435) * Remove deprecations for cli `--format` and metadata.rb (#3452) * Remove deprecated database_helpers stderr/stdout methods. Update deprecation text for processes/apache. * Remove deprecations for `--format` and metadata.rb Remove deprecated `format` code. Remove deprecated code test and change json-config format test to use reporter. Remove deprecated metadata.rb code Remove deprecation notice for old supports syntax. Deprecate metadata.rb from source_reader Remove rubocop disables as they are no longer required for this code block. Remove deprecated legacy metadata.rb mock profiles. Remove deprecated metadata.rb profile tests. Remove deprecated yumrepo test. * Allow inspec-3.0 branch to be tested. * Allow appveyor to test inspec-3.0 branch * Change runner tests to use reporter rather than format. Remove deprecated `supports: linux` tests. * Remove skip from inherited profiles from showing up in reporting (breaking change) (#3332) * Skip loading dependency profiles if they are unsupported on the current platform. Skip loading dependencies if they are unsupported on the current platform. Wrap our log and next in a conditional checking if the platform is supported. Change a `if !` into a `unless` Check if the backend is a Train Mock Connection and if so say that the profile does support the platform. While iterating through tests being loaded skip when the platform is unsupported. We now log a WARN when a profile is skipped due to unsupported platform, so lets check that. Modified existing test to log that there are 0 skipped tests, instead of 2. Add functional test that loads profile-support-skip with a json reporter to check that our controls are not loaded and that stderr contains our warning. * Rather than iterating through each test return before recursion if the platform is unsupported. * Resolve tests using a supported platform different from testing platform Add a control to `test/unit/mock/profiles/complete-profile` that would work on any OS with a Internet connection. This allows the profile to execute on any OS with success. `filesystem_spec.rb` was a control that would only work on Linux and some BSD's. We want profile tests to consistently work across development and testing platforms, and not get 'skipped' in some cases. Travis-CI tests on Linux, Inspec Dev team uses Linux and MacOS, Appveyor tests on Windows Also Updated `file_provider_test.rb` for `complete-profile` content changes. If you `MockLoader.load_profile` on a unsupported platform you might not hit the usual skip. Lets handle situations where the tests array in Profile#load_checks_params could be nil. * Use safe navigation rather than checking if tests is nil. Update tests to point to unsupported_inspec and account for WARN changes. Make unsupported_inspec profile support os-family 'unsupported_inspec' * Fix skip bug when using include/require controls. (#3487) * Fix skip bug when using include/require controls. * fix test and feedback. * Remove need for UUID detection for Automate report (#3507) * Add json metadata for skipped profiles (#3495) * Add skip metadata to json reports * Unify skip messages. * Update with status field. * Add testing. * Fix tests. * lint * Add skip exit codes for profile skips. * Update website for 3.0 launch Add `plugins` to sidebar. Change 2.0 -> 3.0 in slim files. Update 3.0 features list. * Fix comments * Update float to numeric. * Change Float to numeric. * updated feature list and impact doc * Change "What's new in InSpec 3.0" -> "Announcing InSpec 3.0" * Bump VERSION to 3.0.0 (#3511) * Remove 3.0 testing checks. * Fix azure link.
136 lines
5.3 KiB
Ruby
136 lines
5.3 KiB
Ruby
# encoding: utf-8
|
|
# copyright: 2017, Chef Software Inc.
|
|
|
|
require 'helper'
|
|
|
|
describe Inspec::Runner do
|
|
describe '#load_attributes' do
|
|
let(:runner) { Inspec::Runner.new({ command_runner: :generic }) }
|
|
|
|
before do
|
|
Inspec::Runner.any_instance.stubs(:validate_attributes_file_readability!)
|
|
end
|
|
|
|
describe 'confirm reporter defaults to cli' do
|
|
it 'defaults to cli when format and reporter not set' do
|
|
opts = { command_runner: :generic, backend_cache: true }
|
|
runner = Inspec::Runner.new(opts)
|
|
config = runner.instance_variable_get(:"@conf")
|
|
expected = { 'cli' => { 'stdout' => true } }
|
|
config['reporter'].must_equal expected
|
|
end
|
|
|
|
it 'does not default when format is set' do
|
|
opts = { command_runner: :generic, backend_cache: true, 'reporter' => ['json'] }
|
|
runner = Inspec::Runner.new(opts)
|
|
config = runner.instance_variable_get(:"@conf")
|
|
expected = { 'json' => { 'stdout' => true } }
|
|
config['reporter'].must_equal expected
|
|
end
|
|
|
|
it 'delets format if set to a rspec format' do
|
|
opts = { command_runner: :generic, backend_cache: true, 'reporter' => ['progress'] }
|
|
runner = Inspec::Runner.new(opts)
|
|
config = runner.instance_variable_get(:"@conf")
|
|
config['reporter'].must_equal Hash.new
|
|
end
|
|
end
|
|
|
|
describe 'when backend caching is enabled' do
|
|
it 'returns a backend with caching' do
|
|
opts = { command_runner: :generic, backend_cache: true }
|
|
runner = Inspec::Runner.new(opts)
|
|
backend = runner.instance_variable_get(:@backend)
|
|
backend.backend.cache_enabled?(:command).must_equal true
|
|
end
|
|
end
|
|
|
|
describe 'when backend caching is disabled' do
|
|
it 'returns a backend without caching' do
|
|
opts = { command_runner: :generic, backend_cache: false }
|
|
runner = Inspec::Runner.new(opts)
|
|
backend = runner.instance_variable_get(:@backend)
|
|
backend.backend.cache_enabled?(:command).must_equal false
|
|
end
|
|
|
|
it 'returns a backend without caching as default' do
|
|
backend = runner.instance_variable_get(:@backend)
|
|
backend.backend.cache_enabled?(:command).must_equal false
|
|
end
|
|
end
|
|
|
|
describe 'when no attrs are specified' do
|
|
it 'returns an empty hash' do
|
|
options = {}
|
|
runner.load_attributes(options).must_equal({})
|
|
end
|
|
end
|
|
|
|
describe 'when an attr is provided and does not resolve' do
|
|
it 'raises an exception' do
|
|
options = { attrs: ['nope.jpg'] }
|
|
Inspec::SecretsBackend.expects(:resolve).with('nope.jpg').returns(nil)
|
|
proc { runner.load_attributes(options) }.must_raise Inspec::Exceptions::SecretsBackendNotFound
|
|
end
|
|
end
|
|
|
|
describe 'when an attr is provided and has no attributes' do
|
|
it 'returns an empty hash' do
|
|
secrets = mock
|
|
secrets.stubs(:attributes).returns(nil)
|
|
options = { attrs: ['empty.yaml'] }
|
|
Inspec::SecretsBackend.expects(:resolve).with('empty.yaml').returns(secrets)
|
|
runner.load_attributes(options).must_equal({})
|
|
end
|
|
end
|
|
|
|
describe 'when an attr is provided and has attributes' do
|
|
it 'returns a hash containing the attributes' do
|
|
options = { attrs: ['file1.yaml'] }
|
|
attributes = { foo: 'bar' }
|
|
secrets = mock
|
|
secrets.stubs(:attributes).returns(attributes)
|
|
Inspec::SecretsBackend.expects(:resolve).with('file1.yaml').returns(secrets)
|
|
runner.load_attributes(options).must_equal(attributes)
|
|
end
|
|
end
|
|
|
|
describe 'when multiple attrs are provided and one fails' do
|
|
it 'raises an exception' do
|
|
options = { attrs: ['file1.yaml', 'file2.yaml'] }
|
|
secrets = mock
|
|
secrets.stubs(:attributes).returns(nil)
|
|
Inspec::SecretsBackend.expects(:resolve).with('file1.yaml').returns(secrets)
|
|
Inspec::SecretsBackend.expects(:resolve).with('file2.yaml').returns(nil)
|
|
proc { runner.load_attributes(options) }.must_raise Inspec::Exceptions::SecretsBackendNotFound
|
|
end
|
|
end
|
|
|
|
describe 'when multiple attrs are provided and one has no attributes' do
|
|
it 'returns a hash containing the attributes from the valid files' do
|
|
options = { attrs: ['file1.yaml', 'file2.yaml'] }
|
|
attributes = { foo: 'bar' }
|
|
secrets1 = mock
|
|
secrets1.stubs(:attributes).returns(nil)
|
|
secrets2 = mock
|
|
secrets2.stubs(:attributes).returns(attributes)
|
|
Inspec::SecretsBackend.expects(:resolve).with('file1.yaml').returns(secrets1)
|
|
Inspec::SecretsBackend.expects(:resolve).with('file2.yaml').returns(secrets2)
|
|
runner.load_attributes(options).must_equal(attributes)
|
|
end
|
|
end
|
|
|
|
describe 'when multiple attrs are provided and all have attributes' do
|
|
it 'returns a hash containing all the attributes' do
|
|
options = { attrs: ['file1.yaml', 'file2.yaml'] }
|
|
secrets1 = mock
|
|
secrets1.stubs(:attributes).returns({ key1: 'value1' })
|
|
secrets2 = mock
|
|
secrets2.stubs(:attributes).returns({ key2: 'value2' })
|
|
Inspec::SecretsBackend.expects(:resolve).with('file1.yaml').returns(secrets1)
|
|
Inspec::SecretsBackend.expects(:resolve).with('file2.yaml').returns(secrets2)
|
|
runner.load_attributes(options).must_equal({ key1: 'value1', key2: 'value2' })
|
|
end
|
|
end
|
|
end
|
|
end
|