test(add-contributors-list): Improve the clarity of the add-contributors-list tests

[jdalrymple]

What: Switch to async/await syntax for the add-contributors-list test file, and fixed one of the unlink function call

Why: It was causing an issue during the overhaul here #375

How: Switch to async/await and fs/promises

Checklist:

• Documentation
• Tests
• Ready to be merged
• Added myself to contributors table

[jdalrymple]

I need to double check the snapshots. Ill get back to this later today

[lwasser]

@jdalrymple thank you again and again for this. I just opened #397 to set up our tests on GitHub Actions. We don't have proper access to Circle CI right now, and I think GitHub is simpler for us. Just so you know!!

I merged a few things earlier, so you can rebase when you have time!! And finally, all of the deps will get updated via Dependabot as soon as my test pr is merged. We are so grateful to have your help here, thank you again!!

[lwasser]

@all-contributors please add @jdalrymple for tests

[allcontributors]

@lwasser

@jdalrymple already contributed before to test

[jdalrymple]

@jdalrymple thank you again and again for this. I just opened #397 to set up our tests on GitHub Actions. We don't have proper access to Circle CI right now, and I think GitHub is simpler for us. Just so you know!!

I merged a few things earlier, so you can rebase when you have time!! And finally, all of the deps will get updated via Dependabot as soon as my test pr is merged. We are so grateful to have your help here, thank you again!!

Sounds good! Happy to help 🫡

[lwasser]

This looks great! thank you again. cleaner tests docs and the move to async/await is much nicer!!