zhangyiss/spack
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Delete unused "mutable_mock_store" fixture (#21632)

Browse Source 
The fixture was introduced in #19690 maybe accidentally.

It's not used in unit tests, and though it should be
mutable it seems an exact copy of it's immutable version.
This commit is contained in:
Massimiliano Culpo
2021-02-17 02:26:16 +01:00
committed by GitHub
parent 5b0507cc65
commit ee1b0b9fb9
1 changed files with 0 additions and 31 deletions
Download Patch File Download Diff File Expand all files Collapse all files

31
lib/spack/spack/test/conftest.py
View File

@@ -610,37 +610,6 @@ def mock_store(tmpdir_factory, mock_repo_path, mock_configuration_scopes,
store_path.join('.spack-db').chmod(mode=0o755, rec=1) store_path.join('.spack-db').chmod(mode=0o755, rec=1)
@pytest.fixture(scope='function')
def mutable_mock_store(
tmpdir_factory, mock_repo_path, mock_configuration_scopes,
_store_dir_and_cache
):
"""Creates a read-only mock database with some packages installed note
that the ref count for dyninst here will be 3, as it's recycled
across each install.
This does not actually activate the store for use by Spack -- see the
``database`` fixture for that.
"""
store_path, store_cache = _store_dir_and_cache
# If the cache does not exist populate the store and create it
if not os.path.exists(str(store_cache.join('.spack-db'))):
with spack.config.use_configuration(*mock_configuration_scopes):
with spack.store.use_store(str(store_path)) as store:
with spack.repo.use_repositories(mock_repo_path):
_populate(store.db)
store_path.copy(store_cache, mode=True, stat=True)
# Make the DB filesystem read-only to ensure we can't modify entries
store_path.join('.spack-db').chmod(mode=0o555, rec=1)
yield store
store_path.join('.spack-db').chmod(mode=0o755, rec=1)
@pytest.fixture(scope='function') @pytest.fixture(scope='function')
def database(mock_store, mock_packages, config, monkeypatch): def database(mock_store, mock_packages, config, monkeypatch):
"""This activates the mock store, packages, AND config.""" """This activates the mock store, packages, AND config."""