fix paths to tests

This commit is contained in:
Will Greenberg 2026-01-13 13:46:21 -08:00
parent 3130d60bd1
commit 9e4718129c
21 changed files with 26 additions and 28 deletions

View file

@ -51,7 +51,7 @@ ignore=CVS
# CERTBOT COMMENT
# Changing this line back to the default of `ignore-paths=` is being tracked by
# https://github.com/certbot/certbot/issues/7908.
ignore-paths=.*/_internal/tests/,.*/apache/tests,.*/nginx/tests
ignore-paths=.*/_internal/tests/
# Files or directories matching the regular expression patterns are skipped.
# The regex matches against base names, not paths. The default value ignores

View file

@ -5,10 +5,10 @@ recursive-include docs *
recursive-include examples *
recursive-include src/certbot/tests/testdata *
recursive-include src/certbot/_internal/plugins/nginx/tls_configs *.conf
recursive-include src/certbot/_internal/plugins/nginx/tests/testdata *
recursive-include src/certbot/_internal/tests/plugins/nginx/testdata *
recursive-include src/_internal/plugins/apache/augeas_lens *.aug
recursive-include src/_internal/plugins/apache/tls_configs *.conf
recursive-include src/_internal/plugins/apache/tests/testdata *
recursive-include src/_internal/tests/plugins/apache/testdata *
include src/certbot/ssl-dhparams.pem
include src/certbot/py.typed
global-exclude __pycache__

View file

@ -7,7 +7,7 @@ import pytest
from certbot import errors
from certbot._internal.plugins.apache import assertions
from certbot._internal.plugins.apache import augeasparser
from certbot._internal.plugins.apache.tests import util
from certbot._internal.tests.plugins.apache import util
def _get_augeasnode_mock(filepath):

View file

@ -8,7 +8,7 @@ import pytest
from certbot import errors
from certbot._internal.plugins.apache import constants
from certbot._internal.plugins.apache.tests import util
from certbot._internal.tests.plugins.apache import util
class AutoHSTSTest(util.ApacheTest):

View file

@ -9,7 +9,7 @@ from certbot.compat import filesystem
from certbot.compat import os
from certbot._internal.plugins.apache import obj
from certbot._internal.plugins.apache import override_centos
from certbot._internal.plugins.apache.tests import util
from certbot._internal.tests.plugins.apache import util
def get_vh_truth(temp_dir, config_name):

View file

@ -5,7 +5,7 @@ import pytest
from certbot import errors
from certbot.compat import os
from certbot._internal.plugins.apache.tests import util
from certbot._internal.tests.plugins.apache import util
class ComplexParserTest(util.ParserTest):

View file

@ -5,7 +5,7 @@ from unittest import mock
import pytest
from certbot import errors
from certbot._internal.plugins.apache.tests import util
from certbot._internal.tests.plugins.apache import util
class ConfiguratorReverterTest(util.ApacheTest):

View file

@ -21,7 +21,7 @@ from certbot._internal.plugins.apache import apache_util
from certbot._internal.plugins.apache import constants
from certbot._internal.plugins.apache import obj
from certbot._internal.plugins.apache import parser
from certbot._internal.plugins.apache.tests import util
from certbot._internal.tests.plugins.apache import util
class MultipleVhostsTest(util.ApacheTest):

View file

@ -10,7 +10,7 @@ from certbot.compat import os
from certbot.tests import util as certbot_util
from certbot._internal.plugins.apache import apache_util
from certbot._internal.plugins.apache import obj
from certbot._internal.plugins.apache.tests import util
from certbot._internal.tests.plugins.apache import util
class MultipleVhostsTestDebian(util.ApacheTest):

View file

@ -10,7 +10,7 @@ from certbot.display import util as display_util
from certbot.tests import util as certbot_util
from certbot._internal.plugins.apache import obj
from certbot._internal.plugins.apache.display_ops import select_vhost_multiple
from certbot._internal.plugins.apache.tests import util
from certbot._internal.tests.plugins.apache import util
class SelectVhostMultiTest(unittest.TestCase):

View file

@ -9,7 +9,7 @@ from certbot.compat import filesystem
from certbot.compat import os
from certbot._internal.plugins.apache import obj
from certbot._internal.plugins.apache import override_fedora
from certbot._internal.plugins.apache.tests import util
from certbot._internal.tests.plugins.apache import util
def get_vh_truth(temp_dir, config_name):

View file

@ -9,7 +9,7 @@ from certbot.compat import filesystem
from certbot.compat import os
from certbot._internal.plugins.apache import obj
from certbot._internal.plugins.apache import override_gentoo
from certbot._internal.plugins.apache.tests import util
from certbot._internal.tests.plugins.apache import util
def get_vh_truth(temp_dir, config_name):

View file

@ -12,7 +12,7 @@ from certbot.compat import filesystem
from certbot.compat import os
from certbot.tests import acme_util
from certbot._internal.plugins.apache.parser import get_aug_path
from certbot._internal.plugins.apache.tests import util
from certbot._internal.tests.plugins.apache import util
NUM_ACHALLS = 3

View file

@ -6,7 +6,7 @@ import pytest
from certbot import errors
from certbot.compat import os
from certbot._internal.plugins.apache.tests import util
from certbot._internal.tests.plugins.apache import util
class BasicParserTest(util.ParserTest):

