Skip to content

Commit f8541da

Browse files
committed
Merge branch 'lammps_hook' into test_lammps_hook
2 parents 8bd8c41 + a76c78d commit f8541da

1 file changed

Lines changed: 4 additions & 8 deletions

File tree

eb_hooks.py

Lines changed: 4 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -1592,14 +1592,6 @@ def pre_test_hook_ignore_failing_tests_LAMMPS_ARM_generic(self, *args, **kwargs)
15921592
print(build_option('ignore_test_failure'))
15931593

15941594

1595-
def post_test_hook(self, *args, **kwargs):
1596-
"""
1597-
If self.orig_ignore_test_failure exists reset ignore_test_failure to its original value.
1598-
"""
1599-
if hasattr(self, "orig_ignore_test_failure") and self.orig_ignore_test_failure != build_option('ignore_test_failure'):
1600-
update_build_option('ignore_test_failure', self.orig_ignore_test_failure)
1601-
1602-
16031595
def pre_single_extension_hook(ext, *args, **kwargs):
16041596
"""Main pre-extension: trigger custom functions based on software name."""
16051597
if ext.name in PRE_SINGLE_EXTENSION_HOOKS:
@@ -1947,6 +1939,10 @@ def post_easyblock_hook(self, *args, **kwargs):
19471939
post_easyblock_hook_copy_easybuild_subdir(self, *args, **kwargs)
19481940
else:
19491941
self.log.debug("No CVMFS/site installation requested, not running post_easyblock_hook_copy_easybuild_subdir.")
1942+
1943+
# If self.orig_ignore_test_failure is set return it to its original value.
1944+
if hasattr(self, "orig_ignore_test_failure") and self.orig_ignore_test_failure != build_option('ignore_test_failure'):
1945+
update_build_option('ignore_test_failure', self.orig_ignore_test_failure)
19501946
else:
19511947
print_warning(f"Not enabling the post_easybuild_hook, as it requires EasyBuild 5.1.1 or newer (you are using {EASYBUILD_VERSION}).")
19521948

0 commit comments

Comments
 (0)