Fixed the Problem(Step 1): Dataverse Transfer Fails at "Parse Dataverse METS …

[Meghanxuxx]

Issue Related: archivematica/Issues#1745

After reviewing the code, I found that we don't really need the extract_and_remove_bundle step for the extraction. The extract_contents.py already recursively looks for files that need to be extracted, so we can finish all the extraction in this step. Additionally, this file seems to only remove the "outermost" package, while keeping the default setting delete to be False for files other than the "outermost". This helps prevent RData files from being deleted once they have been unpacked.

For this case, the code change in this PR is to remove the extract_and_remove_bundle function.

[replaceafill]

@Meghanxuxx I think you need to rebase your pull request branch on top of the latest qa/0.x branch. For example, all of our imports in Python are absolute now.

Also notice the linting job is failing in the test workflow. We use pre-commit for linting the Archivematica code bases. Once you install it you can run pre-commit run --all-files on the top of any Archivematica repository and it should automatically fix linting problems or point where the errors are coming from.

[sarah-mason]

Hi @Meghanxuxx - as I mentioned in your Archivematica PRs, we'll be starting testing in a little over a week for 1.18 so we'll stop accepting contributions around then. If you want to address the failed tests in your PRs and have any questions, please let us know. Thank you for your contributions!