Skip to content

[FIX] Errors parsing $DISPLAY #2363

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
effigies merged 14 commits into from
Jan 19, 2018
Merged

[FIX] Errors parsing $DISPLAY #2363

Changes from 1 commit
Commits
Show all changes
14 commits
Select commit Hold shift + click to select a range
1a35f44
[FIX] Errors parsing ``$DISPLAY``
oesteban Jan 9, 2018
e1ef2e5
add test
oesteban Jan 9, 2018
6a9bc46
Update CHANGES
oesteban Jan 9, 2018
2383177
fix test name
oesteban Jan 10, 2018
fbac4ee
fix implementation of get_display()
oesteban Jan 10, 2018
4dc5896
use mokeypatch.setenv
oesteban Jan 10, 2018
5de504a
fix test for Travis and Circle
oesteban Jan 10, 2018
1286d1c
Merge remote-tracking branch 'upstream/master' into fix/displaynumber
oesteban Jan 18, 2018
22c1a27
update travis
oesteban Jan 18, 2018
ba25260
Merge upstream/master
effigies Jan 18, 2018
28e2cd6
Merge remote-tracking branch 'upstream/master' into fix/displaynumber
oesteban Jan 19, 2018
9cc40da
revise tests
oesteban Jan 19, 2018
7d80634
roll back travis setting
oesteban Jan 19, 2018
603ae77
Merge upstream/master [skip ci]
effigies Jan 19, 2018
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
revise tests
  • Loading branch information
@oesteban
oesteban committed Jan 19, 2018
commit 9cc40da4b97730fff6db068be54ea854d210ba0b
50 changes: 36 additions & 14 deletions nipype/utils/tests/test_config.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -28,20 +28,15 @@
spec=['vdisplay_num', 'start', 'stop'], vdisplay_num=2010)


@pytest.mark.parametrize('disp_var', [':0', 'localhost:0', 'localhost:0.1'])
def test_display_parse(monkeypatch, disp_var):
@pytest.mark.parametrize('dispvar', [':12', 'localhost:12', 'localhost:12.1'])
def test_display_parse(monkeypatch, dispvar):
"""Check that when $DISPLAY is defined, the display is correctly parsed"""
config._display = None
dispstr = ':0'
old_disp = os.getenv('DISPLAY')
os.environ['DISPLAY'] = disp_var
assert config.get_display() == dispstr
config._config.remove_option('execution', 'display_variable')
monkeypatch.setenv('DISPLAY', dispvar)
assert config.get_display() == ':12'
# Test that it was correctly cached
assert config.get_display() == dispstr
if old_disp is not None:
os.environ['DISPLAY'] = old_disp
else:
del os.environ['DISPLAY']
assert config.get_display() == ':12'


@pytest.mark.parametrize('dispnum', range(5))
Expand Down Expand Up @@ -88,10 +83,17 @@ def test_display_noconfig_nosystem_patched(monkeypatch):
config._config.remove_option('execution', 'display_variable')
monkeypatch.delitem(os.environ, 'DISPLAY', raising=False)
monkeypatch.setitem(sys.modules, 'xvfbwrapper', xvfbpatch)
monkeypatch.setattr(sys, 'platform', value='linux')
assert config.get_display() == ":2010"
# Test that it was correctly cached
assert config.get_display() == ':2010'

# Check that raises in Mac
config._display = None
monkeypatch.setattr(sys, 'platform', value='darwin')
with pytest.raises(RuntimeError):
config.get_display()


