* refactored async/await for import helper to not also use then/catch
* added enum
* refactor webhookHelper and tests to not use then/catch
* changed docstring
* refactoring bot and tests to not use then/catch
* refactoring commands.js and tests to not use then/catch
* refactoring memberHelper.js and tests to not use then/catch
* removing then/catch from messageHelper.test.js
* fixed set up for commands tests
* edited bot to have top level main function
* one more test in commands.js, and removed console.error
* fixed typo in webhookHelper
* forgot to switch over some tests in bot.test and commands.test
* removed console.log from import helper
* put console.error in commands
* converted utils.js to not use then/catch
* tested utils checkImageFormatValidity
* removed jest-fetch-mock since it turns out I was just manually mocking it anyway
* refactored database to not use then/catch
* added dash to commands.js and test to pass
* added the remaining webhook tests
* changed utils to check for 10MB size not 1MB
* removed unnecessary try/catch from utils
* Simplify getWebhook to use .find() instead of foreach logic
* make memberCommand exit when error occurs with parseMemberCommand
* changed commands.js to not have user interaction within the catch
* updated console.error message in database.js
* made importHelper mock throw error instead of "resolve" error
* replaced "pk;" with "pf;" in test
* Got rid of unnecessary check for empty message from user (Fluxer doesn't allow this to happen)
Removed export of token
* getAllMembersInfo checks for fields.length
* added default case to memberCommandHandler to throw error if command is not recognized
* reversed check for valid proxy (was returning valid if the proxy existed and invalid if it didn't)
* pushes e.message instead of full error object to errors array in importHelper
* adjusted tests to properly use mockRejectedValue for async rejections
* changed getAllMembersInfo map to say `index` not `name` as it actually gets the index of a member and then the member object
* adjusted importHelper to properly test throwing of aggregate error
* revamped setting of expiration warning (moved to utils and changed logic, wrote tests)
---------
Co-authored-by: Aster Fialla <asterfialla@gmail.com>
Tests are mostly complete (though some are failing -- to be fixed in the
async/await refactor.
Also refactored, fixed bugs found while testing, and added ability to
type `pf;m` instead of `pf;member`.
---------
Co-authored-by: Aster Fialla <asterfialla@gmail.com>
