Refactor S3 Uploads to be testable, move related s3 utility functions into their own file

[hunterwerlla]

Types of changes

• Bug fix (non-breaking change which fixes an issue)
• New feature (non-breaking change which adds functionality)
• Code cleanup

Description

• Make awsConnectionParameters.ts into it's own file, working closer towards AWS TasksParametersBase should be deleted in favor of AWSConnectionParameters #166 . AWSTasksParametersBase contains a ton of functions that are not actually member functions, this creates the AwsConnectionParameters interface for those items
• Refactor S3Upload to be testable
• Add functional tests for S3 Upload
• Refactor S3 Upload to also use aws ConnectionParameters
• Take changes from issue Add support to specify content encoding in upload task #114 out of Apply missing patches from (the now unused) internal repo #156 and put them in this PR as the whole module is refactored
• Split out an S3 utility function used by both upload and download into its own file

Related Issue(s), If Filed

#114

Checklist

• I have read the README document

• Local run of npm run package succeeds
• My code follows the code style of this project
• I have added tests to cover my changes
• All new and existing tests passed
• A short description of the change has been added to the CHANGELOG (CHANGELOG.md)

License

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

[codecov-io]

Codecov Report

Merging #177 into master will increase coverage by 2.95%.
The diff coverage is 38.07%.

@@           Coverage Diff            @@
##           master   #177      +/-   ##
========================================
+ Coverage    4.35%   7.3%   +2.95%     
========================================
  Files          67     69       +2     
  Lines        2366   2463      +97     
  Branches      273    293      +20     
========================================
+ Hits          103    180      +77     
- Misses       2263   2283      +20

Impacted Files	Coverage Δ
Tasks/Common/sdkutils/awsTaskParametersBase.ts	0% <0%> (-7.5%)	⬇️
Tasks/Common/sdkutils.ts	0% <0%> (ø)
Tasks/S3Upload/S3Upload.ts	0% <0%> (ø)	⬆️
Tasks/Common/defaultClients.ts	0% <0%> (ø)
Tasks/S3Download/S3Download.ts	0% <0%> (ø)	⬆️
Tasks/Common/s3.ts	100% <100%> (ø)
Tasks/S3Download/DownloadTaskOperations.ts	82.5% <100%> (-2.69%)	⬇️
Tasks/S3Download/DownloadTaskParameters.ts	54.54% <100%> (ø)	⬆️
Tasks/Common/awsConnectionParameters.ts	13.76% <13.76%> (ø)
Tasks/Common/sdkutils/sdkutils.ts	23.17% <25%> (+0.94%)	⬆️
... and 8 more

---

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 5d867a3...ffc8df8. Read the comment docs.