Add functional marker

tests/conftest.py

@@ -2,6 +2,13 @@ import pytest
 from mock import Mock
 
 
+def pytest_addoption(parser):
+    """Adds `--run-without-docker` argument."""
+    group = parser.getgroup("thefuck")
+    group.addoption('--enable-functional', action="store_true", default=False,
+                    help="Enable functional tests")
+
+
 @pytest.fixture
 def no_memoize(monkeypatch):
     monkeypatch.setattr('thefuck.utils.memoize.disabled', True)
@@ -15,3 +22,10 @@ def settings():
 @pytest.fixture(autouse=True)
 def no_cache(monkeypatch):
     monkeypatch.setattr('thefuck.utils.cache.disabled', True)
+
+
+@pytest.fixture(autouse=True)
+def functional(request):
+    if request.node.get_marker('functional') \
+            and not request.config.getoption('enable_functional'):
+        pytest.skip('functional tests are disabled')

tests/functional/test_bash.py

@@ -2,7 +2,6 @@ import pytest
 from tests.functional.plots import with_confirmation, without_confirmation, \
     refuse_with_confirmation, history_changed, history_not_changed, \
     select_command_with_arrows
-from tests.functional.utils import functional
 
 containers = ((u'thefuck/ubuntu-python3-bash',
                u'''FROM ubuntu:latest
@@ -30,25 +29,25 @@ def proc(request, spawnu, run_without_docker):
     return proc
 
 
-@functional
+@pytest.mark.functional
 def test_with_confirmation(proc, TIMEOUT):
     with_confirmation(proc, TIMEOUT)
     history_changed(proc, TIMEOUT, u'echo test')
 
 
-@functional
+@pytest.mark.functional
 def test_select_command_with_arrows(proc, TIMEOUT):
     select_command_with_arrows(proc, TIMEOUT)
     history_changed(proc, TIMEOUT, u'git push')
 
 
-@functional
+@pytest.mark.functional
 def test_refuse_with_confirmation(proc, TIMEOUT):
     refuse_with_confirmation(proc, TIMEOUT)
     history_not_changed(proc, TIMEOUT)
 
 
-@functional
+@pytest.mark.functional
 def test_without_confirmation(proc, TIMEOUT):
     without_confirmation(proc, TIMEOUT)
     history_changed(proc, TIMEOUT, u'echo test')

tests/functional/test_fish.py

@@ -1,7 +1,6 @@
 import pytest
 from tests.functional.plots import with_confirmation, without_confirmation, \
     refuse_with_confirmation, select_command_with_arrows
-from tests.functional.utils import functional
 
 containers = (('thefuck/ubuntu-python3-fish',
                u'''FROM ubuntu:latest
@@ -29,25 +28,25 @@ def proc(request, spawnu):
     return proc
 
 
-@functional
+@pytest.mark.functional
 @pytest.mark.skip_without_docker
 def test_with_confirmation(proc, TIMEOUT):
     with_confirmation(proc, TIMEOUT)
 
 
-@functional
+@pytest.mark.functional
 @pytest.mark.skip_without_docker
 def test_select_command_with_arrows(proc, TIMEOUT):
     select_command_with_arrows(proc, TIMEOUT)
 
 
-@functional
+@pytest.mark.functional
 @pytest.mark.skip_without_docker
 def test_refuse_with_confirmation(proc, TIMEOUT):
     refuse_with_confirmation(proc, TIMEOUT)
 
 
-@functional
+@pytest.mark.functional
 @pytest.mark.skip_without_docker
 def test_without_confirmation(proc, TIMEOUT):
     without_confirmation(proc, TIMEOUT)

tests/functional/test_install.py

@@ -1,7 +1,5 @@
 import pytest
 from thefuck.main import _get_current_version
-from tests.functional.utils import functional
-
 
 envs = ((u'bash', 'thefuck/ubuntu-bash', u'''
 FROM ubuntu:latest
@@ -13,7 +11,7 @@ RUN dnf install -yy python-devel sudo wget gcc
 '''))
 
 
-@functional
+@pytest.mark.functional
 @pytest.mark.skip_without_docker
 @pytest.mark.parametrize('shell, tag, dockerfile', envs)
 def test_installation(spawnu, shell, TIMEOUT, tag, dockerfile):

tests/functional/test_performance.py

@@ -1,6 +1,5 @@
 import pytest
 import time
-from tests.functional.utils import functional
 
 dockerfile = u'''
 FROM ubuntu:latest
@@ -40,7 +39,7 @@ def plot(proc, TIMEOUT):
     assert proc.expect([TIMEOUT, u'test'])
 
 
-@functional
+@pytest.mark.functional
 @pytest.mark.skip_without_docker
 @pytest.mark.benchmark(min_rounds=10)
 def test_performance(spawnu, TIMEOUT, benchmark):

tests/functional/test_tcsh.py

@@ -1,5 +1,4 @@
 import pytest
-from tests.functional.utils import functional
 from tests.functional.plots import with_confirmation, without_confirmation, \
     refuse_with_confirmation, select_command_with_arrows
 
@@ -30,22 +29,22 @@ def proc(request, spawnu, run_without_docker):
     return proc
 
 
-@functional
+@pytest.mark.functional
 def test_with_confirmation(proc, TIMEOUT):
     with_confirmation(proc, TIMEOUT)
 
 
-@functional
+@pytest.mark.functional
 def test_select_command_with_arrows(proc, TIMEOUT):
     select_command_with_arrows(proc, TIMEOUT)
 
 
-@functional
+@pytest.mark.functional
 def test_refuse_with_confirmation(proc, TIMEOUT):
     refuse_with_confirmation(proc, TIMEOUT)
 
 
-@functional
+@pytest.mark.functional
 def test_without_confirmation(proc, TIMEOUT):
     without_confirmation(proc, TIMEOUT)
 

tests/functional/test_zsh.py

@@ -1,5 +1,4 @@
 import pytest
-from tests.functional.utils import functional
 from tests.functional.plots import with_confirmation, without_confirmation, \
     refuse_with_confirmation, history_changed, history_not_changed, select_command_with_arrows
 
@@ -34,25 +33,25 @@ def proc(request, spawnu, run_without_docker):
     return proc
 
 
-@functional
+@pytest.mark.functional
 def test_with_confirmation(proc, TIMEOUT):
     with_confirmation(proc, TIMEOUT)
     history_changed(proc, TIMEOUT, u'echo test')
 
 
-@functional
+@pytest.mark.functional
 def test_select_command_with_arrows(proc, TIMEOUT):
     select_command_with_arrows(proc, TIMEOUT)
     history_changed(proc, TIMEOUT, u'git push')
 
 
-@functional
+@pytest.mark.functional
 def test_refuse_with_confirmation(proc, TIMEOUT):
     refuse_with_confirmation(proc, TIMEOUT)
     history_not_changed(proc, TIMEOUT)
 
 
-@functional
+@pytest.mark.functional
 def test_without_confirmation(proc, TIMEOUT):
     without_confirmation(proc, TIMEOUT)
     history_changed(proc, TIMEOUT, u'echo test')

tests/functional/utils.py

@@ -1,8 +0,0 @@
-import pytest
-import os
-
-enabled = os.environ.get('FUNCTIONAL')
-
-functional = pytest.mark.skipif(
-    not enabled,
-    reason='Functional tests are disabled by default.')