The dry run always returns an empty list for the list of new assets, meaning it will always report that it would create 0 new albums. This PR changes this to return a list of dummy assets instead for accurate reporting.
It also adds a bunch of e2e tests for dry runs since we seem to not test this flag at all right now.
Lastly, it makes dry runs incompatible with the --skip-hash flag since duplicates can't be accurately handled in this case.
Fixes #8966
How Has This Been Tested?
Tested that the dry run lines up with the real run with a number of different commands.
Description
The dry run always returns an empty list for the list of new assets, meaning it will always report that it would create 0 new albums. This PR changes this to return a list of dummy assets instead for accurate reporting.
It also adds a bunch of e2e tests for dry runs since we seem to not test this flag at all right now.
Lastly, it makes dry runs incompatible with the
--skip-hashflag since duplicates can't be accurately handled in this case.Fixes #8966
How Has This Been Tested?
Tested that the dry run lines up with the real run with a number of different commands.