astropy · pllim · Apr 22, 2022 · Jan 17, 2022 · Jan 17, 2022 · Feb 17, 2022
diff --git a/.github/workflows/ci_workflows.yml b/.github/workflows/ci_workflows.yml
@@ -69,7 +69,7 @@ jobs:
             python: 3.9
             toxenv: py39-test-alldeps
             toxargs: -v --develop
-            toxposargs: --open-files
+            toxposargs: --open-files --run-slow
 
           - name: Python 3.8 with oldest supported version of all dependencies
             os: ubuntu-18.04
@@ -87,7 +87,7 @@ jobs:
             os: macos-latest
             python: 3.9
             toxenv: py39-test-alldeps
-            toxposargs: --durations=50
+            toxposargs: --durations=50 --run-slow
 
     steps:
     - name: Checkout code

diff --git a/.pyinstaller/run_astropy_tests.py b/.pyinstaller/run_astropy_tests.py
@@ -106,7 +106,8 @@
 # Run the tests!
 sys.exit(pytest.main(['astropy_tests',
                       '-k ' + ' and '.join('not ' + test for test in SKIP_TESTS)],
-                     plugins=['pytest_doctestplus.plugin',
+                     plugins=['pytest_astropy.plugin',
+                              'pytest_doctestplus.plugin',
                               'pytest_openfiles.plugin',
                               'pytest_remotedata.plugin',
                               'pytest_astropy_header.display']))
@@ -99,6 +99,7 @@ def test_gaussian_eval_2D(mode):
 
 
 @pytest.mark.skipif('not HAS_SCIPY')
+@pytest.mark.slow
 def test_gaussian_eval_2D_integrate_mode():
     """
     Discretize Gaussian with integrate mode

diff --git a/astropy/coordinates/tests/test_spectral_coordinate.py b/astropy/coordinates/tests/test_spectral_coordinate.py
@@ -839,6 +839,7 @@ def test_spectral_coord_from_sky_coord_without_distance():
 
 
 @pytest.mark.parametrize('specsys', list(EXPECTED_VELOCITY_FRAMES))
+@pytest.mark.slow
 def test_spectralcoord_accuracy(specsys):
 
     # This is a test to check the numerical results of transformations between

@@ -1038,16 +1038,14 @@ def test_fast_tab_with_names(parallel, read_tab):
     read_tab(content, data_start=1, parallel=parallel, names=head)
 
 
-@pytest.mark.skipif(not os.getenv('TEST_READ_HUGE_FILE'),
-                    reason='Environment variable TEST_READ_HUGE_FILE must be '
-                    'defined to run this test')
+@pytest.mark.hugemem
 def test_read_big_table(tmpdir):
     """Test reading of a huge file.
 
     This test generates a huge CSV file (~2.3Gb) before reading it (see
     https://github.com/astropy/astropy/pull/5319). The test is run only if the
-    environment variable ``TEST_READ_HUGE_FILE`` is defined. Note that running
-    the test requires quite a lot of memory (~18Gb when reading the file) !!
+    ``--run-hugemem`` cli option is given. Note that running the test requires
+    quite a lot of memory (~18Gb when reading the file) !!
 
     """
     NB_ROWS = 250000
@@ -1072,9 +1070,7 @@ def test_read_big_table(tmpdir):
     assert len(t) == NB_ROWS
 
 
-@pytest.mark.skipif(not os.getenv('TEST_READ_HUGE_FILE'),
-                    reason='Environment variable TEST_READ_HUGE_FILE must be '
-                    'defined to run this test')
+@pytest.mark.hugemem
 def test_read_big_table2(tmpdir):
     """Test reading of a file with a huge column.
     """

@@ -215,6 +215,7 @@ def test_quiet(self, capsys):
         assert out == ""
         assert err == ""
 
+    @pytest.mark.slow
     def test_path(self, capsys):
         os.mkdir(self.temp('sub/'))
         tmp_b = self.temp('sub/ascii.fits')

@@ -1241,6 +1241,7 @@ def test_open_comp_image_in_update_mode(self):
         # opening and closing it.
         assert mtime == os.stat(self.temp('comp.fits')).st_mtime
 
+    @pytest.mark.slow
     def test_open_scaled_in_update_mode_compressed(self):
         """
         Regression test for https://aeon.stsci.edu/ssb/trac/pyfits/ticket/88 2

@@ -2682,6 +2682,7 @@ def readfile(filename):
             readfile(self.data('memtest.fits'))
 
     @pytest.mark.skipif('not HAVE_OBJGRAPH')
+    @pytest.mark.slow
     def test_reference_leak2(self, tmpdir):
         """
         Regression test for https://github.com/astropy/astropy/pull/4539

@@ -194,6 +194,7 @@ def asdf_check(ff):
     helpers.assert_roundtrip_tree(tree, tmpdir, asdf_check_func=asdf_check)
 
 
+@pytest.mark.slow
 def test_generic_projections(tmpdir):
     from astropy.io.misc.asdf.tags.transform import projections
 

diff --git a/astropy/samp/tests/test_hub.py b/astropy/samp/tests/test_hub.py
@@ -2,9 +2,10 @@
 
 import time
 
-from astropy.samp.hub import SAMPHubServer
+import pytest
 
 from astropy.samp import conf
+from astropy.samp.hub import SAMPHubServer
 
 
 def setup_module(module):
@@ -24,6 +25,7 @@ def test_SAMPHubServer_run():
     hub.stop()
 
 
+@pytest.mark.slow
 def test_SAMPHubServer_run_repeated():
     """
     Test that SAMPHub can be restarted after it has been stopped, including

diff --git a/astropy/samp/tests/test_hub_script.py b/astropy/samp/tests/test_hub_script.py
@@ -1,8 +1,9 @@
 import sys
 
-from astropy.samp.hub_script import hub_script
+import pytest
 
 from astropy.samp import conf
+from astropy.samp.hub_script import hub_script
 
 
 def setup_module(module):
@@ -18,6 +19,7 @@ def teardown_function(function):
     sys.argv = function.sys_argv_orig
 
 
+@pytest.mark.slow
 def test_hub_script():
     sys.argv.append('-m')  # run in multiple mode
     sys.argv.append('-w')  # disable web profile

diff --git a/astropy/units/tests/test_quantity_ufuncs.py b/astropy/units/tests/test_quantity_ufuncs.py
@@ -83,6 +83,7 @@ def test_removal_addition(self):
         assert np.add in qh.UFUNC_HELPERS
         assert np.add not in qh.UNSUPPORTED_UFUNCS
 
+    @pytest.mark.slow
     def test_thread_safety(self, fast_thread_switching):
         def dummy_ufunc(*args, **kwargs):
             return np.sqrt(*args, **kwargs)

diff --git a/astropy/utils/tests/test_data.py b/astropy/utils/tests/test_data.py
@@ -775,6 +775,7 @@ def test_download_parallel_partial_success(temp_cache, valid_urls, invalid_urls)
     # assert not any([is_url_in_cache(u) for (u, c) in td])
 
 
+@pytest.mark.slow
 def test_download_parallel_partial_success_lock_safe(temp_cache, valid_urls, invalid_urls):
     """Check that a partially successful parallel download leaves the cache unlocked.
 

diff --git a/astropy/utils/tests/test_decorators.py b/astropy/utils/tests/test_decorators.py
@@ -528,6 +528,7 @@ def _get_foo(cls): return 1
     assert B.__dict__['foo'].__doc__ == "The foo."
 
 
+@pytest.mark.slow
 def test_classproperty_lazy_threadsafe(fast_thread_switching):
     """
     Test that a class property with lazy=True is thread-safe.
@@ -555,6 +556,7 @@ def foo(cls):
             assert values == [values[0]] * workers
 
 
+@pytest.mark.slow
 def test_lazyproperty_threadsafe(fast_thread_switching):
     """
     Test thread safety of lazyproperty.

diff --git a/docs/development/releasing.rst b/docs/development/releasing.rst
@@ -233,7 +233,7 @@ of the tests actually run), using tox to do a thorough test in an isolated
 environment::
 
    $ pip install tox --upgrade
-   $ TEST_READ_HUGE_FILE=1 tox -e test-alldeps -- --remote-data=any
+   $ tox -e test-alldeps -- --remote-data=any --run-slow --run-hugemem
 
 Additional notes
 ----------------

diff --git a/setup.cfg b/setup.cfg
@@ -57,7 +57,7 @@ test =  # Required to run the astropy test suite.
     pytest>=7.0
     pytest-doctestplus>=0.12
     pytest-astropy-header>=0.2.1
-    pytest-astropy>=0.9
+    pytest-astropy>=0.10
     pytest-xdist
 test_all =  # Required for testing, plus packages used by particular tests.
     pytest>=7.0

diff --git a/tox.ini b/tox.ini
@@ -19,7 +19,7 @@ isolated_build = true
 pypi_filter = https://raw.githubusercontent.com/astropy/ci-helpers/main/pip_pinnings.txt
 
 # Pass through the following environment variables which are needed for the CI
-passenv = HOME WINDIR LC_ALL LC_CTYPE CC CI IS_CRON ARCH_ON_CI TEST_READ_HUGE_FILE
+passenv = HOME WINDIR LC_ALL LC_CTYPE CC CI IS_CRON ARCH_ON_CI
 
 # For coverage, we need to pass extra options to the C compiler
 setenv =
@@ -151,6 +151,7 @@ commands =
                 --exclude-module tkinter \
                 --collect-submodules=py \
                 --hidden-import pytest \
+                --hidden-import pytest_astropy.plugin \
                 --hidden-import pytest_openfiles.plugin \
                 --hidden-import pytest_remotedata.plugin \
                 --hidden-import pytest_doctestplus.plugin \