Address reviewer feedback

Author: douglasjacobsen

var/ramble/repos/builtin/package_managers/spack-lightweight/test/spack_runner.py

@@ -378,19 +378,23 @@ def test_new_compiler_installs(tmpdir, capsys, request):
             "config:spack", {"global": {"flags": f"-C {config_path}"}}
         ):
             try:
+                pkg_spec = "[email protected] +binutils"
+                compiler_spec = "[email protected]"
                 sr = SpackRunner(dry_run=True)
                 sr.create_env(os.getcwd())
                 sr.activate()
                 sr.add_include_file(packages_path)
                 sr.add_include_file(compilers_path)
                 sr.generate_env_file()
-                sr.install_compiler("[email protected]", "[email protected]")
+                sr.install_compiler("[email protected] +binutils", "[email protected]")
                 captured = capsys.readouterr()
 
                 assert (
-                    "[email protected] is already an available compiler"
+                    f"{compiler_spec} is already an available compiler"
                     in captured.out
                 )
+
+                assert f"{pkg_spec}" not in captured.out
             except RunnerError as e:
                 pytest.skip("%s" % e)
 
@@ -563,11 +567,13 @@ def test_config_compiler_find_attribute(
                 "config:spack", {"compiler_find": {attr: value}}
             ):
                 try:
+                    pkg_spec = "[email protected] +binutils"
+                    compiler_spec = "[email protected]"
                     sr = SpackRunner(dry_run=True)
                     sr.create_env(os.getcwd())
                     sr.activate()
                     sr.add_include_file(compilers_path)
-                    sr.install_compiler("[email protected]", "[email protected]")
+                    sr.install_compiler(pkg_spec, compiler_spec)
                     captured = capsys.readouterr()
 
                     assert expected_str in captured.out