diff --git a/dandi/tests/fixtures.py b/dandi/tests/fixtures.py index 7f747a9a2..bad37bfe1 100644 --- a/dandi/tests/fixtures.py +++ b/dandi/tests/fixtures.py @@ -413,7 +413,6 @@ def zarr_dandiset(new_dandiset: SampleDandiset) -> SampleDandiset: @pytest.fixture() def bids_dandiset(new_dandiset: SampleDandiset, bids_examples: str) -> SampleDandiset: - shutil.copytree( os.path.join(bids_examples, "asl003"), os.path.join(str(new_dandiset.dspath), "asl003"), diff --git a/dandi/tests/test_upload.py b/dandi/tests/test_upload.py index fd0e5bbc9..3bab8ecec 100644 --- a/dandi/tests/test_upload.py +++ b/dandi/tests/test_upload.py @@ -181,10 +181,10 @@ def test_upload_sync_folder( text_dandiset.dandiset.get_asset_by_path("subdir2/banana.txt") -def test_upload_sync_bids(mocker, bids_dandiset): - confirm_mock = mocker.patch("click.confirm", return_value=True) - bids_dandiset.upload() - confirm_mock.assert_called_with("Delete 1 asset on server?") +def test_upload_bids(mocker: MockerFixture, bids_dandiset: SampleDandiset) -> None: + iter_upload_spy = mocker.spy(LocalFileAsset, "iter_upload") + bids_dandiset.upload(existing="forced") + iter_upload_spy.assert_called() def test_upload_sync_zarr(mocker, zarr_dandiset):