Enhance unit tests requirements handling #897
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This patch renames the optional dependences to export-mpl, matrix-mkl, matrix-scipy and import-gmsh, i.e. with the former underscore replaced by a dash. This makes explicit that is already performed by the flit build systems, rendering the old names invalid.
gertjanvanzwieten
force-pushed
the
requires
branch
8 times, most recently
from
327425d to
a1af06e
Compare
This patch modifies the _util.loadlib function to search and load a dynamic link library by its base name, i.e. without any prefix like lib, suffix like .so, .dylib, or version number. Compared to ctypes.util.find_library it adds the functionality of searching the site-package directory in case the library is not found in the default locations. The _mkl module is modified to use the new loadlib. The configure_mkl devtool is removed for being no longer required, along with the NUTILS_MATRIX_MKL_LIB environment variable.
This patch introduces the require_module method to testing.TestCase, and uses the new capability to simplify the setUp method of DocTestCase.
This patch adds the TestCase.require_application method and testing.requires_application decorator, which signals to unit tests that a test can be skipped if an application is not installed on the system. The new decorator is used to simplify conditional skipping of test_graph.generate.test_export_graphviz.
This patch adds the TestCase.require_library method and testing.requires_library decorator, which signals to unit tests that a test can be skipped if a library is not present on the system. The new decorator is used to change conditional skipping of the various matrix backend tests.
This patch adds the NUTILS_REQUIRES environment variable, which instruct the require methods and requires decorators in testing that any missing item is not acceptable and should not result in a skipped test. The variables are set in appropriate places of the test workflow.
gertjanvanzwieten
force-pushed
the
requires
branch
from
a1af06e to
bb4cb43
Compare
joostvanzwieten
approved these changes
Feb 3, 2025
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.