EDSC-4351: Implements AWS CDK

[macrouch]

Overview

What is the feature?

Implements AWS CDK

What areas of the application does this impact?

New deployment process, new local environment

Testing

Local env testing: follow README updates to set up local env, general testing of app

Checklist

• I have added automated tests that prove my fix is effective or that my feature works
• New and existing unit tests pass locally with my changes
• I have performed a self-review of my own code
• I have commented my code, particularly in hard-to-understand areas
• I have made corresponding changes to the documentation
• My changes generate no new warnings

[codecov]

Codecov Report

Attention: Patch coverage is 44.87179% with 86 lines in your changes missing coverage. Please review.

Project coverage is 93.33%. Comparing base (0853c40) to head (4e15f63).
Report is 2 commits behind head on main.

Files with missing lines	Patch %	Lines
serverless/src/util/mockStepFunction.js	8.23%	71 Missing and 7 partials ⚠️
serverless/src/util/getQueueUrl.js	80.95%	3 Missing and 1 partial ⚠️
serverless/src/submitRetrieval/handler.js	50.00%	3 Missing ⚠️
...nents/CollectionResults/CollectionResultsTable.jsx	0.00%	1 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #1849      +/-   ##
==========================================
- Coverage   93.72%   93.33%   -0.39%     
==========================================
  Files         778      777       -1     
  Lines       19023    19114      +91     
  Branches     4894     4954      +60     
==========================================
+ Hits        17829    17841      +12     
- Misses       1113     1218     +105     
+ Partials       81       55      -26     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[eudoroolivares2016]

npm install --include=optional sharp
npm i @rollup/rollup-darwin-x64

I needed to install these two locally to get my env to work. The rollup I think was a originally coming in from serverless framework

[eudoroolivares2016]

any performance issues with continuously polling the queues?

[macrouch]

Not that I have noticed, but it is an optional piece to run the application locally. The majority of things we do we don't need to run SQS at all

[eudoroolivares2016]

Tested that:

• Ensure basic application start up etc
• Ensure invoking a lambda works
• Ensure we can use the admin pages
• Ensure that new code changes get picked up
• Step functions work
• Redis cache works
• Admin fails if you change the admin list locally
• Local s3 works
  All worked locally

[dpesall]

Changes look good, but it looks like CodeCov says it's still missing coverage on a few files. Approved under the assumption that these will be covered.

[macrouch]

Changes look good, but it looks like CodeCov says it's still missing coverage on a few files. Approved under the assumption that these will be covered.

The only file the report is calling out that has significant changes is serverless/src/util/mockStepFunction.js, which is only used in development, just like the new scripts in bin. I included it in serverless/src/util because it felt weird to import a file from that bin directory into a lambda handler. We don't worry about testing any of those local dev files, so I don't want to spend time testing this file either.