Skip to content

Commit 642b66f

Browse files
committed
Tweak comments
1 parent 9cfe1fc commit 642b66f

1 file changed

Lines changed: 5 additions & 5 deletions

File tree

test/test_empty_stdin.rb

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -6,11 +6,11 @@
66

77
describe Subprocess do
88
describe "communicate with empty string input" do
9-
# Bug report: subprocess.communicate("") doesn't properly
10-
# close stdin, causing you to end up a broken pipe.
9+
# Bug report: subprocess.communicate("") doesn't properly handle stdin,
10+
# causing it to close incorrectly and result in a broken pipe.
1111
it "should not raise IOError when passing empty string" do
1212
# Before the fix, this would raise: IOError: closed stream
13-
Subprocess.check_call(['cat'],
13+
Subprocess.check_call(['cat'],
1414
stdin: Subprocess::PIPE,
1515
stdout: Subprocess::PIPE) do |p|
1616
stdout, stderr = p.communicate("")
@@ -21,7 +21,7 @@
2121

2222
it "should work correctly with non-empty string input" do
2323
test_input = "hello world"
24-
Subprocess.check_call(['cat'],
24+
Subprocess.check_call(['cat'],
2525
stdin: Subprocess::PIPE,
2626
stdout: Subprocess::PIPE) do |p|
2727
stdout, stderr = p.communicate(test_input)
@@ -31,7 +31,7 @@
3131
end
3232

3333
it "should work correctly with nil input" do
34-
Subprocess.check_call(['cat'],
34+
Subprocess.check_call(['cat'],
3535
stdin: Subprocess::PIPE,
3636
stdout: Subprocess::PIPE) do |p|
3737
stdout, stderr = p.communicate(nil)

0 commit comments

Comments
 (0)