From 4cfb3535ee1cfd952f9a5a6b0bb6edf0317c5b60 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?C=C3=A9dric=20Krier?= <ced@b2ck.com>
Date: Sat, 16 Apr 2022 18:30:18 +0200
Subject: [PATCH] Replace test setuptools command by unittest discover

issue9215
review389851002
---
 setup.py                        |  6 ++---
 tests/__init__.py               |  8 ------
 tests/test_module.py            | 12 +++++++++
 tests/test_product_cost_fifo.py | 45 ---------------------------------
 tests/test_scenario.py          | 22 ++++++++++++++++
 tox.ini                         |  3 ++-
 6 files changed, 39 insertions(+), 57 deletions(-)
 create mode 100644 tests/test_module.py
 delete mode 100644 tests/test_product_cost_fifo.py
 create mode 100644 tests/test_scenario.py

diff --git a/setup.py b/setup.py
index cceb2ec70a..794d2f94f4 100755
--- a/setup.py
+++ b/setup.py
@@ -138,13 +138,13 @@ setup(name=name,
     license='GPL-3',
     python_requires='>=3.7',
     install_requires=requires,
+    extras_require={
+        'test': tests_require,
+        },
     dependency_links=dependency_links,
     zip_safe=False,
     entry_points="""
     [trytond.modules]
     product_cost_fifo = trytond.modules.product_cost_fifo
     """,
-    test_suite='tests',
-    test_loader='trytond.test_loader:Loader',
-    tests_require=tests_require,
     )
diff --git a/tests/__init__.py b/tests/__init__.py
index 1916fe99ba..4effdfa4a2 100644
--- a/tests/__init__.py
+++ b/tests/__init__.py
@@ -1,10 +1,2 @@
 # This file is part of Tryton.  The COPYRIGHT file at the top level of
 # this repository contains the full copyright notices and license terms.
-
-try:
-    from trytond.modules.product_cost_fifo.tests.test_product_cost_fifo import \
-        suite  # noqa: E501
-except ImportError:
-    from .test_product_cost_fifo import suite
-
-__all__ = ['suite']
diff --git a/tests/test_module.py b/tests/test_module.py
new file mode 100644
index 0000000000..1d5c34c882
--- /dev/null
+++ b/tests/test_module.py
@@ -0,0 +1,12 @@
+# This file is part of Tryton.  The COPYRIGHT file at the top level of
+# this repository contains the full copyright notices and license terms.
+
+from trytond.tests.test_tryton import ModuleTestCase
+
+
+class ProductCostFIFOTestCase(ModuleTestCase):
+    'Test ProductCostFIFO module'
+    module = 'product_cost_fifo'
+
+
+del ModuleTestCase
diff --git a/tests/test_product_cost_fifo.py b/tests/test_product_cost_fifo.py
deleted file mode 100644
index 2082832f85..0000000000
--- a/tests/test_product_cost_fifo.py
+++ /dev/null
@@ -1,45 +0,0 @@
-# This file is part of Tryton.  The COPYRIGHT file at the top level of
-# this repository contains the full copyright notices and license terms.
-import doctest
-import unittest
-
-import trytond.tests.test_tryton
-from trytond.tests.test_tryton import (
-    ModuleTestCase, doctest_checker, doctest_teardown)
-
-
-class ProductCostFIFOTestCase(ModuleTestCase):
-    'Test ProductCostFIFO module'
-    module = 'product_cost_fifo'
-
-
-def suite():
-    suite = trytond.tests.test_tryton.suite()
-    suite.addTests(unittest.TestLoader().loadTestsFromTestCase(
-        ProductCostFIFOTestCase))
-    suite.addTests(doctest.DocFileSuite(
-            'scenario_product_cost_fifo.rst',
-            tearDown=doctest_teardown, encoding='utf-8',
-            checker=doctest_checker,
-            optionflags=doctest.REPORT_ONLY_FIRST_FAILURE))
-    suite.addTests(doctest.DocFileSuite(
-            'scenario_product_cost_fifo_uom.rst',
-            tearDown=doctest_teardown, encoding='utf-8',
-            checker=doctest_checker,
-            optionflags=doctest.REPORT_ONLY_FIRST_FAILURE))
-    suite.addTests(doctest.DocFileSuite(
-            'scenario_product_cost_fifo_no_in.rst',
-            tearDown=doctest_teardown, encoding='utf-8',
-            checker=doctest_checker,
-            optionflags=doctest.REPORT_ONLY_FIRST_FAILURE))
-    suite.addTests(doctest.DocFileSuite(
-            'scenario_product_cost_fifo_recompute_cost_price.rst',
-            tearDown=doctest_teardown, encoding='utf-8',
-            checker=doctest_checker,
-            optionflags=doctest.REPORT_ONLY_FIRST_FAILURE))
-    suite.addTests(doctest.DocFileSuite(
-            'scenario_product_cost_fifo_recompute_cost_price_production.rst',
-            tearDown=doctest_teardown, encoding='utf-8',
-            checker=doctest_checker,
-            optionflags=doctest.REPORT_ONLY_FIRST_FAILURE))
-    return suite
diff --git a/tests/test_scenario.py b/tests/test_scenario.py
new file mode 100644
index 0000000000..c80e682a7f
--- /dev/null
+++ b/tests/test_scenario.py
@@ -0,0 +1,22 @@
+# This file is part of Tryton.  The COPYRIGHT file at the top level of
+# this repository contains the full copyright notices and license terms.
+
+import doctest
+import glob
+import os
+
+from trytond.tests.test_tryton import doctest_checker, doctest_teardown
+
+
+def load_tests(loader, tests, pattern):
+    cwd = os.getcwd()
+    try:
+        os.chdir(os.path.dirname(__file__))
+        for scenario in glob.glob('*.rst'):
+            tests.addTests(doctest.DocFileSuite(
+                    scenario, tearDown=doctest_teardown, encoding='utf-8',
+                    checker=doctest_checker,
+                    optionflags=doctest.REPORT_ONLY_FIRST_FAILURE))
+    finally:
+        os.chdir(cwd)
+    return tests
diff --git a/tox.ini b/tox.ini
index 2bdc04b1e9..75d8f383d4 100644
--- a/tox.ini
+++ b/tox.ini
@@ -2,8 +2,9 @@
 envlist = {py37,py38,py39,py310}-{sqlite,postgresql}
 
 [testenv]
+extras = test
 commands =
-    coverage run --include=.*/product_cost_fifo/* setup.py test
+    coverage run --include=.*/product_cost_fifo/* -m unittest discover -s tests
     coverage report --include=.*/product_cost_fifo/* --omit=*/tests/*
 deps =
     coverage