Skip to content

Commit ffbba91

Browse files
NicolasHugfacebook-github-bot
authored andcommitted
[fbsync] test_prototype_datasets_builtin: Properly close all streams (#7403)
Summary: Co-authored-by: Philip Meier <[email protected]> Reviewed By: vmoens Differential Revision: D44416630 fbshipit-source-id: efee1b0849c7e19ba1859f185bff421400046a12
1 parent 30b3418 commit ffbba91

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

test/test_prototype_datasets_builtin.py

+1-1
Original file line numberDiff line numberDiff line change
@@ -123,7 +123,7 @@ def log_session_streams(self):
123123
def test_stream_closing(self, log_session_streams, dataset_mock, config):
124124
def make_msg_and_close(head):
125125
unclosed_streams = []
126-
for stream in StreamWrapper.session_streams.keys():
126+
for stream in list(StreamWrapper.session_streams.keys()):
127127
unclosed_streams.append(repr(stream.file_obj))
128128
stream.close()
129129
unclosed_streams = "\n".join(unclosed_streams)

0 commit comments

Comments
 (0)