Add and update specs for IO methods

core/io/binmode_spec.rb

@@ -31,6 +31,142 @@
     @io.binmode
     @io.internal_encoding.should == nil
   end
+
+  it "disables newline conversion for #read" do
+    data = "line1\r\nline2\r\n"
+
+    @io = new_io(@name, "wb")
+    @io.write(data)
+    @io.close
+
+    @io = new_io(@name, "rt")
+    @io.set_encoding("utf-8:ISO-8859-1", newline: :universal)
+    @io.binmode
+    @io.read.should == data
+  end
+
+  it "disables newline conversion for #gets" do
+    data = "line1\r\nline2\r\n"
+
+    @io = new_io(@name, "wb")
+    @io.write(data)
+    @io.close
+
+    @io = new_io(@name, "rt")
+    @io.set_encoding("utf-8:ISO-8859-1", newline: :universal)
+    @io.binmode
+    @io.gets.should == "line1\r\n"
+    @io.gets.should == "line2\r\n"
+  end
+
+  it "disables newline conversion for #readline" do
+    data = "line1\r\nline2\r\n"
+
+    @io = new_io(@name, "wb")
+    @io.write(data)
+    @io.close
+
+    @io = new_io(@name, "rt")
+    @io.set_encoding("utf-8:ISO-8859-1", newline: :universal)
+    @io.binmode
+    @io.readline.should == "line1\r\n"
+    @io.readline.should == "line2\r\n"
+  end
+
+  it "disables newline conversion for #readlines" do
+    data = "line1\r\nline2\r\n"
+
+    @io = new_io(@name, "wb")
+    @io.write(data)
+    @io.close
+
+    @io = new_io(@name, "rt")
+    @io.set_encoding("utf-8:ISO-8859-1", newline: :universal)
+    @io.binmode
+    @io.readlines.should == ["line1\r\n", "line2\r\n"]
+  end
+
+  it "disables newline conversion for #each" do
+    data = "line1\r\nline2\r\n"
+
+    @io = new_io(@name, "wb")
+    @io.write(data)
+    @io.close
+
+    @io = new_io(@name, "rt")
+    @io.set_encoding("utf-8:ISO-8859-1", newline: :universal)
+    @io.binmode
+    @io.each.to_a.should == ["line1\r\n", "line2\r\n"]
+  end
+
+  it "disables newline conversion for #each_line" do
+    data = "line1\r\nline2\r\n"
+
+    @io = new_io(@name, "wb")
+    @io.write(data)
+    @io.close
+
+    @io = new_io(@name, "rt")
+    @io.set_encoding("utf-8:ISO-8859-1", newline: :universal)
+    @io.binmode
+    @io.each_line.to_a.should == ["line1\r\n", "line2\r\n"]
+  end
+
+  it "disables newline conversion for #getc" do
+    data = "line1\r\n"
+
+    @io = new_io(@name, "wb")
+    @io.write(data)
+    @io.close
+
+    @io = new_io(@name, "rt")
+    @io.set_encoding("utf-8:ISO-8859-1", newline: :universal)
+    @io.binmode
+    5.times { @io.getc }
+    @io.getc.should == "\r"
+    @io.getc.should == "\n"
+  end
+
+  it "disables newline conversion for #readchar" do
+    data = "line1\r\n"
+
+    @io = new_io(@name, "wb")
+    @io.write(data)
+    @io.close
+
+    @io = new_io(@name, "rt")
+    @io.set_encoding("utf-8:ISO-8859-1", newline: :universal)
+    @io.binmode
+    5.times { @io.readchar }
+    @io.readchar.should == "\r"
+    @io.readchar.should == "\n"
+  end
+
+  it "disables newline conversion for #each_char" do
+    data = "line1\r\n"
+
+    @io = new_io(@name, "wb")
+    @io.write(data)
+    @io.close
+
+    @io = new_io(@name, "rt")
+    @io.set_encoding("utf-8:ISO-8859-1", newline: :universal)
+    @io.binmode
+    @io.each_char.to_a.should == ["l", "i", "n", "e", "1", "\r", "\n"]
+  end
+
+  it "disables newline conversion for #each_codepoint" do
+    data = "line1\r\n"
+
+    @io = new_io(@name, "wb")
+    @io.write(data)
+    @io.close
+
+    @io = new_io(@name, "rt")
+    @io.set_encoding("utf-8:ISO-8859-1", newline: :universal)
+    @io.binmode
+    @io.each_codepoint.to_a.should == [108, 105, 110, 101, 49, 13, 10]
+  end
 end
 
 describe "IO#binmode?" do

core/io/getbyte_spec.rb

@@ -25,6 +25,22 @@
   it "raises an IOError on closed stream" do
     -> { IOSpecs.closed_io.getbyte }.should.raise(IOError)
   end
+
+  it "reads after ungetc without character conversion" do
+    @io.set_encoding("utf-8")
+    c = @io.getc
+    @io.ungetc(c)
+    @io.getbyte.should == 86
+  end
+
+  it "raises an exception after ungetc with character conversion" do
+    @io.set_encoding("utf-8:utf-16be")
+    c = @io.getc
+    @io.ungetc(c)
+    -> do
+      @io.getbyte
+    end.should.raise(IOError, "byte oriented read for character buffered IO")
+  end
 end
 
 describe "IO#getbyte" do

core/io/internal_encoding_spec.rb

@@ -93,6 +93,11 @@
       @io = new_io @name, "#{@object}:binary"
       @io.internal_encoding.should == nil
     end
