add a test to expect exception

Author: wgtmac

parquet-avro/src/test/java/org/apache/parquet/UntrustedStringableClass.java

@@ -0,0 +1,38 @@
+/**
+ * Licensed to the Apache Software Foundation (ASF) under one
+ * or more contributor license agreements.  See the NOTICE file
+ * distributed with this work for additional information
+ * regarding copyright ownership.  The ASF licenses this file
+ * to you under the Apache License, Version 2.0 (the
+ * "License"); you may not use this file except in compliance
+ * with the License.  You may obtain a copy of the License at
+ * <p>
+ * http://www.apache.org/licenses/LICENSE-2.0
+ * <p>
+ * Unless required by applicable law or agreed to in writing,
+ * software distributed under the License is distributed on an
+ * "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
+ * KIND, either express or implied.  See the License for the
+ * specific language governing permissions and limitations
+ * under the License.
+ */
+package org.apache.parquet;
+
+public class UntrustedStringableClass {
+
+  private final String value;
+
+  public UntrustedStringableClass(String value) {
+    this.value = value;
+  }
+
+  @Override
+  public String toString() {
+    return this.value;
+  }
+
+  @Override
+  public boolean equals(Object other) {
+    return other instanceof UntrustedStringableClass && this.value.equals(((UntrustedStringableClass) other).value);
+  }
+}

parquet-avro/src/test/java/org/apache/parquet/avro/TestReflectReadWrite.java

@@ -37,6 +37,7 @@
 import org.apache.avro.util.Utf8;
 import org.apache.hadoop.conf.Configuration;
 import org.apache.hadoop.fs.Path;
+import org.apache.parquet.UntrustedStringableClass;
 import org.apache.parquet.hadoop.ParquetReader;
 import org.apache.parquet.hadoop.ParquetWriter;
 import org.apache.parquet.hadoop.metadata.CompressionCodecName;
@@ -76,6 +77,40 @@ public void testWriteReflectReadGeneric() throws IOException {
     }
   }
 
+  @Test(expected = SecurityException.class)
+  public void testUntrustedStringableClass() throws IOException {
+    new AvroConverters.FieldStringableConverter(
+        new ParentValueContainer() {
+          @Override
+          public void add(Object value) {}
+
+          @Override
+          public void addBoolean(boolean value) {}
+
+          @Override
+          public void addInt(int value) {}
+
+          @Override
+          public void addLong(long value) {}
+
+          @Override
+          public void addFloat(float value) {}
+
+          @Override
+          public void addDouble(double value) {}
+
+          @Override
+          public void addChar(char value) {}
+
+          @Override
+          public void addByte(byte value) {}
+
+          @Override
+          public void addShort(short value) {}
+        },
+        UntrustedStringableClass.class);
+  }
+
   private GenericRecord getGenericPojoUtf8() {
     Schema schema = ReflectData.get().getSchema(Pojo.class);
     GenericData.Record record = new GenericData.Record(schema);