JUnit runner fix for len(args) > max_args in argfile.safe_args

Review Request #4294 - Created Oct. 11, 2016 and submitted

Yi Cheng
benjyw, kwlzn, mateor, nhoward_tw, stuhood, zundel

There is currently no test for len(args) > max_args in (https://github.com/pantsbuild/pants/blob/27cff2bba343824bc6745b6b03ec940f23e26ebf/src/python/pants/backend/jvm/argfile.py#L37-L55), and all the test cases in the repo have args passing through argfile.safe_args. Whereas the junit runner change adopted _Test class, so in the case of len(args) > max_args, writing args (list of _Test) into a file errors out:

    self._run_tests(test_registry, output_dir, coverage)
  File "/Users/yic/workspace/pants/src/python/pants/backend/jvm/tasks/junit_run.py", line 526, in _run_tests
    with argfile.safe_args(batch, self.get_options()) as batch_tests:
  File "/opt/twitter/Cellar/python/2.7.10_2/Frameworks/Python.framework/Versions/2.7/lib/python2.7/contextlib.py", line 17, in __enter__
    return self.gen.next()
  File "/Users/yic/workspace/pants/src/python/pants/backend/jvm/argfile.py", line 53, in safe_args
    yield create_argfile(fp)
  File "/Users/yic/workspace/pants/src/python/pants/backend/jvm/argfile.py", line 40, in create_argfile

Exception message: sequence item 0: expected string or Unicode, _Test found

This change corrects it by making sure string/unicode is passed into the argfile.safe_args function.

Other changes:

  • Privatize _execute_junit_runner.
  • Change _execute_junit_runner to support testing multiple files.
  • Add test for argfile.safe_args.


Yi Cheng
Nick Howard (Twitter)
Benjy Weinberger
Mateo Rodriguez
Stu Hood
Yi Cheng
Yi Cheng
Review request changed

Status: Closed (submitted)

Change Summary: