amplify-cli icon indicating copy to clipboard operation
amplify-cli copied to clipboard

feat: add AppSync Todo template for Java

Open ykethan opened this issue 2 years ago • 1 comments

Description of changes

DRAFT - add Java AppSync Todo templates

Note: Not to merge this until https://github.com/aws-amplify/amplify-cli/pull/10841 has been merged.

The PR is to add Java template examples that allow Lambda functions to call a AppSync API. Changes include

  1. adding AppSync Todo templates for IAM and API key based examples.
  2. added new prompt that inputs auth type.
  3. Modify lambda-function.ts file to move cursor based on auth type
  4. added new attribute to the core settings to allow auth selection
  5. add e2e tests for both the templates

Issue #, if available

Description of how you validated changes

Wrote e2e test and ran them locally. also used the amplify-dev in a frontend app to test the flow.

Checklist

  • [x] PR description included
  • [ ] yarn test passes
  • [x] Tests are changed or added
  • [ ] Relevant documentation is changed or added (and PR referenced)
  • [ ] New AWS SDK calls or CloudFormation actions have been added to relevant test and service IAM policies

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

ykethan avatar Aug 09 '22 23:08 ykethan

This pull request introduces 1 alert when merging a96a444af4909945e9c27b568382570abd476b2b into 9100adfbfe98311bacfaa84a9e195946fd078c56 - view on LGTM.com

new alerts:

  • 1 for Unused variable, import, function or class

lgtm-com[bot] avatar Aug 09 '22 23:08 lgtm-com[bot]

This pull request introduces 1 alert when merging 2c367332348b2c65ee63c2ea99e41efbc5982899 into f0eb84472ac76d7b632b9716b43185ce0eaee3ce - view on LGTM.com

new alerts:

  • 1 for Unused variable, import, function or class

lgtm-com[bot] avatar Aug 24 '22 21:08 lgtm-com[bot]

Codecov Report

Merging #10867 (2c36733) into dev (9100adf) will decrease coverage by 0.16%. The diff coverage is 44.52%.

@@            Coverage Diff             @@
##              dev   #10867      +/-   ##
==========================================
- Coverage   47.43%   47.26%   -0.17%     
==========================================
  Files         673      674       +1     
  Lines       33274    33211      -63     
  Branches     6724     6698      -26     
==========================================
- Hits        15782    15697      -85     
- Misses      15803    15827      +24     
+ Partials     1689     1687       -2     
Impacted Files Coverage Δ
...s/awscloudformation/utils/trigger-file-uploader.js 18.00% <0.00%> (ø)
...fy-category-notifications/lib/multi-env-manager.js 0.00% <0.00%> (ø)
.../amplify-cli-core/src/state-manager/pathManager.ts 73.33% <ø> (ø)
...amplify-cli-core/src/state-manager/stateManager.ts 37.56% <0.00%> (ø)
...r-awscloudformation/src/amplify-service-manager.js 5.60% <0.00%> (-0.14%) :arrow_down:
...r-awscloudformation/src/amplify-service-migrate.js 10.71% <0.00%> (ø)
...y-provider-awscloudformation/src/attach-backend.js 8.86% <0.00%> (-0.41%) :arrow_down:
...es/amplify-provider-awscloudformation/src/index.ts 53.12% <ø> (-0.49%) :arrow_down:
...y-provider-awscloudformation/src/initialize-env.ts 22.03% <ø> (-0.55%) :arrow_down:
...ider-awscloudformation/src/upload-appsync-files.js 11.20% <0.00%> (ø)
... and 60 more

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

codecov-commenter avatar Aug 26 '22 16:08 codecov-commenter