put_file: default to overwrite=True (#419)

* put_file: default to overwrite=True

* changelog: document _put_file(overwrite=True) behavior change

* update tests

CHANGELOG.md

@@ -4,6 +4,7 @@ Unreleased
 ----------
 
 - Handle mixed casing for `hdi_isfolder` metadata when determining whether a blob should be treated as a folder.
+- `_put_file`: `overwrite` now defaults to `True`.
 
 2023.10.0
 ---------

adlfs/spec.py

@@ -1548,18 +1548,19 @@ async def _put_file(
         lpath,
         rpath,
         delimiter="/",
-        overwrite=False,
+        overwrite=True,
         callback=None,
         max_concurrency=None,
-        **kwargws,
+        **kwargs,
     ):
         """
         Copy single file to remote
 
         :param lpath: Path to local file
         :param rpath: Path to remote file
         :param delimitier: Filepath delimiter
-        :param overwrite: Boolean (False).  If True, overwrite the existing file present
+        :param overwrite: Boolean (True). Whether to overwrite any existing file
+            (True) or raise if one already exists (False).
         """
 
         container_name, path, _ = self.split_path(rpath, delimiter=delimiter)

adlfs/tests/test_spec.py

@@ -1854,7 +1854,7 @@ async def test_put_file_timeout(storage, mocker, tmp_path):
     upload_blob.assert_called_once_with(
         mocker.ANY,
         metadata={"is_directory": "false"},
-        overwrite=False,
+        overwrite=True,
         raw_response_hook=None,
         max_concurrency=fs.max_concurrency,
         timeout=11,