11 files changed, 47 insertions, 35 deletions
diff --git a/certbot-dns-ovh/Dockerfile b/certbot-dns-ovh/Dockerfile
deleted file mode 100644
index e8da96d95..000000000
--- a/certbot-dns-ovh/Dockerfile
+++ /dev/null
@@ -1,5 +0,0 @@
-FROM certbot/certbot
-
-COPY . src/certbot-dns-ovh
-
-RUN pip install --no-cache-dir --editable src/certbot-dns-ovh
diff --git a/certbot-dns-ovh/MANIFEST.in b/certbot-dns-ovh/MANIFEST.in
index 18f018c08..5a661cef6 100644
--- a/certbot-dns-ovh/MANIFEST.in
+++ b/certbot-dns-ovh/MANIFEST.in
@@ -1,3 +1,6 @@
 include LICENSE.txt
 include README.rst
 recursive-include docs *
+recursive-include tests *
+global-exclude __pycache__
+global-exclude *.py[cod]
diff --git a/certbot-dns-ovh/certbot_dns_ovh/_internal/__init__.py b/certbot-dns-ovh/certbot_dns_ovh/_internal/__init__.py
new file mode 100644
index 000000000..133694b9e
--- /dev/null
+++ b/certbot-dns-ovh/certbot_dns_ovh/_internal/__init__.py
@@ -0,0 +1 @@
+"""Internal implementation of `~certbot_dns_ovh.dns_ovh` plugin."""
diff --git a/certbot-dns-ovh/certbot_dns_ovh/dns_ovh.py b/certbot-dns-ovh/certbot_dns_ovh/_internal/dns_ovh.py
index 84771b0a8..a495983f2 100644
--- a/certbot-dns-ovh/certbot_dns_ovh/dns_ovh.py
+++ b/certbot-dns-ovh/certbot_dns_ovh/_internal/dns_ovh.py
@@ -1,8 +1,8 @@
 """DNS Authenticator for OVH DNS."""
 import logging
 
-import zope.interface
 from lexicon.providers import ovh
+import zope.interface
 
 from certbot import errors
 from certbot import interfaces
diff --git a/certbot-dns-ovh/docs/api.rst b/certbot-dns-ovh/docs/api.rst
index 8668ec5d8..ac13c3df2 100644
--- a/certbot-dns-ovh/docs/api.rst
+++ b/certbot-dns-ovh/docs/api.rst
@@ -2,7 +2,4 @@
 API Documentation
 =================
 
