Skip to content

Commit

Permalink
Merge pull request #20 from siboehm/fsstore_test
Browse files Browse the repository at this point in the history
Fix fs store test patching & creating directories
  • Loading branch information
adityajaroli authored Jan 12, 2024
2 parents 6baa2d5 + b73dca6 commit 52a5023
Showing 1 changed file with 5 additions and 2 deletions.
7 changes: 5 additions & 2 deletions tests/test_store_creation.py
Original file line number Diff line number Diff line change
Expand Up @@ -138,6 +138,7 @@ def test_create_store_s3(mocker):

def test_create_store_hfs(mocker):
mock_hfs = mocker.patch("storefact._hstores.HFilesystemStore")
mock_makedirs = mocker.patch("os.makedirs")
create_store(
"hfs",
{
Expand All @@ -147,11 +148,12 @@ def test_create_store_hfs(mocker):
},
)
mock_hfs.assert_called_once_with('this/is/a/relative/path')
mock_makedirs.assert_called_once_with('this/is/a/relative/path')


@pytest.mark.skip(reason="some issue here")
def test_create_store_fs(mocker):
mock_fs = mocker.patch("simplekv.fs.FilesystemStore")
mock_fs = mocker.patch("storefact._store_creation.FilesystemStore")
mock_makedirs = mocker.patch("os.makedirs")
create_store(
"fs",
{
Expand All @@ -161,6 +163,7 @@ def test_create_store_fs(mocker):
}
)
mock_fs.assert_called_once_with('this/is/a/relative/fspath')
mock_makedirs.assert_called_once_with('this/is/a/relative/fspath')


def test_create_store_mem(mocker):
Expand Down

0 comments on commit 52a5023

Please sign in to comment.