Merge pull request #5174 from blueyed/no-terminal

Terminal plugin is not semi-essential anymore

Author: blueyed

src/_pytest/config/__init__.py

@@ -123,7 +123,7 @@ def directory_arg(path, optname):
 )
 
 default_plugins = essential_plugins + (
-    "terminal",  # Has essential options, but xdist uses -pno:terminal.
+    "terminal",
     "debugging",
     "unittest",
     "capture",

testing/test_config.py

@@ -1232,8 +1232,10 @@ def test_config_blocked_default_plugins(testdir, plugin):
     if plugin != "terminal":
         result.stdout.fnmatch_lines(["* 1 passed in *"])
 
-    if plugin != "terminal":  # fails to report due to its options being used elsewhere.
-        p = testdir.makepyfile("def test(): assert 0")
-        result = testdir.runpytest(str(p), "-pno:%s" % plugin)
-        assert result.ret == EXIT_TESTSFAILED
+    p = testdir.makepyfile("def test(): assert 0")
+    result = testdir.runpytest(str(p), "-pno:%s" % plugin)
+    assert result.ret == EXIT_TESTSFAILED
+    if plugin != "terminal":
         result.stdout.fnmatch_lines(["* 1 failed in *"])
+    else:
+        assert result.stdout.lines == [""]