[PATCH] xfstests: more tests for test case btrfs/030
Dave Chinner
david at fromorbit.com
Sun Feb 2 15:57:20 CST 2014
On Sat, Feb 01, 2014 at 02:05:32AM +0000, Filipe David Borba Manana wrote:
> This change adds some new tests for btrfs' incremental send feature.
> These are all related with inverting the parent-child relationship
> of directories, and cover the cases:
>
> * when the new parent didn't get renamed (just moved)
> * when a child file of the former parent gets renamed too
>
> These new cases are fixed by the following btrfs linux kernel patches:
>
> * "Btrfs: more send support for parent/child dir relationship inversion"
> * "Btrfs: fix send dealing with file renames and directory moves"
>
> Signed-off-by: Filipe David Borba Manana <fdmanana at gmail.com>
Rather than modifying 030 which will cause it to fail on kernels
where it previously passed, can you factor out the common code and
create a new test with the additional coverage?
i.e. the rule of thumb is that once a test is "done" we don't go
back and modify it in significant ways - we write a new unit test
that covers the new/extended functionality. Redundancy in unit tests
is not a bad thing...
Cheers,
Dave.
--
Dave Chinner
david at fromorbit.com
More information about the xfs
mailing list