Skip to content

Commit

Permalink
(FM-1911) Fixup passenger system test
Browse files Browse the repository at this point in the history
Previously we were running a command via sudo, which requires a TTY. The
testing framework does not allocate a TTY by default, passing the
`:pty => true` option corrects this.
  • Loading branch information
justinstoller committed Sep 23, 2014
1 parent 5ef11d7 commit a53b9ed
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion spec/acceptance/mod_passenger_spec.rb
Original file line number Diff line number Diff line change
Expand Up @@ -262,7 +262,7 @@ class { 'apache::mod::passenger':
end

it 'should output status via passenger-memory-stats' do
shell("sudo /usr/bin/passenger-memory-stats") do |r|
shell("sudo /usr/bin/passenger-memory-stats", :pty => true) do |r|
expect(r.stdout).to match(/Apache processes/)
expect(r.stdout).to match(/Nginx processes/)
expect(r.stdout).to match(/Passenger processes/)
Expand Down

0 comments on commit a53b9ed

Please sign in to comment.