Don't suppress exception chaining for optional dependencies

doc/source/whatsnew/v1.4.0.rst

@@ -127,7 +127,8 @@ Other enhancements
 - :meth:`IntegerArray.all` , :meth:`IntegerArray.any`, :meth:`FloatingArray.any`, and :meth:`FloatingArray.all` use Kleene logic (:issue:`41967`)
 - Added support for nullable boolean and integer types in :meth:`DataFrame.to_stata`, :class:`~pandas.io.stata.StataWriter`, :class:`~pandas.io.stata.StataWriter117`, and :class:`~pandas.io.stata.StataWriterUTF8` (:issue:`40855`)
 - :meth:`DataFrame.__pos__`, :meth:`DataFrame.__neg__` now retain ``ExtensionDtype`` dtypes (:issue:`43883`)
-
+- The error raised when an optional dependency can't be imported now includes the original exception, for easier investigation (:issue:`43882`)
+-
 
 .. ---------------------------------------------------------------------------
 

pandas/compat/_optional.py

@@ -115,7 +115,7 @@ def import_optional_dependency(
         module = importlib.import_module(name)
     except ImportError:
         if errors == "raise":
-            raise ImportError(msg) from None
+            raise ImportError(msg)
         else:
             return None
 

pandas/tests/test_optional_dependency.py

@@ -13,8 +13,10 @@
 
 def test_import_optional():
     match = "Missing .*notapackage.* pip .* conda .* notapackage"
-    with pytest.raises(ImportError, match=match):
+    with pytest.raises(ImportError, match=match) as exc_info:
         import_optional_dependency("notapackage")
+    # The original exception should be there as context:
+    assert isinstance(exc_info.value.__context__, ImportError)
 
     result = import_optional_dependency("notapackage", errors="ignore")
     assert result is None