-.. toctree::
-   :glob:
-
-   api/**
+Certbot plugins implement the Certbot plugins API, and do not otherwise have an external API.
diff --git a/certbot-dns-ovh/docs/api/dns_ovh.rst b/certbot-dns-ovh/docs/api/dns_ovh.rst
deleted file mode 100644
index 79863d05f..000000000
--- a/certbot-dns-ovh/docs/api/dns_ovh.rst
+++ /dev/null
@@ -1,5 +0,0 @@
-:mod:`certbot_dns_ovh.dns_ovh`
-------------------------------
-
-.. automodule:: certbot_dns_ovh.dns_ovh
-   :members:
diff --git a/certbot-dns-ovh/docs/conf.py b/certbot-dns-ovh/docs/conf.py
index 57194666e..56e24a920 100644
--- a/certbot-dns-ovh/docs/conf.py
+++ b/certbot-dns-ovh/docs/conf.py
@@ -17,6 +17,7 @@
 # documentation root, use os.path.abspath to make it absolute, like shown here.
 #
 import os
+
 # import sys
 # sys.path.insert(0, os.path.abspath('.'))
 
@@ -37,7 +38,7 @@ extensions = ['sphinx.ext.autodoc',
     'sphinx.ext.viewcode']
 
 autodoc_member_order = 'bysource'
-autodoc_default_flags = ['show-inheritance', 'private-members']
+autodoc_default_flags = ['show-inheritance']
 
 # Add any paths that contain templates here, relative to this directory.
 templates_path = ['_templates']
diff --git a/certbot-dns-ovh/local-oldest-requirements.txt b/certbot-dns-ovh/local-oldest-requirements.txt
index 01cbcb317..c55e0d570 100644
--- a/certbot-dns-ovh/local-oldest-requirements.txt
+++ b/certbot-dns-ovh/local-oldest-requirements.txt
@@ -1,3 +1,4 @@
--e acme[dev]
--e .[dev]
+# Remember to update setup.py to match the package versions below.
+acme[dev]==0.31.0
+certbot[dev]==1.1.0
 dns-lexicon==2.7.14
diff --git a/certbot-dns-ovh/readthedocs.org.requirements.txt b/certbot-dns-ovh/readthedocs.org.requirements.txt
index 0780e12a1..3c21ae0ce 100644
--- a/certbot-dns-ovh/readthedocs.org.requirements.txt
+++ b/certbot-dns-ovh/readthedocs.org.requirements.txt
@@ -1,12 +1,12 @@
 # readthedocs.org gives no way to change the install command to "pip
-# install -e .[docs]" (that would in turn install documentation
+# install -e certbot-dns-ovh[docs]" (that would in turn install documentation
 # dependencies), but it allows to specify a requirements.txt file at
 # https://readthedocs.org/dashboard/letsencrypt/advanced/ (c.f. #259)
 
 # Although ReadTheDocs certainly doesn't need to install the project
-# in --editable mode (-e), just "pip install .[docs]" does not work as
-# expected and "pip install -e .[docs]" must be used instead
+# in --editable mode (-e), just "pip install certbot-dns-ovh[docs]" does not work as
+# expected and "pip install -e certbot-dns-ovh[docs]" must be used instead
 
 -e acme
--e .
+-e certbot
 -e certbot-dns-ovh[docs]
diff --git a/certbot-dns-ovh/setup.py b/certbot-dns-ovh/setup.py
index 5b3329568..8f5b052a2 100644
--- a/certbot-dns-ovh/setup.py
+++ b/certbot-dns-ovh/setup.py
@@ -1,14 +1,16 @@
-from setuptools import setup
-from setuptools import find_packages
+import sys
 
+from setuptools import find_packages
+from setuptools import setup
+from setuptools.command.test import test as TestCommand
 
-version = '0.31.0.dev0'
+version = '1.3.0.dev0'
 
 # Remember to update local-oldest-requirements.txt when changing the minimum
 # acme/certbot version.
 install_requires = [
-    'acme>=0.31.0.dev0',
-    'certbot>=0.31.0.dev0',
+    'acme>=0.31.0',
+    'certbot>=1.1.0',
     'dns-lexicon>=2.7.14',  # Correct proxy use on OVH provider
     'mock',
     'setuptools',
@@ -20,6 +22,20 @@ docs_extras = [
     'sphinx_rtd_theme',
 ]
 
+class PyTest(TestCommand):
+    user_options = []
+
+    def initialize_options(self):
+        TestCommand.initialize_options(self)
+        self.pytest_args = ''
+
+    def run_tests(self):
+        import shlex
+        # import here, cause outside the eggs aren't loaded
+        import pytest
+        errno = pytest.main(shlex.split(self.pytest_args))
+        sys.exit(errno)
+
 setup(
     name='certbot-dns-ovh',
     version=version,
@@ -28,9 +44,9 @@ setup(
     author="Certbot Project",
     author_email='client-dev@letsencrypt.org',
     license='Apache License 2.0',
-    python_requires='>=2.7, !=3.0.*, !=3.1.*, !=3.2.*, !=3.3.*',
+    python_requires='>=2.7, !=3.0.*, !=3.1.*, !=3.2.*, !=3.3.*, !=3.4.*',
     classifiers=[
-        'Development Status :: 3 - Alpha',
+        'Development Status :: 5 - Production/Stable',
         'Environment :: Plugins',
         'Intended Audience :: System Administrators',
         'License :: OSI Approved :: Apache Software License',
@@ -39,9 +55,10 @@ setup(
         'Programming Language :: Python :: 2',
         'Programming Language :: Python :: 2.7',
         'Programming Language :: Python :: 3',
-        'Programming Language :: Python :: 3.4',
         'Programming Language :: Python :: 3.5',
         'Programming Language :: Python :: 3.6',
+        'Programming Language :: Python :: 3.7',
+        'Programming Language :: Python :: 3.8',
         'Topic :: Internet :: WWW/HTTP',
         'Topic :: Security',
         'Topic :: System :: Installation/Setup',
@@ -58,8 +75,10 @@ setup(
     },
     entry_points={
         'certbot.plugins': [
-            'dns-ovh = certbot_dns_ovh.dns_ovh:Authenticator',
+            'dns-ovh = certbot_dns_ovh._internal.dns_ovh:Authenticator',
         ],
     },
+    tests_require=["pytest"],
     test_suite='certbot_dns_ovh',
+    cmdclass={"test": PyTest},
 )
diff --git a/certbot-dns-ovh/certbot_dns_ovh/dns_ovh_test.py b/certbot-dns-ovh/tests/dns_ovh_test.py
index f2a10485d..a420239ab 100644
--- a/certbot-dns-ovh/certbot_dns_ovh/dns_ovh_test.py
+++ b/certbot-dns-ovh/tests/dns_ovh_test.py
@@ -1,11 +1,11 @@
-"""Tests for certbot_dns_ovh.dns_ovh."""
+"""Tests for certbot_dns_ovh._internal.dns_ovh."""
 
-import os
 import unittest
 
 import mock
 from requests.exceptions import HTTPError
 
+from certbot.compat import os
 from certbot.plugins import dns_test_common
 from certbot.plugins import dns_test_common_lexicon
 from certbot.tests import util as test_util
@@ -22,7 +22,7 @@ class AuthenticatorTest(test_util.TempDirTestCase,
     def setUp(self):
         super(AuthenticatorTest, self).setUp()
 
-        from certbot_dns_ovh.dns_ovh import Authenticator
+        from certbot_dns_ovh._internal.dns_ovh import Authenticator
 
         path = os.path.join(self.tempdir, 'file.ini')
         credentials = {
@@ -48,7 +48,7 @@ class OVHLexiconClientTest(unittest.TestCase, dns_test_common_lexicon.BaseLexico
     LOGIN_ERROR = HTTPError('403 Client Error: Forbidden for url: https://eu.api.ovh.com/1.0/...')
 
     def setUp(self):
-        from certbot_dns_ovh.dns_ovh import _OVHLexiconClient
+        from certbot_dns_ovh._internal.dns_ovh import _OVHLexiconClient
 
         self.client = _OVHLexiconClient(
             ENDPOINT, APPLICATION_KEY, APPLICATION_SECRET, CONSUMER_KEY, 0