use a context manager

Author: tfeldmann

fs/test.py

@@ -1757,19 +1757,19 @@ def test_move_dir_same_fs(self):
 
     def _test_move_dir_write(self, protocol):
         # Test moving to this filesystem from another.
-        other_fs = open_fs(protocol)
-        other_fs.makedirs("foo/bar/baz")
-        other_fs.makedir("egg")
-        other_fs.writetext("top.txt", "Hello, World")
-        other_fs.writetext("/foo/bar/baz/test.txt", "Goodbye, World")
-
-        fs.move.move_dir(other_fs, "/", self.fs, "/")
-
-        expected = {"/egg", "/foo", "/foo/bar", "/foo/bar/baz"}
-        self.assertEqual(other_fs.listdir("/"), [])
-        self.assertEqual(set(walk.walk_dirs(self.fs)), expected)
-        self.assert_text("top.txt", "Hello, World")
-        self.assert_text("/foo/bar/baz/test.txt", "Goodbye, World")
+        with open_fs(protocol) as other_fs:
+            other_fs.makedirs("foo/bar/baz")
+            other_fs.makedir("egg")
+            other_fs.writetext("top.txt", "Hello, World")
+            other_fs.writetext("/foo/bar/baz/test.txt", "Goodbye, World")
+
+            fs.move.move_dir(other_fs, "/", self.fs, "/")
+
+            expected = {"/egg", "/foo", "/foo/bar", "/foo/bar/baz"}
+            self.assertEqual(other_fs.listdir("/"), [])
+            self.assertEqual(set(walk.walk_dirs(self.fs)), expected)
+            self.assert_text("top.txt", "Hello, World")
+            self.assert_text("/foo/bar/baz/test.txt", "Goodbye, World")
 
     def test_move_dir_mem(self):
         self._test_move_dir_write("mem://")