Merge pull request #3059 from asottile/yesqa

Remove unnecessary `# noqa` comments.

testing/code/test_excinfo.py

@@ -542,7 +542,7 @@ class FakeRawTB(object):
         tb = FakeRawTB()
         excinfo.traceback = Traceback(tb)
 
-        fail = IOError()  # noqa
+        fail = IOError()
         repr = pr.repr_excinfo(excinfo)
         assert repr.reprtraceback.reprentries[0].lines[0] == ">   ???"
         if py.std.sys.version_info[0] >= 3:

testing/test_runner.py

@@ -32,7 +32,7 @@ def test_setup_fails_and_failure_is_cached(self, testdir):
             def setup_module(mod):
                 raise ValueError(42)
             def test_func(): pass
-        """)  # noqa
+        """)
         ss = runner.SetupState()
         pytest.raises(ValueError, lambda: ss.prepare(item))
         pytest.raises(ValueError, lambda: ss.prepare(item))
@@ -564,7 +564,7 @@ def f():
         importorskip("asdlkj")
 
     try:
-        sys = importorskip("sys")  # noqa
+        sys = importorskip("sys")
         assert sys == py.std.sys
         # path = pytest.importorskip("os.path")
         # assert path == py.std.os.path

testing/test_skipping.py

@@ -589,7 +589,7 @@ def test_skipif_conditional(self, testdir):
             @pytest.mark.skipif("hasattr(os, 'sep')")
             def test_func():
                 pass
-        """)  # noqa
+        """)
         x = pytest.raises(pytest.skip.Exception, lambda:
                           pytest_runtest_setup(item))
         assert x.value.msg == "condition: hasattr(os, 'sep')"