From dc334a7c8f9ff232342ad3feab3daa97191431a8 Mon Sep 17 00:00:00 2001
From: Elliott Hughes <enh@google.com>
Date: Tue, 11 May 2021 16:27:15 -0700
Subject: [PATCH] Remove String16::remove tests.

These tests are only in internal right now, preventing me from removing
this function entirely. So let's get them out of the way...

Test: treehugger
Ignore-AOSP-First: workaround for presubmit merge conflict
Bug: http://b/35267372
Change-Id: Ib5de178ffad833c87e8e93bfd33e85a0ff504ac0
---
 libutils/String16_test.cpp | 29 -----------------------------
 1 file changed, 29 deletions(-)

diff --git a/libutils/String16_test.cpp b/libutils/String16_test.cpp
index 788da8e0e..54662ac41 100644
--- a/libutils/String16_test.cpp
+++ b/libutils/String16_test.cpp
@@ -90,27 +90,6 @@ TEST(String16Test, Insert) {
     EXPECT_STR16EQ(u"VerifyInsert me", tmp);
 }
 
-TEST(String16Test, RemoveDefault) {
-    String16 tmp("Verify me");
-    tmp.remove(4);
-    EXPECT_EQ(4U, tmp.size());
-    EXPECT_STR16EQ(u"Veri", tmp);
-}
-
-TEST(String16Test, Remove) {
-    String16 tmp("Verify me");
-    tmp.remove(2, 6);
-    EXPECT_EQ(2U, tmp.size());
-    EXPECT_STR16EQ(u" m", tmp);
-}
-
-TEST(String16Test, RemoveOutOfBounds) {
-    String16 tmp("Verify me");
-    tmp.remove(100, 6);
-    EXPECT_EQ(3U, tmp.size());
-    EXPECT_STR16EQ(u" me", tmp);
-}
-
 TEST(String16Test, ReplaceAll) {
     String16 tmp("Verify verify Verify");
     tmp.replaceAll(u'r', u'!');
@@ -175,14 +154,6 @@ TEST(String16Test, StaticStringInsert) {
     EXPECT_FALSE(tmp.isStaticString());
 }
 
-TEST(String16Test, StaticStringRemove) {
-    StaticString16 tmp(u"Verify me");
-    tmp.remove(2, 6);
-    EXPECT_EQ(2U, tmp.size());
-    EXPECT_STR16EQ(u" m", tmp);
-    EXPECT_FALSE(tmp.isStaticString());
-}
-
 TEST(String16Test, StaticStringReplaceAll) {
     StaticString16 tmp(u"Verify verify Verify");
     tmp.replaceAll(u'r', u'!');