Fix X display handling in rspec test suite

* Hide any available X display from test suite context;
* Ensure the manager never connect or block.
This commit is contained in:
Thibault Jouan 2015-04-10 10:16:30 +00:00
parent 59e9562279
commit c9e6d1807b
3 changed files with 9 additions and 2 deletions

View File

@ -10,4 +10,9 @@ RSpec.configure do |config|
end
config.disable_monkey_patching!
config.before :all do
# Ensure current X display is not available from rspec test suite.
ENV.delete 'DISPLAY'
end
end

View File

@ -15,8 +15,7 @@ module Uh
described_class.run arguments, stdout: stdout, stderr: stderr
end
# FIXME: remove this hack we currently need to prevent the Runner from
# blocking.
# Prevent Runner from connecting a Manager and blocking.
before { allow(Runner).to receive :run }
it 'builds a new CLI with given arguments' do

View File

@ -14,6 +14,9 @@ module Uh
describe '#connect' do
let(:block) { proc { } }
# Prevent Manager to open a real display.
before { allow(display).to receive :open }
it 'opens the display' do
expect(manager.display).to receive :open
manager.connect