--encrypt-key-file is not covered by the test suite

[Alexey Kopytov]

xb_encrypt.sh has the following line:

echo -n "Percona XtraBackup is awesome!!!" > $encrypt_key_file

But $encrypt_key_file is not initialized anywhere. That would result in a failed test if xb_encrypt.sh included inc/common.sh (which does "set -eu"), but it's not included in this test.

In my test suite cleanups I changed it so that each test is executed with "set -eu" no matter if and when it includes common.sh (and removed the line which now results in a failure).

But we need --encrypt-key-file coverage anyway.

[George Ormond Lorch III (gl-az)]

Hmmm, that looks like a remnant of when all tests used the key file which I eventually changed to just using the command line option to avoid having random files laying about for every test.

I do remember creating an explicit test for the key file and do not see it in the repository so I wonder if I failed to add it to the bzr tree somewhere along the line. I think now though that instead of creating another explicit test for the key file it would probably be better to just make the basic xb_encrypt test use the key file and let all other tests use the --encrypt-key= instead to help keep the test count down and still have adequate coverage.

I'll get this fixed up and submitted...

[Shahriyar Rzayev (rzayev-sehriyar)]

Percona now uses JIRA for bug reports so this bug report is migrated to: https://jira.percona.com/browse/PXB-648