Merge pull request #3402 from Xarthisius/fix_small_patch

Change the loop order to avoid executing the same test 3 times
yt-project · Jul 7, 2021 · 26c32e9 · 26c32e9
2 parents 7cc5f89 + 20cceed
commit 26c32e9
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/yt/utilities/answer_testing/framework.py b/yt/utilities/answer_testing/framework.py
@@ -1193,13 +1193,13 @@ def small_patch_amr(ds_fn, fields, input_center="max", input_weight=("gas", "den
     yield ParentageRelationshipsTest(ds_fn)
     for field in fields:
         yield GridValuesTest(ds_fn, field)
-        for axis in [0, 1, 2]:
-            for dobj_name in dso:
+        for dobj_name in dso:
+            for axis in [0, 1, 2]:
                 for weight_field in [None, input_weight]:
                     yield ProjectionValuesTest(
                         ds_fn, axis, field, weight_field, dobj_name
                     )
-                yield FieldValuesTest(ds_fn, field, dobj_name)
+            yield FieldValuesTest(ds_fn, field, dobj_name)
 
 
 def big_patch_amr(ds_fn, fields, input_center="max", input_weight=("gas", "density")):