run_templates crashed with TypeError in _reader(): Can't convert 'bytes' object to str implicitly

Thank you for taking the time to report this bug and helping to make Ubuntu better. This is probably related to string encoding, and seems to be a problem in run_templates.

Steps to reproduce:

echo "hello" | /usr/share/checkbox/scripts/run_templates -t -s 'cat /etc/passwd'

Expected result:
hello

Actual result:
Traceback (most recent call last):
  File "/usr/share/checkbox/scripts/run_templates", line 142, in <module>
    sys.exit(main(sys.argv[1:]))
  File "/usr/share/checkbox/scripts/run_templates", line 137, in main
    runner.process(args, options.shell)
  File "/usr/share/checkbox/scripts/run_templates", line 39, in process
    records = self.process_output(process.stdout)
  File "/usr/share/checkbox/scripts/run_templates", line 88, in process_output
    return template.load_file(output)
  File "/usr/lib/python3/dist-packages/checkbox/lib/template.py", line 73, in load_file
    for string in self._reader(file):
  File "/usr/lib/python3/dist-packages/checkbox/lib/template.py", line 35, in _reader
    lines = re.split(delimiter, buffer_old + buffer_new)
TypeError: Can't convert 'bytes' object to str implicitly

I'll set to triaged, importance: high as this is potentially affecting everyone. Thanks!

This bug was fixed in the package checkbox - 0.14.1

---------------
checkbox (0.14.1) quantal-proposed; urgency=low

  * New upstream release (LP: #1018571)

  [Brendan Donegan]
  * Fixed up a few things with the gpu_lockup tests. Removed depends,
    renamed to gpu_lockup_suspend to reflect behaviour and removed the
    requirement on Firefox
  * Changed suspend_advanced and suspend_advanced_auto to use less
    strict definition of fwts s3 test.

  [Javier Collado]
  * Make sure that jobs are topologically ordered (LP: #990075)
  * Keep job ordering as close to whitelist as possible (LP: #1017951)

  [Marc Tardif]
  * New version 0.14.1 for Quantal Quetzal development.
  * jobs/suspend.txt.in: Fixed trailing newline on otherwise empty line.
  * scripts/run_templates: Fixed calls to Popen to use universal_newlines
    to return strings instead of bytes (LP: #1018354)

  [Daniel Manrique]
  * Fixed duplicate suspend/bluetooth_obex_after_suspend job name.
  * scripts/dpkg_resource: Changed encoding from ascii to utf-8 to handle
    non-ascii locales (LP: #1018353)

  [Jeff Lane]
  * Migrated audio/external-HDMI-playback into checkbox. Modified the
    command to match our other audio tests that save and reset mixer
    levels.
 -- Javier Collado <email address hidden> Tue, 26 Jun 2012 16:07:04 +0200