Removing explicit Excel engine configuration (#742)

* Removing explicit Excel engine configuration

* [skip ci] - clarifying docs

* Parallelizing test

Author: jaidisido

awswrangler/s3/_read_excel.py

@@ -26,6 +26,11 @@ def read_excel(
     This function accepts any Pandas's read_excel() argument.
     https://pandas.pydata.org/pandas-docs/stable/reference/api/pandas.read_excel.html
 
+    Note
+    ----
+    Depending on the file extension ('xlsx', 'xls', 'odf'...), an additional library
+    might have to be installed first (e.g. xlrd).
+
     Note
     ----
     In case of `use_threads=True` the number of threads
@@ -77,6 +82,5 @@ def read_excel(
         s3_additional_kwargs=s3_additional_kwargs,
         boto3_session=session,
     ) as f:
-        pandas_kwargs["engine"] = "openpyxl"
         _logger.debug("pandas_kwargs: %s", pandas_kwargs)
         return pd.read_excel(f, **pandas_kwargs)

awswrangler/s3/_write_excel.py

@@ -27,6 +27,11 @@ def to_excel(
     This function accepts any Pandas's read_excel() argument.
     https://pandas.pydata.org/pandas-docs/stable/reference/api/pandas.read_excel.html
 
+    Note
+    ----
+    Depending on the file extension ('xlsx', 'xls', 'odf'...), an additional library
+    might have to be installed first (e.g. xlrd).
+
     Note
     ----
     In case of `use_threads=True` the number of threads
@@ -83,7 +88,6 @@ def to_excel(
         s3_additional_kwargs=s3_additional_kwargs,
         boto3_session=session,
     ) as f:
-        pandas_kwargs["engine"] = "openpyxl"
         _logger.debug("pandas_kwargs: %s", pandas_kwargs)
         df.to_excel(f, **pandas_kwargs)
     return path

tests/test_s3_excel.py

@@ -8,10 +8,12 @@
 logging.getLogger("awswrangler").setLevel(logging.DEBUG)
 
 
+@pytest.mark.parametrize("ext", ["xlsx", "xlsm", "xls", "odf"])
 @pytest.mark.parametrize("use_threads", [True, False, 2])
-def test_excel(path, use_threads):
-    file_path = f"{path}0.xlsx"
+def test_excel(path, ext, use_threads):
     df = pd.DataFrame({"c0": [1, 2, 3], "c1": ["foo", "boo", "bar"]})
+
+    file_path = f"{path}0.{ext}"
     wr.s3.to_excel(df, file_path, use_threads=use_threads, index=False)
     df2 = wr.s3.read_excel(file_path, use_threads=use_threads)
     assert df.equals(df2)