Bug Fix: pq_file.schema.names(): TypeError: 'list' object is not callable (#1412)

* Reference attribute not callable

* adding test to validate feature

* fixing static checks

Author: malachi-constant

awswrangler/s3/_read_parquet.py

@@ -483,7 +483,7 @@ def _read_parquet_file(
             coerce_int96_timestamp_unit=pyarrow_args["coerce_int96_timestamp_unit"],
         )
         if validate_schema and pq_file and columns:
-            pq_file_columns: List[str] = pq_file.schema.names()
+            pq_file_columns: List[str] = pq_file.schema.names
             for column in columns:
                 if column not in pq_file_columns:
                     raise exceptions.InvalidArgument(f"column: {column} does not exist")

tests/test_s3_parquet.py

@@ -485,6 +485,12 @@ def test_timezone_raw_values(path):
     assert df2.equals(df3)
 
 
+def test_validate_columns(path) -> None:
+    wr.s3.to_parquet(pd.DataFrame({"a": [1], "b": [2]}), path, dataset=True)
+    with pytest.raises(wr.exceptions.InvalidArgument):
+        wr.s3.read_parquet(path, columns=["a", "b", "c"], validate_schema=True)
+
+
 @pytest.mark.parametrize("use_threads", [True, False, 2])
 def test_empty_column(path, use_threads):
     df = pd.DataFrame({"c0": [1, 2, 3], "c1": [None, None, None], "par": ["a", "b", "c"]})