Skip to content

TST: Move full conftest.py back to package level #3268

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
62 changes: 62 additions & 0 deletions astroquery/conftest.py
Original file line number Diff line number Diff line change
@@ -0,0 +1,62 @@
# Licensed under a 3-clause BSD style license - see LICENSE.rst

import os
from pathlib import Path

from astropy.utils import minversion
import numpy as np
import pytest
from pytest_astropy_header.display import (PYTEST_HEADER_MODULES,
TESTED_VERSIONS)


# Keep this until we require numpy to be >=2.0
if minversion(np, "2.0.0.dev0+git20230726"):
np.set_printoptions(legacy="1.25")


def pytest_configure(config):
config.option.astropy_header = True

PYTEST_HEADER_MODULES['Astropy'] = 'astropy'
PYTEST_HEADER_MODULES['regions'] = 'regions'
PYTEST_HEADER_MODULES['pyVO'] = 'pyvo'
PYTEST_HEADER_MODULES['mocpy'] = 'mocpy'
PYTEST_HEADER_MODULES['astropy-healpix'] = 'astropy_healpix'
PYTEST_HEADER_MODULES['vamdclib'] = 'vamdclib'

PYTEST_HEADER_MODULES.pop('h5py', None)
PYTEST_HEADER_MODULES.pop('Scipy', None)
PYTEST_HEADER_MODULES.pop('Pandas', None)

# keyring doesn't provide __version__ any more
# PYTEST_HEADER_MODULES['keyring'] = 'keyring'

# add '_testrun' to the version name so that the user-agent indicates that
# it's being run in a test
from astroquery import version
version.version += '_testrun'

TESTED_VERSIONS['astroquery'] = version.version
TESTED_VERSIONS['astropy_helpers'] = version.astropy_helpers_version


def pytest_addoption(parser):
parser.addoption(
'--alma-site',
action='store',
default='almascience.eso.org',
help='ALMA site (almascience.nrao.edu, almascience.eso.org or '
'almascience.nao.ac.jp for example)'
)


@pytest.fixture(scope='function')
def tmp_cwd(tmp_path):
"""Perform test in a pristine temporary working directory."""
old_dir = Path.cwd()
os.chdir(tmp_path)
try:
yield tmp_path
finally:
os.chdir(old_dir)
40 changes: 4 additions & 36 deletions conftest.py
Original file line number Diff line number Diff line change
@@ -1,20 +1,9 @@
# Licensed under a 3-clause BSD style license - see LICENSE.rst
# Duplicate pytest header config here for tox.

import os
from pathlib import Path

from astropy.utils import minversion
import numpy as np
import pytest
from pytest_astropy_header.display import (PYTEST_HEADER_MODULES,
TESTED_VERSIONS)


# Keep this until we require numpy to be >=2.0
if minversion(np, "2.0.0.dev0+git20230726"):
np.set_printoptions(legacy="1.25")


def pytest_configure(config):
config.option.astropy_header = True

Expand All @@ -25,9 +14,9 @@ def pytest_configure(config):
PYTEST_HEADER_MODULES['astropy-healpix'] = 'astropy_healpix'
PYTEST_HEADER_MODULES['vamdclib'] = 'vamdclib'

del PYTEST_HEADER_MODULES['h5py']
del PYTEST_HEADER_MODULES['Scipy']
del PYTEST_HEADER_MODULES['Pandas']
PYTEST_HEADER_MODULES.pop('h5py', None)
PYTEST_HEADER_MODULES.pop('Scipy', None)
PYTEST_HEADER_MODULES.pop('Pandas', None)

# keyring doesn't provide __version__ any more
# PYTEST_HEADER_MODULES['keyring'] = 'keyring'
Expand All @@ -39,24 +28,3 @@ def pytest_configure(config):

TESTED_VERSIONS['astroquery'] = version.version
TESTED_VERSIONS['astropy_helpers'] = version.astropy_helpers_version


def pytest_addoption(parser):
parser.addoption(
'--alma-site',
action='store',
default='almascience.eso.org',
help='ALMA site (almascience.nrao.edu, almascience.eso.org or '
'almascience.nao.ac.jp for example)'
)


@pytest.fixture(scope='function')
def tmp_cwd(tmp_path):
"""Perform test in a pristine temporary working directory."""
old_dir = Path.cwd()
os.chdir(tmp_path)
try:
yield tmp_path
finally:
os.chdir(old_dir)
Loading