Fixup ignore_empty oneof tests + add tests for repeated items and map…

… keys/values (#128)

Oneof tests were redundant (and contradictory to the behavior we're
going for), so removed them in favor of the more exhaustive ones in the
ignore_empty suite. Likewise, added tests to ensure the behavior of
ignore_empty as applied to repeated items and map keys/values is
consistent as well.

Relates to #115

proto/protovalidate-testing/buf/validate/conformance/cases/ignore_empty_proto3.proto

@@ -68,3 +68,23 @@ message IgnoreEmptyProto3Map {
     (buf.validate.field).map.min_pairs = 3
   ];
 }
+
+message IgnoreEmptyRepeatedItems {
+  repeated int32 val = 1 [(buf.validate.field).repeated.items = {
+    ignore_empty: true,
+    int32: {gt: 0},
+  }];
+}
+
+message IgnoreEmptyMapPairs {
+  map<string, int32> val = 1 [
+    (buf.validate.field).map.keys = {
+      ignore_empty: true,
+      string: {min_len: 3},
+    },
+    (buf.validate.field).map.values = {
+      ignore_empty: true,
+      int32: {gt: 0},
+    }
+  ];
+}

proto/protovalidate-testing/buf/validate/conformance/cases/oneofs.proto

@@ -56,29 +56,3 @@ message OneofRequiredWithRequiredField {
     string b = 2;
   }
 }
-
-message OneofIgnoreEmpty {
-  oneof o {
-    string x = 1 [
-      (buf.validate.field).string = {
-        min_len: 3,
-        max_len: 5
-      },
-      (buf.validate.field).ignore_empty = true
-    ];
-    bytes y = 2 [
-      (buf.validate.field).bytes = {
-        min_len: 3,
-        max_len: 5
-      },
-      (buf.validate.field).ignore_empty = true
-    ];
-    int32 z = 3 [
-      (buf.validate.field).int32 = {
-        gte: 128,
-        lte: 256,
-      },
-      (buf.validate.field).ignore_empty = true
-    ];
-  }
-}