+
+    it "returns nil when the external encoding is BINARY and internal encoding is set" do
+      @io = new_io @name, "#{@object}:binary:ibm437"
+      @io.internal_encoding.should == nil
+    end
   end
 end
 

core/io/pwrite_spec.rb

@@ -59,6 +59,12 @@
     }.should.raise(NoMethodError, /undefined method [`']to_s'/)
   end
 
+  it "raises a Errno::EINVAL if the offset is invalid" do
+    -> {
+      @file.pwrite("foo", -3)
+    }.should.raise(Errno::EINVAL)
+  end
+
   it "raises a TypeError if the offset cannot be converted to an Integer" do
     -> {
       @file.pwrite("foo", Object.new)

core/io/read_nonblock_spec.rb

@@ -66,14 +66,16 @@
     @read.read_nonblock(3).should == "bar"
   end
 
-  it "raises an exception after ungetc with data in the buffer and character conversion enabled" do
+  it "raises an exception after ungetc with character conversion enabled" do
     @write.write("foobar")
     @read.set_encoding(
       'utf-8', universal_newline: true
     )
     c = @read.getc
     @read.ungetc(c)
-    -> { @read.read_nonblock(3).should == "foo" }.should.raise(IOError)
+    -> do
+      @read.read_nonblock(3)
+    end.should.raise(IOError, "byte oriented read for character buffered IO")
   end
 
   it "returns less data if that is all that is available" do
@@ -137,6 +139,14 @@
     -> { @read.read_nonblock(5) }.should.raise(EOFError)
   end
 
+  ruby_bug "#18421", ""..."3.0.4" do
+    it "clears and returns the given buffer if the length argument is 0" do
+      buffer = String.new("existing content")
+      @read.read_nonblock(0, buffer).should == buffer
+      buffer.should == ""
+    end
+  end
+
   it "preserves the encoding of the given buffer" do
     buffer = ''.encode(Encoding::ISO_8859_1)
     @write.write("abc")

core/io/read_spec.rb

@@ -672,6 +672,12 @@
         @io.read.encoding.should.equal?(Encoding::EUC_JP)
       end
 
+      it "reads after ungetc" do
+        c = @io.getc
+        @io.ungetc(c)
+        @io.read(2).should == [164, 162].pack('C*').force_encoding(Encoding::BINARY)
+      end
+
       it_behaves_like :io_read_size_internal_encoding, nil
     end
 
@@ -680,6 +686,14 @@
         @io = IOSpecs.io_fixture "read_euc_jp.txt", "r:euc-jp:utf-8"
       end
 
+      it "raises an exception after ungetc" do
+        c = @io.getc
+        @io.ungetc(c)
+        -> do
+          @io.read(2)
+        end.should.raise(IOError, "byte oriented read for character buffered IO")
+      end
+
       it_behaves_like :io_read_internal_encoding, nil
       it_behaves_like :io_read_size_internal_encoding, nil
     end
@@ -689,6 +703,14 @@
         @io = IOSpecs.io_fixture "read_euc_jp.txt", mode: "r:euc-jp:utf-8"
       end
 
+      it "raises an exception after ungetc" do
+        c = @io.getc
+        @io.ungetc(c)
+        -> do
+          @io.read(2)
+        end.should.raise(IOError, "byte oriented read for character buffered IO")
+      end
+
       it_behaves_like :io_read_internal_encoding, nil
       it_behaves_like :io_read_size_internal_encoding, nil
     end
@@ -701,6 +723,14 @@
         @io = IOSpecs.io_fixture "read_euc_jp.txt", options
       end
 
+      it "raises an exception after ungetc" do
+        c = @io.getc
+        @io.ungetc(c)
+        -> do
+          @io.read(2)
+        end.should.raise(IOError, "byte oriented read for character buffered IO")
+      end
+
       it_behaves_like :io_read_internal_encoding, nil
       it_behaves_like :io_read_size_internal_encoding, nil
     end
@@ -711,6 +741,14 @@
         @io = IOSpecs.io_fixture "read_euc_jp.txt", options
       end
 
+      it "raises an exception after ungetc" do
+        c = @io.getc
+        @io.ungetc(c)
+        -> do
+          @io.read(2)
+        end.should.raise(IOError, "byte oriented read for character buffered IO")
+      end
+
       it_behaves_like :io_read_internal_encoding, nil
       it_behaves_like :io_read_size_internal_encoding, nil
     end

core/io/readbyte_spec.rb

@@ -21,4 +21,20 @@
       @io.readbyte
     end.should.raise EOFError
   end
+
+  it "reads after ungetc without character conversion" do
+    @io.set_encoding("utf-8")
+    c = @io.getc
+    @io.ungetc(c)
+    @io.readbyte.should == ?r.getbyte(0)
+  end
+
+  it "raises an exception after ungetc with character conversion" do
+    @io.set_encoding("utf-8:utf-16be")
+    c = @io.getc
+    @io.ungetc(c)
+    -> do
+      @io.readbyte
+    end.should.raise(IOError, "byte oriented read for character buffered IO")
+  end
 end

core/io/readpartial_spec.rb

@@ -58,6 +58,16 @@
     @rd.readpartial(2).should == "b"
   end
 
+  it "raises an exception after ungetc with character conversion enabled" do
+    @wr.write("foobar")
+    @rd.set_encoding('utf-8', 'ascii')
+    c = @rd.getc
+    @rd.ungetc(c)
+    -> do
+      @rd.readpartial(3)
+    end.should.raise(IOError, "byte oriented read for character buffered IO")
+  end
+
   it "discards the existing buffer content upon successful read" do
     buffer = +"existing content"
     @wr.write("hello world")