Delete non-existing injection point in TestValueTest (#4133)

Summary:
There is no injection point named
`"facebook::velox::exec::test::TestObject::internalSet"`.

This patch also fixes some typo, and has no functional change.

Pull Request resolved: #4133

Reviewed By: spershin

Differential Revision: D43740387

Pulled By: xiaoxmeng

fbshipit-source-id: 8978a8c3cc233cd4c5142194948b0430bd521a47

velox/common/testutil/tests/TestValueTest.cpp

@@ -36,10 +36,10 @@ class TestObject {
 
  private:
   int count_ = 0;
-  int value_;
+  int value_ = 0;
 };
 
-// NOTE: we can only the testvalue tests on debug build.
+// NOTE: we can only build the TestValueTest on debug build.
 #ifndef NDEBUG
 TEST(TestValueTest, testValueDisabled) {
   TestValue::disable();
@@ -61,7 +61,6 @@ TEST(TestValueTest, testValueDisabled) {
   EXPECT_EQ(0, setCount);
 
   TestValue::clear("facebook::velox::exec::test::TestObject::set");
-  TestValue::clear("facebook::velox::exec::test::TestObject::internalSet");
 
   obj.set(value);
   EXPECT_EQ(0, setValue);
@@ -128,7 +127,7 @@ TEST(TestValueTest, scopeUsageEnabled) {
       EXPECT_EQ(value, setValue);
       EXPECT_EQ(1, setCount);
     }
-    // Scoped bject dtor will clear the test value settings.
+    // Scoped object dtor will clear the test value settings.
     setCount = 0;
     setValue = 0;
     obj.set(value + 1);