diff --git a/spec/acceptance/report_spec.rb b/spec/acceptance/report_spec.rb index ec1a865b9..33816885a 100644 --- a/spec/acceptance/report_spec.rb +++ b/spec/acceptance/report_spec.rb @@ -9,7 +9,7 @@ before(:all) do File.open(init_pp, 'w') do |f| f.puts <<~EOS - class report {} + class report EOS end end @@ -18,24 +18,24 @@ class report {} include_context 'with a fake TTY' # Tests writing reports to a file describe command('pdk validate puppet manifests/init.pp --format=text:report.txt') do - its(:exit_status) { is_expected.to eq(0) } + its(:exit_status) { is_expected.to eq(1) } its(:stdout) { is_expected.to have_no_output } its(:stderr) { is_expected.to match(/Checking Puppet manifest syntax/i) } its(:stderr) { is_expected.to match(/Checking Puppet manifest style/i) } describe file('report.txt') do it { is_expected.to exist } - # pdk (WARNING): puppet-lint: class not documented (manifests/init.pp:1:1) - its(:content) { is_expected.to match(/\(warning\):.*class not documented.*\(#{Regexp.escape(init_pp)}.*\)/i) } + # pdk (ERROR): puppet-syntax: Could not parse for environment production: Syntax error at end of input (manifests/init.pp) + its(:content) { is_expected.to match(/\(ERROR\):.*Could not parse for environment production.*\(#{Regexp.escape(init_pp)}.*\)/i) } end end # Tests writing reports to stdout doesn't actually write a file named stdout describe command('pdk validate puppet manifests/init.pp --format=text:stdout') do - its(:exit_status) { is_expected.to eq(0) } + its(:exit_status) { is_expected.to eq(1) } its(:stderr) { is_expected.to match(/Checking Puppet manifest syntax/i) } its(:stderr) { is_expected.to match(/Checking Puppet manifest style/i) } - its(:stdout) { is_expected.to match(/\(warning\):.*class not documented.*\(#{Regexp.escape(init_pp)}.*\)/i) } + its(:stdout) { is_expected.to match(/\(ERROR\):.*Could not parse for environment production.*\(#{Regexp.escape(init_pp)}.*\)/i) } describe file('stdout') do it { is_expected.not_to exist } @@ -44,7 +44,7 @@ class report {} # Tests writing reports to stderr doesn't actually write a file named stderr describe command('pdk validate puppet manifests/init.pp --format=text:stderr') do - its(:exit_status) { is_expected.to eq(0) } + its(:exit_status) { is_expected.to eq(1) } its(:stdout) { is_expected.to have_no_output } its(:stderr) { is_expected.to match(/Checking Puppet manifest syntax/i) } its(:stderr) { is_expected.to match(/Checking Puppet manifest style/i) } @@ -52,7 +52,7 @@ class report {} its(:stderr) do # Due to spinners writing at arbitrary cursor locations, we can't depend on the text # being at a the beginning of a line. - is_expected.to match(/\(warning\):.*class not documented.*\(#{Regexp.escape(init_pp)}.*\)/i) + is_expected.to match(/\(ERROR\):.*Could not parse for environment production.*\(#{Regexp.escape(init_pp)}.*\)/i) end describe file('stderr') do @@ -63,10 +63,10 @@ class report {} context 'when not run interactively' do describe command('pdk validate puppet manifests/init.pp') do - its(:exit_status) { is_expected.to eq(0) } + its(:exit_status) { is_expected.to eq(1) } its(:stderr) { is_expected.to match(/using ruby \d+\.\d+\.\d+/i) } its(:stderr) { is_expected.to match(/using puppet \d+\.\d+\.\d+/i) } - its(:stdout) { is_expected.to match(/\(warning\):.*class not documented.*\(#{Regexp.escape(init_pp)}.*\)/i) } + its(:stdout) { is_expected.to match(/\(ERROR\):.*Could not parse for environment production.*\(#{Regexp.escape(init_pp)}.*\)/i) } end end end diff --git a/spec/acceptance/validate_puppet_spec.rb b/spec/acceptance/validate_puppet_spec.rb index 27452cc73..4df2d9f8d 100644 --- a/spec/acceptance/validate_puppet_spec.rb +++ b/spec/acceptance/validate_puppet_spec.rb @@ -81,7 +81,7 @@ class foo { end describe command('pdk validate puppet --format text:stdout --format junit:report.xml') do - its(:exit_status) { is_expected.to eq(0) } + its(:exit_status) { is_expected.to eq(1) } its(:stderr) { is_expected.not_to match(epp_spinner_text) } its(:stderr) { is_expected.to match(syntax_spinner_text) } its(:stderr) { is_expected.to match(lint_spinner_text) } @@ -180,7 +180,10 @@ class foo { context 'with a parsable file and some style warnings' do before(:all) do File.open(init_pp, 'w') do |f| - f.puts 'class foo {}' + f.puts <<-EOS.gsub(/^ {10}/, '') + # pdk_in_gemfile + class pdk_in_gemfile {} + EOS end end @@ -189,8 +192,8 @@ class foo { end describe command('pdk validate puppet --format text:stdout --format junit:report.xml') do - its(:exit_status) { is_expected.to eq(0) } - its(:stdout) { is_expected.to match(/\(warning\):.*class not documented.*\(#{Regexp.escape(init_pp)}.+\)/i) } + its(:exit_status) { is_expected.to eq(1) } + its(:stdout) { is_expected.to match(/\(warning\):.*indent should be 0 chars and is 2.*\(#{Regexp.escape(init_pp)}.+\)/i) } its(:stderr) { is_expected.not_to match(epp_spinner_text) } its(:stderr) { is_expected.to match(syntax_spinner_text) } its(:stderr) { is_expected.to match(lint_spinner_text) } @@ -221,7 +224,7 @@ class foo { its(:content) do is_expected.to have_junit_testcase.in_testsuite('puppet-lint').with_attributes( - 'classname' => 'puppet-lint.documentation', + 'classname' => 'puppet-lint.strict_indent', 'name' => a_string_starting_with(init_pp) ).that_failed end @@ -289,11 +292,11 @@ class foo { end end - context 'with a parsable file and some style errors' do + context 'with a parsable file and some errors' do before(:all) do File.open(example_pp, 'w') do |f| f.puts '# some documentation' - f.puts 'class foo::bar {}' + f.puts 'class foo::bar' end end @@ -303,7 +306,7 @@ class foo { describe command('pdk validate puppet --format text:stdout --format junit:report.xml') do its(:exit_status) { is_expected.not_to eq(0) } - its(:stdout) { is_expected.to match(/autoload module layout.*\(#{Regexp.escape(example_pp)}.+\)/i) } + its(:stdout) { is_expected.to match(/Syntax error at end of input.*\(#{Regexp.escape(example_pp)}.+\)/i) } its(:stderr) { is_expected.not_to match(epp_spinner_text) } its(:stderr) { is_expected.to match(syntax_spinner_text) } its(:stderr) { is_expected.to match(lint_spinner_text) } @@ -312,15 +315,15 @@ class foo { its(:content) { is_expected.to contain_valid_junit_xml } its(:content) do - is_expected.to have_junit_testsuite('puppet-lint').with_attributes( + is_expected.to have_junit_testsuite('puppet-syntax').with_attributes( 'failures' => eq(1), 'tests' => eq(1) ) end its(:content) do - is_expected.to have_junit_testcase.in_testsuite('puppet-lint').with_attributes( - 'classname' => 'puppet-lint.autoloader_layout', + is_expected.to have_junit_testcase.in_testsuite('puppet-syntax').with_attributes( + 'classname' => 'puppet-syntax', 'name' => a_string_starting_with(example_pp) ).that_failed end @@ -388,7 +391,7 @@ class foo { before(:all) do FileUtils.mkdir_p(another_problem_dir) File.open(another_problem_pp, 'w') do |f| - f.puts 'class foo::bar::whoops {}' + f.puts 'class foo::bar::whoops' end end @@ -408,15 +411,15 @@ class foo { its(:content) { is_expected.to contain_valid_junit_xml } its(:content) do - is_expected.to have_junit_testsuite('puppet-lint').with_attributes( - 'failures' => eq(2), - 'tests' => eq(2) + is_expected.to have_junit_testsuite('puppet-syntax').with_attributes( + 'failures' => eq(1), + 'tests' => eq(1) ) end its(:content) do - is_expected.to have_junit_testcase.in_testsuite('puppet-lint').with_attributes( - 'classname' => a_string_starting_with('puppet-lint'), + is_expected.to have_junit_testcase.in_testsuite('puppet-syntax').with_attributes( + 'classname' => a_string_starting_with('puppet-syntax'), 'name' => a_string_starting_with(another_problem_pp) ).that_failed end