diff options
| author | Harmen Stoppels <harmenstoppels@gmail.com> | 2022-02-01 21:59:29 +0100 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2022-02-01 21:59:29 +0100 |
| commit | b93b64ca67768dfb2c377dd9c64452cb60ad7e23 (patch) | |
| tree | 186a5e08311c8184ada112aa3ec52299b9549b5e /lib | |
| parent | 7c31d4d27942a66e64d691af4fde4006128095e6 (diff) | |
| download | spack-b93b64ca67768dfb2c377dd9c64452cb60ad7e23.tar.gz spack-b93b64ca67768dfb2c377dd9c64452cb60ad7e23.tar.bz2 spack-b93b64ca67768dfb2c377dd9c64452cb60ad7e23.tar.xz spack-b93b64ca67768dfb2c377dd9c64452cb60ad7e23.zip | |
TermStatusLine: fix python 2.7 and add test (#28715)
Co-authored-by: Massimiliano Culpo <massimiliano.culpo@gmail.com>
Diffstat (limited to 'lib')
| -rw-r--r-- | lib/spack/spack/installer.py | 2 | ||||
| -rw-r--r-- | lib/spack/spack/test/installer.py | 11 |
2 files changed, 12 insertions, 1 deletions
diff --git a/lib/spack/spack/installer.py b/lib/spack/spack/installer.py index dec5bab626..d08e828c80 100644 --- a/lib/spack/spack/installer.py +++ b/lib/spack/spack/installer.py @@ -683,7 +683,7 @@ class TermStatusLine(object): return self.pkg_set.clear() - self.pkg_list.clear() + self.pkg_list = [] # Move the cursor to the beginning of the first "Waiting for" message and clear # everything after it. diff --git a/lib/spack/spack/test/installer.py b/lib/spack/spack/test/installer.py index 826c0534d3..bc1398afff 100644 --- a/lib/spack/spack/test/installer.py +++ b/lib/spack/spack/test/installer.py @@ -1272,3 +1272,14 @@ def test_overwrite_install_backup_failure(temporary_store, config, mock_packages # Make sure that `remove` was called on the database after an unsuccessful # attempt to restore the backup. assert fake_db.called + + +def test_term_status_line(): + # Smoke test for TermStatusLine; to actually test output it would be great + # to pass a StringIO instance, but we use tty.msg() internally which does not + # accept that. `with log_output(buf)` doesn't really work because it trims output + # and we actually want to test for escape sequences etc. + x = inst.TermStatusLine(enabled=True) + x.add("a") + x.add("b") + x.clear() |