Minor fixes. (#29)

* Check for Status explicitly when calling IsOk.

* Remove unused "using" statement.

* Fix tests for TRUE filter.

Author: tcsnfkx

src/test/cc/wfa/virtual_people/common/field_filter/BUILD.bazel

@@ -122,7 +122,7 @@ cc_test(
 
 cc_test(
     name = "true_filter_test",
-    srcs = ["and_filter_test.cc"],
+    srcs = ["true_filter_test.cc"],
     deps = [
         "//src/main/cc/wfa/virtual_people/common/field_filter",
         "//src/main/proto/wfa/virtual_people/common:field_filter_cc_proto",

src/test/cc/wfa/virtual_people/common/field_filter/true_filter_test.cc

@@ -27,6 +27,7 @@ namespace wfa_virtual_people {
 namespace {
 
 using ::wfa::StatusIs;
+using ::wfa_virtual_people::test::TestProto;
 
 TEST(TrueFilterTest, TestIsMatch) {
   FieldFilterProto field_filter_proto;
@@ -47,7 +48,7 @@ TEST(TrueFilterTest, TestIsMatchNotEmptyEvent) {
       FieldFilter::New(TestProto().GetDescriptor(), field_filter_proto));
 
   TestProto test_proto;
-  test_proto.set_person_country_code("COUNTRY_1");
+  test_proto.mutable_a()->mutable_b()->set_string_value("a");
   EXPECT_TRUE(field_filter->IsMatch(test_proto));
 }
 

src/test/cc/wfa/virtual_people/common/field_filter/utils/field_util_test.cc

@@ -34,7 +34,6 @@ using ::google::protobuf::Message;
 using ::testing::FieldsAre;
 using ::wfa::EqualsProto;
 using ::wfa::IsOk;
-using ::wfa::IsOkAndHolds;
 using ::wfa::StatusIs;
 using ::wfa_virtual_people::test::TestProto;
 using ::wfa_virtual_people::test::TestProtoB;