Skip to content

Commit

Permalink
Merge pull request #1388 from puppetlabs/CAT-2026
Browse files Browse the repository at this point in the history
(CAT-2026) Update how `pdk test unit --list` detects an error
  • Loading branch information
LukasAud authored Aug 29, 2024
2 parents 9041f64 + e1c7350 commit ace2f29
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 8 deletions.
6 changes: 3 additions & 3 deletions lib/pdk/tests/unit.rb
Original file line number Diff line number Diff line change
Expand Up @@ -246,10 +246,10 @@ def self.list(options = {})
raise PDK::CLI::FatalError, format('Failed to find valid JSON in output from rspec: %{output}', output: output[:stdout]) unless rspec_json

if rspec_json['examples'].empty?
rspec_message = rspec_json['messages'][0]
return [] if rspec_message == 'No examples found.'
return [] if rspec_json['messages'][0] == 'No examples found.'
return [] if rspec_json['messages'].include?("\nAll examples were filtered out")

raise PDK::CLI::FatalError, format('Unable to enumerate examples. rspec reported: %{message}', message: rspec_message)
raise PDK::CLI::FatalError, format('Unable to enumerate examples. rspec reported: %{message}', message: rspec_json['messages'])
else
examples = []
rspec_json['examples'].each do |example|
Expand Down
8 changes: 3 additions & 5 deletions spec/acceptance/test_unit_spec.rb
Original file line number Diff line number Diff line change
Expand Up @@ -47,11 +47,9 @@
require 'spec_helper'
RSpec.describe 'passing test' do
on_supported_os(:facterversion => '2.4.6').each do |os, facts|
context "On OS \#{os}" do
it 'should pass' do
expect(true).to eq(true)
end
context 'On OS' do
it 'should pass' do
expect(true).to eq(true)
end
end
end
Expand Down

0 comments on commit ace2f29

Please sign in to comment.