mv: Convert tests to ATF.

[des]

Summary

Sponsored by: Klara, Inc.

[markj]

bin/mv/tests/mv_test.sh

9

Our usual shell style puts opening braces on their own lines, though admittedly a number of sh test files don't do this.

31

Should this be wrapped with atf_check?

42

Should this be wrapped with atf_check?

[des]

bin/mv/tests/mv_test.sh

9

I think we mostly don't in tests... maybe tests can weigh in

31

it should, thank you

42

it should, thank you

[des]

review feedback

[asomers]

Shouldn't you delete cvt.pl? That looks like a one-time script to me.

[des]

In D47569#1085032, @asomers wrote:

Shouldn't you delete cvt.pl? That looks like a one-time script to me.

I never intended to add it, sorry

[des]

rm cvt.pl

[asomers]

LGTM. If you're feeling ambitious you can add test cases for symlink destinations and the other command line options.

[des]

descriptive names