pw_tokenizer: Remove the Vector overload for Base64 encoding

pw::Vector is not the right abstraction for this because it doesn't
guarantee that the null terminator after size() bytes stays when the
pw::Vector is copied.

Bug: 285
Change-Id: I76580d612c189b8c5a574e02a280427e0329ff53
Reviewed-on: https://pigweed-review.googlesource.com/c/pigweed/pigweed/+/22157
Reviewed-by: Armando Montanez <amontanez@google.com>
Commit-Queue: Wyatt Hepler <hepler@google.com>

1 files changed, 0 insertions, 24 deletions

diff --git a/pw_tokenizer/base64_test.cc b/pw_tokenizer/base64_test.cc
index fbbd06412..d751b8ee1 100644
--- a/pw_tokenizer/base64_test.cc
+++ b/pw_tokenizer/base64_test.cc
@@ -99,30 +99,6 @@ TEST_F(PrefixedBase64, Encode_NoRoomForNullAfterMessage_OnlyNullTerminates) {
   EXPECT_EQ(kUnset, base64_[1]);
 }
 
-TEST_F(PrefixedBase64, EncodeToVector_EmptyInput_WritesPrefix) {
-  auto buffer = PrefixedBase64Encode(std::span<byte>());
-  ASSERT_EQ(1u, buffer.size());
-  EXPECT_EQ('$', buffer[0]);
-  EXPECT_EQ('\0', buffer[1]);
-}
-
-TEST_F(PrefixedBase64, EncodeToVector_Successful) {
-  auto buffer = PrefixedBase64Encode(kTestData[5].binary);
-  ASSERT_EQ(buffer.size(), kTestData[5].base64.size());
-  EXPECT_EQ(
-      0, std::memcmp(buffer.data(), kTestData[5].base64.data(), buffer.size()));
-  EXPECT_EQ('\0', buffer[buffer.size()]);
-}
-
-TEST_F(PrefixedBase64, EncodeToVector_VectorTooSmall_OnlyNullTerminates) {
-  constexpr byte big[Base64EncodedBufferSize(
-      PW_TOKENIZER_CFG_ENCODING_BUFFER_SIZE_BYTES + 1)] = {};
-
-  auto buffer = PrefixedBase64Encode(big);
-  ASSERT_EQ(0u, buffer.size());
-  EXPECT_EQ('\0', buffer[0]);
-}
-
 TEST_F(PrefixedBase64, Base64EncodedBufferSize_Empty_RoomForPrefixAndNull) {
   EXPECT_EQ(2u, Base64EncodedBufferSize(0));
 }