tree b4eda4f997b299b7d2e9a762357256d11a3312ee
parent 056955ceed2aac3f53a345c079a312088028a7cc
author Jorge Lucangeli Obes <jorgelo@google.com> 1569596385 -0400
committer Jorge Lucangeli Obes <jorgelo@google.com> 1569849121 -0400

Fix -K check.

The -K check is not currently very useful, and was incomplete:
|skip_remount| was being set to 1 even in the cases where the remount
was happening with different flags (and therefore not being skipped.)

Fix this by renaming the |skip_remount| variable to |change_remount|
so that it better reflects what we want to test, and adding a clearer
error message (with a suggestion to explicitly add '-v'.)

Also fix some long lines and a comment.

Bug: chromium:1008895
Test: Unit tests pass.
Test: minijail0 -p -K -- /bin/id prints error.
Test: minijail0 -p -Kslave -- /bin/id prints error.
Test: minijail0 -pv -K -- /bin/id runs successfully.
Test: minijail0 -pv -Kslave -- /bin/id runs successfully.
Change-Id: I19b4bf4f495bd5ec40338f2502854d4054be4c5c