From a58acdc84f346ebd77a5b5ddd7a33ec8116b48e7 Mon Sep 17 00:00:00 2001 From: Paul RASCLE Date: Sat, 13 Jan 2018 14:44:54 +0100 Subject: [PATCH] fix YacsLoaderTest_swig: abort at the end of python -m unittest discover, in runUnitTests.sh. test15 in testSaveLadRun.py leads to a decrement on ref count visible only at the end on garbage collection. The decrement is visible on PythonNode.cxx when activating trace with _DEVDEBUG_. The decrement is related to the use of get() on output port instead of getPyObj() (see comment on PythonPorts.hxx). Should we remove the use of get() in SWIG or correct it in SWIG ? Use of get() as it is in Python leads to a potential abort later... --- src/yacsloader_swig/Test/testSaveLoadRun.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/yacsloader_swig/Test/testSaveLoadRun.py b/src/yacsloader_swig/Test/testSaveLoadRun.py index c81ac983b..f7ac36e70 100755 --- a/src/yacsloader_swig/Test/testSaveLoadRun.py +++ b/src/yacsloader_swig/Test/testSaveLoadRun.py @@ -1250,7 +1250,7 @@ for i in i8: ex = pilot.ExecutorSwig() self.assertEqual(p.getState(),pilot.READY) ex.RunW(p,0) - self.assertEqual(res.get(),6) + self.assertEqual(res.getPyObj(),6) self.assertEqual(p.getState(),pilot.DONE) pass -- 2.39.2