chore: migrate to aws-sdk v3

[jsetton]

Description of changes:

This change migrates aws-sdk support from v2 to v3. The former is going into maintenance mode sometimes this year. This will remove the below warning from being generated during the skill infrastructure deployment.

==================== Deploy Skill Infrastructure ====================
(node:927) NOTE: We are formalizing our plans to enter AWS SDK for JavaScript (v2) into maintenance mode in 2023.

Please migrate your code to use AWS SDK for JavaScript (v3).
For more information, check the migration guide at https://a.co/7PzMCcy
(Use `node --trace-warnings ...` to show where the warning was created)

It includes some minor code refactoring and the removal of a duplicate test file.

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

[doiron]

wow this is really great! My entire team here is very grateful for your time / efforts on getting this done!

thank you! thank you! thank you!

[doiron]

ignore the bad merge. fixing it.

[jsetton]

Do you want me to resolve the review comments you added?

[doiron]

no I'm about done testing it right now. sorry about that was attempting to merge the package-lock.json file manually using the browser lol bad idea..

[jsetton]

I fixed the conflicts and applied the changes based on your review.

[jsetton]

I just realized you created #458.

[doiron]

thanks for fixing the conflict and updating, I got side tracked trying to fix some of the tests that are failing. I'm seeing functional test failures with these changes locally so currently trying to debug what is going on there. Once I can understand what needs to be done to fix them, I'll merge.
thanks again this PR is a enormous help!

update: oh one more thing, if you rebase now the integration tests should hopefully work.

[doiron]

functional tests are failing, but I have a local fix for it, will push a PR after this bigger change is merged.