Skip to content

Commit

Permalink
Apply requests from review
Browse files Browse the repository at this point in the history
  • Loading branch information
benibus committed Sep 22, 2022
1 parent 93fa7dc commit af17b3d
Showing 1 changed file with 3 additions and 2 deletions.
5 changes: 3 additions & 2 deletions cpp/src/arrow/filesystem/test_util.cc
Original file line number Diff line number Diff line change
Expand Up @@ -979,6 +979,7 @@ void GenericFileSystemTest::TestOpenInputStream(FileSystem* fs) {
ASSERT_OK_AND_ASSIGN(buffer, stream->Read(1));
AssertBufferEqual(*buffer, "");
ASSERT_OK_AND_EQ(9, stream->Tell());
ASSERT_OK(stream->Close());

ASSERT_OK_AND_ASSIGN(stream, fs->OpenInputStream("AB/abc"));
ASSERT_OK(stream->Advance(4));
Expand Down Expand Up @@ -1077,8 +1078,8 @@ void GenericFileSystemTest::TestOpenInputFile(FileSystem* fs) {
ASSERT_OK_AND_ASSIGN(buffer, file->Read(6));
AssertBufferEqual(*buffer, "other ");
// Should return the same slice independent of the current position
ASSERT_OK_AND_ASSIGN(buffer, file->ReadAt(0, 4));
AssertBufferEqual(*buffer, "some");
ASSERT_OK_AND_ASSIGN(buffer, file->ReadAt(2, 3));
AssertBufferEqual(*buffer, "me ");
ASSERT_OK_AND_EQ(15, file->GetSize());
ASSERT_OK(file->Close());
ASSERT_RAISES(Invalid, file->ReadAt(1, 1)); // Stream is closed
Expand Down

0 comments on commit af17b3d

Please sign in to comment.