Merge pull request #314 from dandi/close-file

Close a file in a test case
dandi · Dec 14, 2020 · cc4a03c · cc4a03c
2 parents 8afaaaf + fd13c75
commit cc4a03c
Showing 1 changed file with 2 additions and 1 deletion.
diff --git a/dandi/tests/test_organize.py b/dandi/tests/test_organize.py
@@ -63,7 +63,8 @@ def c():  # shortcut
     # and if we set units and redo -- years should stay unchanged, while other fields change
     m = yaml_load(path.read())
     m["age"]["units"] = "years"
-    ruamel.yaml.YAML().dump(m, open(path, "w"))
+    with open(path, "w") as fp:
+        ruamel.yaml.YAML().dump(m, fp)
 
     populate_dataset_yml(str(path), metadata[:1])
     assert c() == {