As per #366, ImportData was removing users every time it was re-importing the scripts. As there is no need to do so and it could create catastrophic data loss in production scenarios, I've simply removed 'Users' from the removal process.
Motivation and Context
I also investigated the possibility of adding a 'collectionsToRemove' array in cleanup that could then inform the ImportData function, but didn't have a use case that warranted it.
How Has This Been Tested?
Poses no issues locally.
Types of changes
[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
Checklist:
[x] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
Coverage increased (+0.05%) to 80.554% when pulling c379517c552ec180d74613a1bb4de51a0797b788 on cleanup_update into 77327d1fef04ba9637ca15749083ddfab58d673c on master.
Description
As per #366, ImportData was removing users every time it was re-importing the scripts. As there is no need to do so and it could create catastrophic data loss in production scenarios, I've simply removed 'Users' from the removal process.
Motivation and Context
I also investigated the possibility of adding a 'collectionsToRemove' array in cleanup that could then inform the ImportData function, but didn't have a use case that warranted it.
How Has This Been Tested?
Poses no issues locally.
Types of changes
Checklist: