diff options
author | Joe Onorato <joeo@google.com> | 2019-05-10 16:33:49 -0700 |
---|---|---|
committer | Joe Onorato <joeo@google.com> | 2019-05-16 00:51:18 +0000 |
commit | 05270525ee11c38de50c7db2fcf03763960eb1fa (patch) | |
tree | 296d4565be8de148a641bbd8163f5016620e5b60 | |
parent | 373c37f7eccbb91a25ff2c45b4c9b25759f907f1 (diff) |
Fix EncodedBuffer_test to run after making EncodedBuffer inherit from RefBase
Test: bit libprotoutil_test:*
Bug: 129717537
Change-Id: I44cf2e631a5ac1bb8b4ee2c269cfc50438fb1782
-rw-r--r-- | libs/protoutil/tests/EncodedBuffer_test.cpp | 122 |
1 files changed, 61 insertions, 61 deletions
diff --git a/libs/protoutil/tests/EncodedBuffer_test.cpp b/libs/protoutil/tests/EncodedBuffer_test.cpp index 398af609c083..f895154c4983 100644 --- a/libs/protoutil/tests/EncodedBuffer_test.cpp +++ b/libs/protoutil/tests/EncodedBuffer_test.cpp @@ -29,101 +29,101 @@ static void expectPointer(EncodedBuffer::Pointer* p, size_t pos) { } TEST(EncodedBufferTest, WriteSimple) { - EncodedBuffer buffer(TEST_CHUNK_SIZE); - EXPECT_EQ(buffer.size(), 0UL); - expectPointer(buffer.wp(), 0); - EXPECT_EQ(buffer.currentToWrite(), TEST_CHUNK_SIZE); + sp<EncodedBuffer> buffer = new EncodedBuffer(TEST_CHUNK_SIZE); + EXPECT_EQ(buffer->size(), 0UL); + expectPointer(buffer->wp(), 0); + EXPECT_EQ(buffer->currentToWrite(), TEST_CHUNK_SIZE); for (size_t i = 0; i < TEST_CHUNK_HALF_SIZE; i++) { - buffer.writeRawByte(50 + i); + buffer->writeRawByte(50 + i); } - EXPECT_EQ(buffer.size(), TEST_CHUNK_HALF_SIZE); - expectPointer(buffer.wp(), TEST_CHUNK_HALF_SIZE); - EXPECT_EQ(buffer.currentToWrite(), TEST_CHUNK_HALF_SIZE); + EXPECT_EQ(buffer->size(), TEST_CHUNK_HALF_SIZE); + expectPointer(buffer->wp(), TEST_CHUNK_HALF_SIZE); + EXPECT_EQ(buffer->currentToWrite(), TEST_CHUNK_HALF_SIZE); for (size_t i = 0; i < TEST_CHUNK_SIZE; i++) { - buffer.writeRawByte(80 + i); + buffer->writeRawByte(80 + i); } - EXPECT_EQ(buffer.size(), TEST_CHUNK_SIZE + TEST_CHUNK_HALF_SIZE); - expectPointer(buffer.wp(), TEST_CHUNK_SIZE + TEST_CHUNK_HALF_SIZE); - EXPECT_EQ(buffer.currentToWrite(), TEST_CHUNK_HALF_SIZE); + EXPECT_EQ(buffer->size(), TEST_CHUNK_SIZE + TEST_CHUNK_HALF_SIZE); + expectPointer(buffer->wp(), TEST_CHUNK_SIZE + TEST_CHUNK_HALF_SIZE); + EXPECT_EQ(buffer->currentToWrite(), TEST_CHUNK_HALF_SIZE); // verifies the buffer's data - expectPointer(buffer.ep(), 0); + expectPointer(buffer->ep(), 0); for (size_t i = 0; i < TEST_CHUNK_HALF_SIZE; i++) { - EXPECT_EQ(buffer.readRawByte(), 50 + i); + EXPECT_EQ(buffer->readRawByte(), 50 + i); } for (size_t i = 0; i < TEST_CHUNK_SIZE; i++) { - EXPECT_EQ(buffer.readRawByte(), 80 + i); + EXPECT_EQ(buffer->readRawByte(), 80 + i); } // clears the buffer - buffer.clear(); - EXPECT_EQ(buffer.size(), 0UL); - expectPointer(buffer.wp(), 0); + buffer->clear(); + EXPECT_EQ(buffer->size(), 0UL); + expectPointer(buffer->wp(), 0); } TEST(EncodedBufferTest, WriteVarint) { - EncodedBuffer buffer(TEST_CHUNK_SIZE); + sp<EncodedBuffer> buffer = new EncodedBuffer(TEST_CHUNK_SIZE); size_t expected_buffer_size = 0; - EXPECT_EQ(buffer.writeRawVarint32(13), 1); + EXPECT_EQ(buffer->writeRawVarint32(13), 1); expected_buffer_size += 1; - EXPECT_EQ(buffer.size(), expected_buffer_size); - EXPECT_EQ(buffer.writeRawVarint32(UINT32_C(-1)), 5); + EXPECT_EQ(buffer->size(), expected_buffer_size); + EXPECT_EQ(buffer->writeRawVarint32(UINT32_C(-1)), 5); expected_buffer_size += 5; - EXPECT_EQ(buffer.size(), expected_buffer_size); + EXPECT_EQ(buffer->size(), expected_buffer_size); - EXPECT_EQ(buffer.writeRawVarint64(200), 2); + EXPECT_EQ(buffer->writeRawVarint64(200), 2); expected_buffer_size += 2; - EXPECT_EQ(buffer.size(), expected_buffer_size); - EXPECT_EQ(buffer.writeRawVarint64(UINT64_C(-1)), 10); + EXPECT_EQ(buffer->size(), expected_buffer_size); + EXPECT_EQ(buffer->writeRawVarint64(UINT64_C(-1)), 10); expected_buffer_size += 10; - EXPECT_EQ(buffer.size(), expected_buffer_size); + EXPECT_EQ(buffer->size(), expected_buffer_size); - buffer.writeRawFixed32(UINT32_C(-1)); + buffer->writeRawFixed32(UINT32_C(-1)); expected_buffer_size += 4; - EXPECT_EQ(buffer.size(), expected_buffer_size); - buffer.writeRawFixed64(UINT64_C(-1)); + EXPECT_EQ(buffer->size(), expected_buffer_size); + buffer->writeRawFixed64(UINT64_C(-1)); expected_buffer_size += 8; - EXPECT_EQ(buffer.size(), expected_buffer_size); + EXPECT_EQ(buffer->size(), expected_buffer_size); - EXPECT_EQ(buffer.writeHeader(32, 2), 2); + EXPECT_EQ(buffer->writeHeader(32, 2), 2); expected_buffer_size += 2; - EXPECT_EQ(buffer.size(), expected_buffer_size); + EXPECT_EQ(buffer->size(), expected_buffer_size); // verify data are correctly written to the buffer. - expectPointer(buffer.ep(), 0); - EXPECT_EQ(buffer.readRawVarint(), UINT32_C(13)); - EXPECT_EQ(buffer.readRawVarint(), UINT32_C(-1)); - EXPECT_EQ(buffer.readRawVarint(), UINT64_C(200)); - EXPECT_EQ(buffer.readRawVarint(), UINT64_C(-1)); - EXPECT_EQ(buffer.readRawFixed32(), UINT32_C(-1)); - EXPECT_EQ(buffer.readRawFixed64(), UINT64_C(-1)); - EXPECT_EQ(buffer.readRawVarint(), UINT64_C((32 << 3) + 2)); - expectPointer(buffer.ep(), expected_buffer_size); + expectPointer(buffer->ep(), 0); + EXPECT_EQ(buffer->readRawVarint(), UINT32_C(13)); + EXPECT_EQ(buffer->readRawVarint(), UINT32_C(-1)); + EXPECT_EQ(buffer->readRawVarint(), UINT64_C(200)); + EXPECT_EQ(buffer->readRawVarint(), UINT64_C(-1)); + EXPECT_EQ(buffer->readRawFixed32(), UINT32_C(-1)); + EXPECT_EQ(buffer->readRawFixed64(), UINT64_C(-1)); + EXPECT_EQ(buffer->readRawVarint(), UINT64_C((32 << 3) + 2)); + expectPointer(buffer->ep(), expected_buffer_size); } TEST(EncodedBufferTest, Edit) { - EncodedBuffer buffer(TEST_CHUNK_SIZE); - buffer.writeRawFixed64(0xdeadbeefdeadbeef); - EXPECT_EQ(buffer.readRawFixed64(), UINT64_C(0xdeadbeefdeadbeef)); + sp<EncodedBuffer> buffer = new EncodedBuffer(TEST_CHUNK_SIZE); + buffer->writeRawFixed64(0xdeadbeefdeadbeef); + EXPECT_EQ(buffer->readRawFixed64(), UINT64_C(0xdeadbeefdeadbeef)); - buffer.editRawFixed32(4, 0x12345678); + buffer->editRawFixed32(4, 0x12345678); // fixed 64 is little endian order. - buffer.ep()->rewind(); // rewind ep for readRawFixed64 from 0 - EXPECT_EQ(buffer.readRawFixed64(), UINT64_C(0x12345678deadbeef)); - - buffer.wp()->rewind(); - expectPointer(buffer.wp(), 0); - buffer.copy(4, 3); - buffer.ep()->rewind(); // rewind ep for readRawFixed64 from 0 - EXPECT_EQ(buffer.readRawFixed64(), UINT64_C(0x12345678de345678)); + buffer->ep()->rewind(); // rewind ep for readRawFixed64 from 0 + EXPECT_EQ(buffer->readRawFixed64(), UINT64_C(0x12345678deadbeef)); + + buffer->wp()->rewind(); + expectPointer(buffer->wp(), 0); + buffer->copy(4, 3); + buffer->ep()->rewind(); // rewind ep for readRawFixed64 from 0 + EXPECT_EQ(buffer->readRawFixed64(), UINT64_C(0x12345678de345678)); } TEST(EncodedBufferTest, ReadSimple) { - EncodedBuffer buffer(TEST_CHUNK_SIZE); + sp<EncodedBuffer> buffer = new EncodedBuffer(TEST_CHUNK_SIZE); for (size_t i = 0; i < TEST_CHUNK_3X_SIZE; i++) { - buffer.writeRawByte(i); + buffer->writeRawByte(i); } - sp<ProtoReader> reader1 = buffer.read(); + sp<ProtoReader> reader1 = buffer->read(); EXPECT_EQ(reader1->size(), TEST_CHUNK_3X_SIZE); EXPECT_EQ(reader1->bytesRead(), 0); @@ -132,7 +132,7 @@ TEST(EncodedBufferTest, ReadSimple) { } EXPECT_EQ(reader1->bytesRead(), TEST_CHUNK_3X_SIZE); - sp<ProtoReader> reader2 = buffer.read(); + sp<ProtoReader> reader2 = buffer->read(); uint8_t val = 0; while (reader2->hasNext()) { EXPECT_EQ(reader2->next(), val); @@ -143,10 +143,10 @@ TEST(EncodedBufferTest, ReadSimple) { } TEST(EncodedBufferTest, ReadVarint) { - EncodedBuffer buffer; + sp<EncodedBuffer> buffer = new EncodedBuffer(); uint64_t val = UINT64_C(1522865904593); - size_t len = buffer.writeRawVarint64(val); - sp<ProtoReader> reader = buffer.read(); + size_t len = buffer->writeRawVarint64(val); + sp<ProtoReader> reader = buffer->read(); EXPECT_EQ(reader->size(), len); EXPECT_EQ(reader->readRawVarint(), val); } |