def test_display_empty_patched(monkeypatch):
"""
Expand All @@ -103,10 +105,16 @@ def test_display_empty_patched(monkeypatch):
config._config.remove_option('execution', 'display_variable')
monkeypatch.setenv('DISPLAY', '')
monkeypatch.setitem(sys.modules, 'xvfbwrapper', xvfbpatch)
monkeypatch.setattr(sys, 'platform', value='linux')
assert config.get_display() == ':2010'
# Test that it was correctly cached
assert config.get_display() == ':2010'

# Check that raises in Mac
config._display = None
monkeypatch.setattr(sys, 'platform', value='darwin')
with pytest.raises(RuntimeError):
config.get_display()

def test_display_noconfig_nosystem_patched_oldxvfbwrapper(monkeypatch):
"""
Expand All @@ -118,10 +126,16 @@ def test_display_noconfig_nosystem_patched_oldxvfbwrapper(monkeypatch):
config._config.remove_option('execution', 'display_variable')
monkeypatch.delitem(os.environ, 'DISPLAY', raising=False)
monkeypatch.setitem(sys.modules, 'xvfbwrapper', xvfbpatch_old)
monkeypatch.setattr(sys, 'platform', value='linux')
assert config.get_display() == ":2010"
# Test that it was correctly cached
assert config.get_display() == ':2010'

# Check that raises in Mac
config._display = None
monkeypatch.setattr(sys, 'platform', value='darwin')
with pytest.raises(RuntimeError):
config.get_display()

def test_display_empty_patched_oldxvfbwrapper(monkeypatch):
"""
Expand All @@ -133,10 +147,16 @@ def test_display_empty_patched_oldxvfbwrapper(monkeypatch):
config._config.remove_option('execution', 'display_variable')
monkeypatch.setenv('DISPLAY', '')
monkeypatch.setitem(sys.modules, 'xvfbwrapper', xvfbpatch_old)
monkeypatch.setattr(sys, 'platform', value='linux')
assert config.get_display() == ':2010'
# Test that it was correctly cached
assert config.get_display() == ':2010'

# Check that raises in Mac
config._display = None
monkeypatch.setattr(sys, 'platform', value='darwin')
with pytest.raises(RuntimeError):
config.get_display()

def test_display_noconfig_nosystem_notinstalled(monkeypatch):
"""
Expand All @@ -146,7 +166,7 @@ def test_display_noconfig_nosystem_notinstalled(monkeypatch):
config._display = None
if config.has_option('execution', 'display_variable'):
config._config.remove_option('execution', 'display_variable')
monkeypatch.delitem(os.environ, 'DISPLAY', raising=False)
monkeypatch.delenv('DISPLAY', raising=False)
monkeypatch.setitem(sys.modules, 'xvfbwrapper', None)
with pytest.raises(RuntimeError):
config.get_display()
Expand All @@ -167,6 +187,7 @@ def test_display_empty_notinstalled(monkeypatch):


@pytest.mark.skipif(not has_Xvfb, reason='xvfbwrapper not installed')
@pytest.mark.skipif('darwin' in sys.platform, reason='macosx requires root for Xvfb')
def test_display_noconfig_nosystem_installed(monkeypatch):
"""
Check that actually uses xvfbwrapper when installed (not mocked)
Expand All @@ -175,14 +196,15 @@ def test_display_noconfig_nosystem_installed(monkeypatch):
config._display = None
if config.has_option('execution', 'display_variable'):
config._config.remove_option('execution', 'display_variable')
monkeypatch.delitem(os.environ, 'DISPLAY', raising=False)
monkeypatch.delenv('DISPLAY', raising=False)
newdisp = config.get_display()
assert int(newdisp.split(':')[-1]) > 1000
# Test that it was correctly cached
assert config.get_display() == newdisp


@pytest.mark.skipif(not has_Xvfb, reason='xvfbwrapper not installed')
@pytest.mark.skipif('darwin' in sys.platform, reason='macosx requires root for Xvfb')
def test_display_empty_installed(monkeypatch):
"""
Check that actually uses xvfbwrapper when installed (not mocked)
Expand All @@ -207,7 +229,7 @@ def test_display_empty_macosx(monkeypatch):
config._display = None
if config.has_option('execution', 'display_variable'):
config._config.remove_option('execution', 'display_variable')
monkeypatch.delitem(os.environ, 'DISPLAY', '')
monkeypatch.delenv('DISPLAY', '')

monkeypatch.setattr(sys, 'platform', 'darwin')
with pytest.raises(RuntimeError):
Expand Down