Fix dlexpect (#185)

* Added download_path to DownloadExpectation

* Added change to CHANGELOG.md

---------

Co-authored-by: Mads Hangaard <[email protected]>
Co-authored-by: Stephan Lensky <[email protected]>

Author: 3 people

CHANGELOG.md

@@ -10,6 +10,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 ### Fixed
 
 - Fix for calls to `evaluate` that return JSON: only set `serialization_options` when `return_by_value=False` @thromer
+- Fix download_path error in `DownloadExpectation` (https://github.com/cdpdriver/zendriver/issues/184) @MadsHangaard
 
 ### Added
 

zendriver/core/expect.py

@@ -202,6 +202,11 @@ def __init__(self, tab: Connection):
         self.default_behavior = (
             self.tab._download_behavior[0] if self.tab._download_behavior else "default"
         )
+        self.download_path = (
+            self.tab._download_behavior[1]
+            if self.tab._download_behavior and len(self.tab._download_behavior) > 1
+            else None
+        )
 
     async def _handler(self, event: cdp.browser.DownloadWillBegin) -> None:
         self._remove_handler()
@@ -225,7 +230,9 @@ async def __aexit__(self, *args: Any) -> None:
         Exit the context manager, removing handler, set download behavior to default.
         """
         await self.tab.send(
-            cdp.browser.set_download_behavior(behavior=self.default_behavior)
+            cdp.browser.set_download_behavior(
+                behavior=self.default_behavior, download_path=self.download_path
+            )
         )
         self._remove_handler()