Handle command failure in script_run_interactive

[ghost]

script_run_interactive can not handle failure situation when running the command. Fix it by involving return value inside the typescript.

Also make it support running the command only - by passing undef at the second parameter. Change aa_genprof to adopt this method.

• Related ticket: https://progress.opensuse.org/issues/48521
• Verification run:
  • Handle failure example: http://10.67.17.9/tests/612#step/usr_lib_dovecot_pop3/38
  • Success example: http://10.67.17.9/tests/622#step/usr_lib_dovecot_pop3/44
  • Make sure the change will not make other depended cases failed:
    • MOK enrollment: http://10.67.17.9/tests/619#step/mokutil_sign/49
    • Apparmor: http://10.67.17.9/tests/628

[ostruzina]

Hello @wnereiz, I believe that these changes may have broken the usr_lib_dovecot_imap module in maintenance tests, e.g. https://openqa.suse.de/tests/2536919#step/usr_lib_dovecot_imap/52

When the logout is completed, there is a return code 1 and the test dies, even though the action was actually completed successfully. Could you take a look?

[ghost]

@ostruzina I saw the failure. It is basically the issue of case "usr_lib_dovecot_imap" (or telnet), script_run_interactive can do nothing about it.

Telnet return 1 even everything in process handled successfully. With some research, we found sending ^] to terminate telnet maybe a workaround. We will modify the case later.