ENH: improve optional import error message (#61597)

* improve optional import error message

* add fix on tests

* fix tests

* fix test_parquest tests

Author: KevsterAmp

pandas/compat/_optional.py

@@ -152,8 +152,8 @@ def import_optional_dependency(
     install_name = package_name if package_name is not None else name
 
     msg = (
-        f"Missing optional dependency '{install_name}'. {extra} "
-        f"Use pip or conda to install {install_name}."
+        f"`Import {install_name}` failed. {extra} "
+        f"Use pip or conda to install the {install_name} package."
     )
     try:
         module = importlib.import_module(name)

pandas/tests/io/test_fsspec.py

@@ -282,7 +282,7 @@ def test_s3_parquet(s3_public_bucket, s3so, df1):
 
 @td.skip_if_installed("fsspec")
 def test_not_present_exception():
-    msg = "Missing optional dependency 'fsspec'|fsspec library is required"
+    msg = "`Import fsspec` failed.  Use pip or conda to install the fsspec package."
     with pytest.raises(ImportError, match=msg):
         read_csv("memory://test/test.csv")
 

pandas/tests/io/test_parquet.py

@@ -325,7 +325,7 @@ def test_get_engine_auto_error_message():
             with pytest.raises(ImportError, match=match):
                 get_engine("auto")
         else:
-            match = "Missing optional dependency .pyarrow."
+            match = "Unable to find a usable engine; tried using: 'pyarrow'"
             with pytest.raises(ImportError, match=match):
                 get_engine("auto")
 
@@ -334,7 +334,7 @@ def test_get_engine_auto_error_message():
             with pytest.raises(ImportError, match=match):
                 get_engine("auto")
         else:
-            match = "Missing optional dependency .fastparquet."
+            match = "Use pip or conda to install the fastparquet package"
             with pytest.raises(ImportError, match=match):
                 get_engine("auto")
 
@@ -1133,9 +1133,9 @@ def test_string_inference(self, tmp_path, pa, using_infer_string):
             index=pd.Index(["a", "b"], dtype=dtype),
             columns=pd.Index(
                 ["a"],
-                dtype=object
-                if pa_version_under19p0 and not using_infer_string
-                else dtype,
+                dtype=(
+                    object if pa_version_under19p0 and not using_infer_string else dtype
+                ),
             ),
         )
         tm.assert_frame_equal(result, expected)

pandas/tests/test_optional_dependency.py

@@ -12,7 +12,7 @@
 
 
 def test_import_optional():
-    match = "Missing .*notapackage.* pip .* conda .* notapackage"
+    match = "Import .*notapackage.* pip .* conda .* notapackage"
     with pytest.raises(ImportError, match=match) as exc_info:
         import_optional_dependency("notapackage")
     # The original exception should be there as context:

pandas/tests/util/test_numba.py

@@ -7,6 +7,6 @@
 
 @td.skip_if_installed("numba")
 def test_numba_not_installed_option_context():
-    with pytest.raises(ImportError, match="Missing optional"):
+    with pytest.raises(ImportError, match="`Import numba` failed"):
         with option_context("compute.use_numba", True):
             pass