Skip to content

run-bubblewrap: Fix parsing of enable-tmp-outputs

Jeremiah Bonney requested to merge jbonney/run-bubblewrap-arg-fix into master

Before raising this MR, consider whether the following are required, and complete if so:

  • Unit tests
  • Metrics
  • Documentation update(s)

Description

This PR builds on top of !602 (merged) to continue to fix-up the --enable-tmp-outputs option added to run-bubblewrap. Right now due to a missing return true the option isn't reported as parsed, leading to the following error:

Invalid option --enable-tmp-outputs

This PR fixes this and adds a test for parsing the extra args to ensure it doesn't break again in the future.

Changes proposed in this merge request:

  • Properly return that --enable-tmp-outputs was parsed
  • Add test

Merge request reports