From ab04d697c7ec6599411c1d7bb9da6e3816df42b1 Mon Sep 17 00:00:00 2001 From: Todd Gamblin Date: Sun, 31 Jul 2022 17:07:23 -0700 Subject: tests: remove old `changed_files()` test This test relied on an old version of the `flake8_package` fixture that modified the spack repository, but it doesn't do that anymore. There are other tests for `changed_files()` that do a better job of mocking up a git repository with changes, so we can just delete this one. --- lib/spack/spack/test/cmd/style.py | 10 ---------- 1 file changed, 10 deletions(-) diff --git a/lib/spack/spack/test/cmd/style.py b/lib/spack/spack/test/cmd/style.py index 7a5926ae29..dec795ca99 100644 --- a/lib/spack/spack/test/cmd/style.py +++ b/lib/spack/spack/test/cmd/style.py @@ -87,16 +87,6 @@ def flake8_package_with_errors(scope="function"): shutil.move(tmp, filename) -def test_changed_files(flake8_package): - # changed_files returns file paths relative to the root - # directory of Spack. Convert to absolute file paths. - files = [os.path.join(spack.paths.prefix, os.path.normpath(path)) for path in changed_files()] - - # There will likely be other files that have changed - # when these tests are run - assert flake8_package in files - - def test_changed_files_from_git_rev_base(tmpdir, capfd): """Test arbitrary git ref as base.""" git = which("git", required=True) -- cgit v1.2.3-60-g2f50