fix(vcs): remove empty string from command in repo.get_tracked_files()

The test should be exercising this code path, so I'm not really sure why
it passes in Taskgraph but failed in another case in CI. Maybe it's
related to the version of Git on the worker.

Author: ahal

src/taskgraph/util/vcs.py

@@ -290,6 +290,7 @@ def _files_template(self, diff_filter):
         return template
 
     def get_tracked_files(self, *paths, rev=None):
+        paths = [p for p in paths if p]
         rev = rev or "."
         return self.run("files", "-r", rev, *paths).splitlines()
 
@@ -476,6 +477,7 @@ def get_commit_message(self, revision=None):
         return self.run("log", "-n1", "--format=%B", revision)
 
     def get_tracked_files(self, *paths, rev=None):
+        paths = [p for p in paths if p]
         rev = rev or "HEAD"
         return self.run("ls-tree", "-r", "--name-only", rev, *paths).splitlines()
 

test/test_util_vcs.py

@@ -229,6 +229,7 @@ def test_get_tracked_files(repo):
     repo.run("commit", "-m", "Add second file")
     rev = ".~1" if repo.tool == "hg" else "HEAD~1"
     assert_files(repo.get_tracked_files(), ["first_file", "subdir/second_file"])
+    assert_files(repo.get_tracked_files(""), ["first_file", "subdir/second_file"])
     assert_files(repo.get_tracked_files("subdir"), ["subdir/second_file"])
     assert_files(repo.get_tracked_files(rev=rev), ["first_file"])