Update fmriprep/utils/tests/test_derivative_cache.py

psadil · effigies · web-flow · commit bef2a21de2db · 2024-10-02T09:29:42.000-04:00
Simplify assertion in test for loaded transforms

Co-authored-by: Chris Markiewicz &lt;effigies@gmail.com&gt;
diff --git a/fmriprep/utils/tests/test_derivative_cache.py b/fmriprep/utils/tests/test_derivative_cache.py
@@ -29,7 +29,4 @@ def test_transforms_found_as_str(tmp_path: Path, xfm: str):
         entities=entities,
         fieldmap_id='auto_00000',
     )
-    transforms_in_derivs = 'transforms' in derivs
-    xfm_in_transforms = xfm in derivs.get('transforms')
-    transform_is_str = isinstance(derivs.get('transforms').get(xfm), str)
-    assert all((transforms_in_derivs, xfm_in_transforms, transform_is_str))
+    assert derivs == {'transforms': {xfm: str(to_find)}}

Original file line number	Diff line number	Diff line change
`@@ -29,7 +29,4 @@ def test_transforms_found_as_str(tmp_path: Path, xfm: str):`
`29`	`29`	`entities=entities,`
`30`	`30`	`fieldmap_id='auto_00000',`
`31`	`31`	`)`
`32`		`- transforms_in_derivs = 'transforms' in derivs`
`33`		`- xfm_in_transforms = xfm in derivs.get('transforms')`
`34`		`- transform_is_str = isinstance(derivs.get('transforms').get(xfm), str)`
`35`		`- assert all((transforms_in_derivs, xfm_in_transforms, transform_is_str))`
	`32`	`+ assert derivs == {'transforms': {xfm: str(to_find)}}`