Update tags for tests enabled by recent developments

aaronj0 · aaronj0 · commit eb0006fa9032 · 2025-04-02T11:46:13.000+02:00
diff --git a/test/test_boost.py b/test/test_boost.py
@@ -28,7 +28,7 @@ def test01_any_class(self):
 
         assert std.list[any]
 
-    @mark.xfail
+    @mark.skip
     def test02_any_usage(self):
         """boost::any assignment and casting"""
 
@@ -101,7 +101,7 @@ def setup_class(cls):
         cppyy.include("boost/variant/variant.hpp")
         cppyy.include("boost/variant/get.hpp")
 
-    @mark.xfail
+    @mark.skip
     def test01_variant_usage(self):
         """boost::variant usage"""
 
@@ -130,6 +130,10 @@ class C { }; } """)
         assert v.back().which() == 2
 
         assert type(boost.get['BV::A'](v[0])) == cpp.BV.A
+
+        # Trying to raise this exception seg faults, by trying to execute an unfit instantiation.
+        # This comes from `Instantiate` obtaining a single handle and providing a result
+        # The same issue happens with trying `BestOverloadFunctionMatch` first since the candidate set is single
         raises(Exception, boost.get['BV::B'], v[0])
         assert type(boost.get['BV::B'](v[1])) == cpp.BV.B
         assert type(boost.get['BV::C'](v[2])) == cpp.BV.C
diff --git a/test/test_conversions.py b/test/test_conversions.py
@@ -87,7 +87,7 @@ def test03_error_handling(self):
         gc.collect()
         assert CC.s_count == 0
 
-    @mark.xfail
+    @mark.xfail(condition = IS_MAC and IS_CLANG_REPL)
     def test04_implicit_conversion_from_tuple(self):
         """Allow implicit conversions from tuples as arguments {}-like"""
 
diff --git a/test/test_cpp11features.py b/test/test_cpp11features.py
@@ -1,6 +1,6 @@
 import py, os, sys
 from pytest import raises, mark
-from .support import setup_make, ispypy, IS_CLANG_REPL, IS_LINUX_ARM
+from .support import setup_make, ispypy, IS_CLANG_REPL, IS_LINUX_ARM, IS_MAC
 
 
 currpath = py.path.local(__file__).dirpath()
@@ -44,7 +44,6 @@ def test01_smart_ptr(self):
             gc.collect()
             assert TestSmartPtr.s_counter == 0
 
-    @mark.xfail(run=False, reason="Crashes")
     def test02_smart_ptr_construction(self):
         """Shared/Unique pointer ctor is templated, requiring special care"""
 
@@ -72,7 +71,6 @@ class C(TestSmartPtr):
             gc.collect()
             assert TestSmartPtr.s_counter == 0
 
-    @mark.xfail(run=False, reason="Crashes")
     def test03_smart_ptr_memory_handling(self):
         """Test shared/unique pointer memory ownership"""
 
@@ -103,7 +101,7 @@ class C(TestSmartPtr):
             gc.collect()
             assert TestSmartPtr.s_counter == 0
 
-    @mark.xfail(run=False, reason="Crashes")
+    @mark.xfail
     def test04_shared_ptr_passing(self):
         """Ability to pass shared_ptr<Derived> through shared_ptr<Base>"""
 
@@ -409,7 +407,7 @@ def test13_stdhash(self):
             assert hash(sw)  == 17
             assert hash(sw)  == 17
 
-    @mark.xfail(run=False, reason="Crashes")
+    @mark.xfail
     def test14_shared_ptr_passing(self):
         """Ability to pass normal pointers through shared_ptr by value"""
 
@@ -435,7 +433,7 @@ def test14_shared_ptr_passing(self):
             gc.collect()
             assert TestSmartPtr.s_counter == 0
 
-    @mark.xfail
+    @mark.xfail(condition=(IS_MAC and not IS_CLANG_REPL), reason = "Fails on OS X Cling")
     def test15_unique_ptr_template_deduction(self):
         """Argument type deduction with std::unique_ptr"""
 
diff --git a/test/test_crossinheritance.py b/test/test_crossinheritance.py
@@ -130,7 +130,7 @@ def get_value(self):
         assert c4.m_int == 88
         assert CX.IBase2.call_get_value(c4) == 77
 
-    @mark.xfail(condition=IS_MAC_ARM, run=not IS_MAC_ARM, reason="Crashes with exception not being caught on Apple Silicon")
+    @mark.xfail(run=False, condition=IS_MAC_ARM, reason="Crashes with exception not being caught on Apple Silicon")
     def test04_arguments(self):
         """Test ability to override functions that take arguments"""
 
@@ -405,7 +405,6 @@ def call(self):
         gc.collect()
         assert CB.s_count == 0 + start_count
 
-    @mark.xfail(run=False, reason="Crashes")
     def test12_python_shared_ptr_memory(self):
         """Usage of Python derived objects with std::shared_ptr"""
 
@@ -1034,7 +1033,7 @@ def return_const(self):
         assert a.return_const().m_value == "abcdef"
         assert ns.callit(a).m_value     == "abcdef"
 
-    @mark.xfail
+    @mark.xfail(condition = IS_MAC and IS_CLANG_REPL)
     def test24_non_copyable(self):
         """Inheriting from a non-copyable base class"""
 
diff --git a/test/test_datatypes.py b/test/test_datatypes.py
@@ -1269,7 +1269,7 @@ def run(self, f, buf, total):
         if self.has_byte:
             run(self, cppyy.gbl.sum_byte_data, buf, total)
 
-    @mark.xfail(run=not(((IS_CLANG_REPL and IS_MAC) or (not IS_CLANG_REPL and IS_MAC))), reason="Crashes")
+    @mark.xfail(run=not(IS_MAC), reason="Crashes on OS X")
     def test26_function_pointers(self):
         """Function pointer passing"""
 
@@ -1332,7 +1332,7 @@ def sum_in_python(i1, i2, i3):
         ns = cppyy.gbl.FuncPtrReturn
         assert ns.foo()() == "Hello, World!"
 
-    @mark.xfail(run=False, condition=(IS_CLANG_REPL and IS_MAC) or (not IS_CLANG_REPL and IS_MAC), reason="Crashes")
+    @mark.xfail(run=False, condition=IS_MAC, reason="Crashes")
     def test27_callable_passing(self):
         """Passing callables through function pointers"""
 
@@ -2011,7 +2011,7 @@ class Atom {
         assert b.name     == "aap"
         assert b.buf_type == ns.SHAPE
 
-    @mark.xfail(run=not IS_CLANG_REPL, reason="Crashes")
+    @mark.xfail
     def test40_more_aggregates(self):
         """More aggregate testings (used to fail/report errors)"""
 
@@ -2049,7 +2049,6 @@ def test40_more_aggregates(self):
             r2 = ns.make_R2()
             assert r2.s.x == 1
 
-    @mark.xfail(run=False, reason="Fails")
     def test41_complex_numpy_arrays(self):
         """Usage of complex numpy arrays"""
 
@@ -2097,7 +2096,7 @@ def pycompdot(a, b, N):
             Ccl = func(Acl, Bcl, 2)
             assert complex(Ccl) == pyCcl
 
-    @mark.xfail
+    @mark.xfail(condition=(IS_MAC and not IS_CLANG_REPL), reason = "Fails on OS X Cling")
     def test42_mixed_complex_arithmetic(self):
         """Mixin of Python and C++ std::complex in arithmetic"""
 
diff --git a/test/test_doc_features.py b/test/test_doc_features.py
@@ -1018,7 +1018,7 @@ def pythonize_topologic_printing(klass, name):
 
         assert str(s) == "hi there!"
 
-    @mark.xfail
+    @mark.xfail(condition = IS_MAC, reason = "Fails on OS X")
     def test10_llvm_blog(self):
         """Test code posted in the LLVM blog posting"""
 
@@ -1092,6 +1092,7 @@ def setup_class(cls):
 
         cppyy.gbl.talk_examples
 
+    # @mark.skip
     def test_template_instantiation(self):
         """Run-time template instantiation example"""
 
diff --git a/test/test_lowlevel.py b/test/test_lowlevel.py
@@ -47,7 +47,6 @@ def test02_builtin_cpp_casts(self):
         assert len(ll.reinterpret_cast['int*'](0)) == 0
         raises(ReferenceError, ll.reinterpret_cast['int*'](0).__getitem__, 0)
 
-    @mark.xfail
     def test03_memory(self):
         """Memory allocation and free-ing"""
 
@@ -681,7 +680,6 @@ def test03_3D_arrays(self):
                         assert arr[i][j][k] == val
                         assert arr[i, j, k] == val
 
-    @mark.xfail
     def test04_malloc(self):
         """Use of malloc to create multi-dim arrays"""
 
diff --git a/test/test_operators.py b/test/test_operators.py
@@ -1,6 +1,6 @@
 import py, os, sys
 from pytest import raises, skip, mark
-from .support import setup_make, pylong, maxvalue, IS_WINDOWS
+from .support import setup_make, pylong, maxvalue, IS_WINDOWS, IS_MAC, IS_CLANG_REPL
 
 currpath = py.path.local(__file__).dirpath()
 test_dct = str(currpath.join("operatorsDict"))
@@ -224,7 +224,6 @@ def test08_call_to_getsetitem_mapping(self):
             assert m[1]    == 74
             assert m(1,2)  == 74
 
-    @mark.xfail
     def test09_templated_operator(self):
         """Templated operator<()"""
 
@@ -296,7 +295,6 @@ def test11_overloaded_operators(self):
 
         assert v-w == 1-3 + 2-4
 
-    @mark.xfail
     def test12_unary_operators(self):
         """Unary operator-+~"""
 
@@ -339,7 +337,7 @@ def test14_single_argument_call(self):
         b = ns.Bar()
         assert b[42] == 42
 
-    @mark.xfail
+    @mark.xfail(condition = (IS_MAC and not IS_CLANG_REPL), reason = "Fails on OS X Cling")
     def test15_class_and_global_mix(self):
         """Iterator methods have both class and global overloads"""
 
diff --git a/test/test_regression.py b/test/test_regression.py
@@ -1063,7 +1063,6 @@ def test36_ctypes_sizeof(self):
 
         assert cppyy.sizeof(param) == ctypes.sizeof(param)
 
-    @mark.xfail
     def test37_array_of_pointers_argument(self):
         """Passing an array of pointers used to crash"""
 
@@ -1089,7 +1088,7 @@ def test37_array_of_pointers_argument(self):
 
             assert cppyy.addressof(res) == cppyy.addressof(arr)
 
-    @mark.xfail(run=not((IS_MAC_ARM or IS_MAC_X86) and not IS_CLANG_REPL))
+    @mark.xfail(run = False, condition = (IS_MAC and not IS_CLANG_REPL), reason = "Crashes on OS X Cling")
     def test38_char16_arrays(self):
         """Access to fixed-size char16 arrays as data members"""
 
diff --git a/test/test_stltypes.py b/test/test_stltypes.py
@@ -442,7 +442,7 @@ def test09_vector_of_string(self):
 
         raises(TypeError, cppyy.gbl.std.vector["std::string"], "abc")
 
-    @mark.xfail
+    @mark.xfail(condition = (IS_MAC and not IS_CLANG_REPL), reason = "Fails on OS X Cling")
     def test10_vector_std_distance(self):
         """Use of std::distance with vector"""
 
diff --git a/test/test_templates.py b/test/test_templates.py
@@ -125,7 +125,7 @@ def test03_templated_function(self):
         assert type(ggsr(vector['int']([5])).m_retval) == int
         assert ggsr(vector['int']([5])).m_retval == 5
 
-    @mark.xfail
+    @mark.xfail(condition = IS_MAC, reason = "Fails on OS X")
     def test04_variadic_function(self):
         """Call a variadic function"""
 
@@ -251,7 +251,6 @@ def test10_templated_hidding_methods(self):
         assert D().callme()  == 2
         assert D().callme(2) == 2
 
-    @mark.xfail
     def test11_templated_ctor(self):
         """Test templated constructors"""
 
@@ -317,7 +316,7 @@ def test12_template_aliases(self):
         assert nsup.Foo
         assert nsup.Bar.Foo       # used to fail
 
-    @mark.xfail
+    @mark.xfail(condition = (IS_MAC and not IS_CLANG_REPL), reason = "Fails on OS X Cling")
     def test13_using_templated_method(self):
         """Access to base class templated methods through 'using'"""
 
@@ -388,7 +387,7 @@ def test15_rvalue_templates(self):
         assert is_valid(1.)
         assert not is_valid(0.)
 
-    @mark.xfail
+    @mark.xfail(condition = IS_MAC, reason = "Fails on OS X")
     def test16_variadic(self):
         """Range of variadic templates"""
 
@@ -477,7 +476,7 @@ def test18_greedy_overloads(self):
         assert g3.get_size(ns.SomeClass()) == cppyy.sizeof(ns.SomeClass)
         assert g3.get_size(cppyy.nullptr, True) == -1
 
-    @mark.xfail
+    @mark.xfail(condition = (IS_MAC and not IS_CLANG_REPL), reason = "Fails on OS X Cling")
     def test19_templated_operator_add(self):
         """Templated operator+ is ambiguous: either __pos__ or __add__"""
 
@@ -507,7 +506,6 @@ class CustomVec {
 
         assert round(q.X() - 6.3, 8) == 0.
 
-    @mark.xfail
     def test20_templated_ctor_with_defaults(self):
         """Templated constructor with defaults used to be ignored"""
 
@@ -616,7 +614,7 @@ def callback(x):
 
         assert cppyy.gbl.std.function['double(std::vector<double>)']
 
-    @mark.xfail(run=(IS_MAC_ARM or IS_MAC_X86) and IS_CLANG_REPL)
+    @mark.xfail(run=(IS_MAC and IS_CLANG_REPL))
     def test25_stdfunction_ref_and_ptr_args(self):
         """Use of std::function with reference or pointer args"""
 
@@ -683,7 +681,6 @@ def test25_stdfunction_ref_and_ptr_args(self):
         foo.fnc = ns.bar
         foo.fnc       # <- this access used to fail
 
-    @mark.xfail
     def test26_partial_templates(self):
         """Deduction of types with partial templates"""
 
@@ -745,7 +742,6 @@ def test26_partial_templates(self):
         assert ns.bar2['double'](17) == 17
         assert ns.bar2['double','int'](17) == 17
 
-    @mark.xfail
     def test27_variadic_constructor(self):
         """Use of variadic template function as contructor"""
 
@@ -780,7 +776,7 @@ class Atom {
         a = ns.Atom(1567.0)
         assert a.m_m == 1567.0
 
-    @mark.xfail
+    @mark.xfail(condition = IS_MAC and IS_CLANG_REPL)
     def test28_enum_in_constructor(self):
         """Use of enums in template function as constructor"""
 
@@ -1124,7 +1120,7 @@ class TNaVU;
                         run_n = getattr(cppyy.gbl, 'TNaRun_%d' % n)
                         getattr(run_n, t)
 
-    @mark.xfail(run=not((IS_MAC_ARM or IS_MAC_X86) and not IS_CLANG_REPL))
+    @mark.xfail(run=False, condition=not(IS_MAC and not IS_CLANG_REPL), reason="Crashes on OS X + Cling")
     def test33_using_template_argument(self):
         """`using` type as template argument"""
 