View file

@ -6,7 +6,7 @@ from unittest import mock
import pytest
from certbot._internal.plugins.apache.tests import util
from certbot._internal.tests.plugins.apache import util
if importlib.util.find_spec('apacheconfig'):

View file

@ -15,7 +15,7 @@ from certbot._internal.plugins.nginx import obj
from certbot._internal.plugins.nginx import parser
from certbot._internal.plugins.nginx.configurator import _redirect_block_for_domain
from certbot._internal.plugins.nginx.nginxparser import UnspacedList
from certbot._internal.plugins.nginx.tests import test_util as util
from certbot._internal.tests.plugins.nginx import test_util as util
class NginxConfiguratorTest(util.NginxTest):

View file

@ -7,7 +7,7 @@ from certbot.display import util as display_util
from certbot.tests import util as certbot_util
from certbot._internal.plugins.nginx import parser
from certbot._internal.plugins.nginx.display_ops import select_vhost_multiple
from certbot._internal.plugins.nginx.tests import test_util as util
from certbot._internal.tests.plugins.nginx import test_util as util
class SelectVhostMultiTest(util.NginxTest):

View file

@ -10,7 +10,7 @@ from certbot import achallenges
from certbot.tests import acme_util
from certbot.tests import util as test_util
from certbot._internal.plugins.nginx.obj import Addr
from certbot._internal.plugins.nginx.tests import test_util as util
from certbot._internal.tests.plugins.nginx import test_util as util
AUTH_KEY = jose.JWKRSA.load(test_util.load_vector("rsa512_key.pem"))

View file

@ -14,7 +14,7 @@ from certbot._internal.plugins.nginx.nginxparser import load
from certbot._internal.plugins.nginx.nginxparser import loads
from certbot._internal.plugins.nginx.nginxparser import RawNginxParser
from certbot._internal.plugins.nginx.nginxparser import UnspacedList
from certbot._internal.plugins.nginx.tests import test_util as util
from certbot._internal.tests.plugins.nginx import test_util as util
FIRST = operator.itemgetter(0)

View file

@ -11,7 +11,7 @@ from certbot.compat import os
from certbot._internal.plugins.nginx import nginxparser
from certbot._internal.plugins.nginx import obj
from certbot._internal.plugins.nginx import parser
from certbot._internal.plugins.nginx.tests import test_util as util
from certbot._internal.tests.plugins.nginx import test_util as util
class NginxParserTest(util.NginxTest):

12
tox.ini
View file

@ -120,8 +120,6 @@ description = Tests Certbot without any additional plugins installed
deps =
{[testenv:isolated-acme]deps}
-e certbot[test]
# we ignore apache/nginx since those tests depend on their respective extra dependencies (which are pulled in by their plugin)
commands = {[base]pytest} certbot --ignore certbot/src/certbot/_internal/plugins/apache --ignore certbot/src/certbot/_internal/plugins/nginx
[testenv:isolated-{apache,cloudflare,digitalocean,dnsimple,dnsmadeeasy,gehirn,google,linode,luadns,nsone,ovh,rfc2136,route53,sakuracloud,nginx}]
description = Tests the plugin without installing any other plugins
@ -165,27 +163,27 @@ deps =
-e certbot
-e certbot-apache
commands =
{toxinidir}/certbot/src/certbot/_internal/plugins/apache/tests/apache-conf-files/apache-conf-test --debian-modules
{toxinidir}/certbot/src/certbot/_internal/tests/plugins/apache/apache-conf-files/apache-conf-test --debian-modules
passenv =
SERVER
allowlist_externals =
{toxinidir}/certbot/src/certbot/_internal/plugins/apache/tests/apache-conf-files/apache-conf-test
{toxinidir}/certbot/src/certbot/_internal/tests/plugins/apache/apache-conf-files/apache-conf-test
[testenv:apacheconftest-external-with-pebble]
description = Run apacheconftest with pebble and Certbot outside of the tox virtual environment.
deps =
-e certbot-ci
allowlist_externals =
{toxinidir}/certbot/src/certbot/_internal/plugins/apache/tests/apache-conf-files/apache-conf-test-pebble.py
{toxinidir}/certbot/src/certbot/_internal/tests/plugins/apache/apache-conf-files/apache-conf-test-pebble.py
commands =
{toxinidir}/certbot/src/certbot/_internal/plugins/apache/tests/apache-conf-files/apache-conf-test-pebble.py --debian-modules
{toxinidir}/certbot/src/certbot/_internal/tests/plugins/apache/apache-conf-files/apache-conf-test-pebble.py --debian-modules
[testenv:apacheconftest-with-pebble]
deps =
{[testenv:apacheconftest]deps}
{[testenv:apacheconftest-external-with-pebble]deps}
allowlist_externals =
{toxinidir}/certbot/src/certbot/_internal/plugins/apache/tests/apache-conf-files/apache-conf-test-pebble.py
{toxinidir}/certbot/src/certbot/_internal/tests/plugins/apache/apache-conf-files/apache-conf-test-pebble.py
commands = {[testenv:apacheconftest-external-with-pebble]commands}
[testenv:nginxroundtrip]