Test suite with trailing blank lines parsing failure

Bug #393907 reported by Javier Collado
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Checkbox
Fix Released
Low
Marc Tardif

Bug Description

When writing a test file from scratch, if extra unexpected blank lines are found at the end of the file, a parsing error happens and no test case is run.

Looking into the logs a backtrace is found at suites_info plugin. Please find attached a test suite to reproduce the problem and the backtrace extracted from the logs.

Revision history for this message
Javier Collado (javier.collado) wrote :
Marc Tardif (cr3)
Changed in checkbox:
assignee: nobody → Marc Tardif (cr3)
importance: Undecided → Low
milestone: none → 0.8-alpha3
status: New → In Progress
Revision history for this message
Marc Tardif (cr3) wrote :

This is actually the proper behavior and your template file does not adhere to the format compliant with Debian templates. If you ran your template against Debconf::Template->load(), you'd get a similar message pertaining to missing fields. The fun part is that this is not detected when building a package but only when installing!

Anyways, aside from the fun of Debian packaging, Checkbox can be slightly more flexible. So, I've pushed a branch for review which extends the parser to allow for an arbitrary number of newlines between and at the end of the test definitions.

Marc Tardif (cr3)
Changed in checkbox:
status: In Progress → Fix Committed
Marc Tardif (cr3)
Changed in checkbox:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.