Fix fake user addition to the config because of Mimikatz

[shreyamalviya]

What does this PR do?

Fixes #1860

PR Checklist

• Have you added an explanation of what your changes do and why you'd like to include them?
• Is the TravisCI build passing?
• Was the CHANGELOG.md updated to reflect the changes?
• Was the documentation framework updated to reflect the changes?
• Have you checked that you haven't introduced any duplicate code?

Testing Checklist

• Added relevant unit tests?
• Have you successfully tested your changes locally? Elaborate:

  Tested by running on Windows

• If applicable, add screenshots or log transcripts of the feature working

[codecov]

Codecov Report

Merging #1902 (9f78e0d) into develop (e91087f) will increase coverage by 0.00%.
The diff coverage is 100.00%.

❗ Current head 9f78e0d differs from pull request most recent head a335f30. Consider uploading reports for the commit a335f30 to get more accurate results

@@           Coverage Diff            @@
##           develop    #1902   +/-   ##
========================================
  Coverage    54.93%   54.94%           
========================================
  Files          446      447    +1     
  Lines        12624    12626    +2     
========================================
+ Hits          6935     6937    +2     
  Misses        5689     5689           

Impacted Files	Coverage Δ
monkey/infection_monkey/consts.py	100.00% <100.00%> (ø)
...imikatz_collector/mimikatz_credential_collector.py	100.00% <100.00%> (ø)
...ost_breach/actions/communicate_as_backdoor_user.py	63.46% <100.00%> (ø)

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update e91087f...a335f30. Read the comment docs.

[VakarisZ]

I don't think it's worth creating a new file just for this. I suppose mimikatz credential collector should just import from PBA, since that's the source of users we don't want to add.

[mssalvatore]

Don't couple the mimikatz collector to the PBAs!

[VakarisZ]

Have you checked? We need to make sure these users aren't left there forever

[shreyamalviya]

Yes, checked

[VakarisZ]

Small changes/tests

[mssalvatore]

I'm concerned about the coupling this introduces between the PBAs and CredentialsCollectors. We may not be able to properly solve this issue without some more infrastructure in place. Ideally, both the PBA and the MimikatzCollector would be configurable so that the user can configure which users MimikatzCollector ignores.

At this time, I don't know that this issue is a significant enough nuisance to warrant coupling the PBAs and the CredentialsCollectors.