fix: adding JSON stringify to example python function

[ashleyoldershaw]

We need a JSON string for the body in order to access it in our response message. See this comment: aws-amplify/amplify-js#6390 (comment)

I had this issue with my own function which was returning null on my JSON.stringify(response). I found the above comment and it made it work! i.e. I could access the return body.

Issue #, if available:
No issue created as it's a one-line change.

Description of changes:
Adding json library import and json.dumps() call in packages/amplify-python-function-template-provider/resources/hello-world/index.py

Updating relevant unit test in packages/amplify-e2e-tests/src/__tests__/function_3.test.ts by adding a JSON.stringify() call to match the json.dumps() call in the source file.

If there's an issue with this PR let me know - this is my first open-source contribution!

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

[codecov]

Codecov Report

Merging #6493 (268679b) into master (525deb6) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #6493   +/-   ##
=======================================
  Coverage   57.28%   57.28%           
=======================================
  Files         479      479           
  Lines       21692    21692           
  Branches     4311     4311           
=======================================
  Hits        12426    12426           
  Misses       8385     8385           
  Partials      881      881           

---

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 525deb6...268679b. Read the comment docs.

[ashleyoldershaw]

Hi there,

Is there anything I need to do to get this approved? It is a very small change so should only take a minute to review, but could save a lot of headache for developers working with Python functions on their amplify backend :)

[edwardfoyle]

Hi @ashleyoldershaw thanks for the contribution! LGTM, we'll get this merged soon.

[ashleyoldershaw]

Hi @ashleyoldershaw thanks for the contribution! LGTM, we'll get this merged soon.

Thanks, Edward! This is my first open-source contribution so excited to get it in - all the best!

[github-actions]

This pull request has been automatically locked since there hasn't been any recent activity after it was closed. Please open a new issue for related bugs.

Looking for a help forum? We recommend joining the Amplify Community Discord server *-help channels for those types of questions.