From 6f7eaecfa0ee0a6bbe5652c9c1d5cc6c367b799a Mon Sep 17 00:00:00 2001 From: Todd Gamblin Date: Sun, 22 Jul 2018 16:14:16 -0700 Subject: [PATCH] tests: correct name of `check_db` to `check_repo` - this test is checking the package *repository*, not the database. --- lib/spack/spack/test/package_sanity.py | 13 ++++++------- 1 file changed, 6 insertions(+), 7 deletions(-) diff --git a/lib/spack/spack/test/package_sanity.py b/lib/spack/spack/test/package_sanity.py index 3f56b7c6ba..0c26fd7d95 100644 --- a/lib/spack/spack/test/package_sanity.py +++ b/lib/spack/spack/test/package_sanity.py @@ -27,13 +27,12 @@ import pytest +import spack.paths import spack.repo -from spack.paths import mock_packages_path -from spack.repo import RepoPath -def check_db(): - """Get all packages in a DB to make sure they work.""" +def check_repo(): + """Get all packages in the builtin repo to make sure they work.""" for name in spack.repo.all_package_names(): spack.repo.get(name) @@ -41,14 +40,14 @@ def check_db(): @pytest.mark.maybeslow def test_get_all_packages(): """Get all packages once and make sure that works.""" - check_db() + check_repo() def test_get_all_mock_packages(): """Get the mock packages once each too.""" - db = RepoPath(mock_packages_path) + db = spack.repo.RepoPath(spack.paths.mock_packages_path) with spack.repo.swap(db): - check_db() + check_repo() def test_all_versions_are_lowercase():