Changeset f354e46 in sasmodels

Timestamp:

Jan 31, 2018 2:21:33 PM (7 years ago)

Author:

Paul Kienzle <pkienzle@…>

Branches:

master, core_shell_microgels, magnetic_model, ticket-1257-vesicle-product, ticket_1156, ticket_1265_superball, ticket_822_more_unit_tests

Children:

335271e

Parents:

e59f60a

Message:

yield tests: need function scope to capture context correctly

File:

• sasmodels/model_test.py (modified) (2 diffs)

sasmodels/model_test.py

@@ -160 +160 @@
 
     return suite
-
 
 def _hide_model_case_from_nose():
@@ -469 +468 @@
     loaders = ['opencl', 'dll'] if core.HAVE_OPENCL else ['dll']
     tests = make_suite(loaders, ['all'])
-    for test in tests:
+    def build_test(test):
         # In order for nosetest to show the test name, wrap the test.run_all
         # instance in function that takes the test name as a parameter which
-        # will be displayed when the test is run.
-        yield lambda name: test.run_all(), test.test_name
+        # will be displayed when the test is run.  Do this as a function so
+        # that it properly captures the context for tests that captured and
+        # run later.  If done directly in the for loop, then the looping
+        # variable test will be shared amongst all the tests, and we will be
+        # repeatedly testing vesicle.
+        return lambda name: test.run_all(), test.test_name
+    for test in tests:
+        yield build_test(test)