[missing-deps-suggest] move buildozer cli to a new line (pantsbuild#6190

) Currently, the buildozer command is on the same line as the rest of the message. This makes it harder to select, particularly when the command is long. Putting the buildozer command on a new line will make it easier to copy / paste.
CMLivingston · Aug 27, 2018 · 5f8e34a · 5f8e34a
1 parent 2be7e20
commit 5f8e34a
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 7 deletions.
diff --git a/src/python/pants/backend/jvm/tasks/jvm_compile/jvm_compile.py b/src/python/pants/backend/jvm/tasks/jvm_compile/jvm_compile.py
@@ -599,12 +599,11 @@ def _find_missing_deps(self, compile_logs, target):
 
         path_to_buildozer = self.get_options().buildozer
         if path_to_buildozer:
-          suggestion_msg += ("\nYou can do this by running {buildozer} "
-                             "'add dependencies {deps}' {target}".format(
-                                 buildozer=path_to_buildozer,
-                                 deps=" ".join(sorted(suggested_deps)),
-                                 target=target.address.spec
-                             )
+          suggestion_msg += ("\nYou can do this by running:\n"
+                             "  {buildozer} 'add dependencies {deps}' {target}".format(
+                               buildozer=path_to_buildozer,
+                               deps=" ".join(sorted(suggested_deps)),
+                               target=target.address.spec)
                             )
 
         self.context.log.info(suggestion_msg)

diff --git a/...on/pants_test/backend/jvm/tasks/jvm_compile/test_missing_dependency_finder_integration.py b/...on/pants_test/backend/jvm/tasks/jvm_compile/test_missing_dependency_finder_integration.py
@@ -33,7 +33,7 @@ def test_missing_deps_found_buildozer(self):
       '--compile-zinc-buildozer=path/to/buildozer',
     ])
     self.assert_failure(run)
-    self.assertTrue(re.search("path/to/buildozer 'add dependencies 3rdparty:guava' " + target,
+    self.assertTrue(re.search("\n *path/to/buildozer 'add dependencies 3rdparty:guava' " + target,
                               run.stdout_data,
                               re.DOTALL))