Update the list of excluded and ignored tests. Fixes #1007

Makefile

@@ -85,6 +85,14 @@ basictest: test_prelim
 
 alltest: basictest
 	cd src/greentest && GEVENT_RESOLVER=ares GEVENTARES_SERVERS=8.8.8.8 ${PYTHON} testrunner.py --config known_failures.py --ignore tests_that_dont_use_resolver.txt --quiet
+# In the past, we included all test files that had a reference to 'subprocess'' somewhere in their
+# text. The monkey-patched stdlib tests were specifically included here.
+# However, we now always also test on AppVeyor (Windows) which only has GEVENT_FILE=thread,
+# so we can save a lot of CI time by reducing the set and excluding the stdlib tests without
+# losing any coverage. See the `threadfiletest` for what command used to run.
+	cd src/greentest && GEVENT_FILE=thread ${PYTHON} testrunner.py --config known_failures.py test__*subprocess*.py --quiet
+
+threadfiletest:
 	cd src/greentest && GEVENT_FILE=thread ${PYTHON} testrunner.py --config known_failures.py `grep -l subprocess test_*.py` --quiet
 
 allbackendtest:

src/greentest/test__issue6.py

@@ -10,8 +10,11 @@ if not sys.argv[1:]:
     assert out.strip() == b'11 chars.', (out, err, code)
     # XXX: This is seen sometimes to fail on Travis with the following value in err but a code of 0;
     # it seems load related:
-    #  'Unhandled exception in thread started by \nsys.excepthook is missing\nlost sys.stderr\n'
-    assert err == b'', (out, err, code)
+    #  'Unhandled exception in thread started by \nsys.excepthook is missing\nlost sys.stderr\n'.
+    # If warnings are enabled, Python 3 has started producing this:
+    # '...importlib/_bootstrap.py:219: ImportWarning: can't resolve package from __spec__
+    #    or __package__, falling back on __name__ and __path__\n  return f(*args, **kwds)\n'
+    assert err == b'' or b'sys.excepthook' in err or b'ImportWarning' in err, (out, err, code)
 
 elif sys.argv[1:] == ['subprocess']:
     import gevent

src/greentest/tests_that_dont_use_resolver.txt

@@ -68,6 +68,46 @@ test__threading_vs_settrace.py
 test__threadpool.py
 test__timeout.py
 
+test__compat.py
+test__core_fork.py
+test__doctests.py
+test__core_loop_run_sig_mod.py
+test__execmodules.py
+test__greenio.py
+test__greenness.py
+test__hub.py
+test__import_blocking_in_greenlet.py
+test__import_wait.py
+test__issue230.py
+test__issue302monkey.py
+test__issue330.py
+test__issue467.py
+test__issue6.py
+test__issue600.py
+test__issue607.py
+test__issue461_471.py
+test__monkey_builtins_future.py
+test__monkey_hub_in_thread.py
+test__monkey_logging.py
+test__monkey_multiple_imports.py
+test__monkey_scope.py
+test__monkey_selectors.py
+test__monkey_sigchld.py
+test__monkey_sigchld_2.py
+test__nondefaultloop.py
+test__monkey_sigchld_3.py
+test__real_greenlet.py
+test__refcount.py
+test__sleep0.py
+test__subprocess_poll.py
+test__threading.py
+test__threading_before_monkey.py
+test__threading_holding_lock_while_monkey.py
+test__threading_monkey_in_thread.py
+test__threading_native_before_monkey.py
+test__threadpool_executor_patched.py
+
+
 # monkey patched standard tests:
 test_queue.py
 test_select.py
@@ -76,3 +116,10 @@ test_subprocess.py
 test_threading_local.py
 test_threading.py
 test_thread.py
+test_selectors.py
+test_timeout.py
+
+# test_asyncore probably does use the resolver, but only
+# implicitly for localhost, which is covered well enough
+# elsewhere that we don't need to spend the 20s (*2)
+test_asyncore.py