Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

local testing #201

Merged
merged 6 commits into from
Mar 20, 2024
Merged

local testing #201

merged 6 commits into from
Mar 20, 2024

Conversation

chaudharysaket
Copy link
Contributor

@chaudharysaket chaudharysaket commented Mar 11, 2024

Local testing of lambda extension.

  • Create binary via release on forked repo. Use the url of the forked repo release to generate the layer
  • Set NEWRELIC_LOCAL_TESTING to true to enable Local testing.
  • Generate the layer which is zip of extension binary and language agent
  • Currently only lambda's Python runtime is used for testing
  • x86 and ARM architecture layers are created using publish.sh script
  • Lambda is created in the AWS test account using test.sh script
  • Lambda logs can be checked for error related to extension

Copy link

codecov bot commented Mar 11, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 80.96%. Comparing base (fcec11b) to head (5340661).

❗ Current head 5340661 differs from pull request most recent head 8a78e6c. Consider uploading reports for the commit 8a78e6c to get more accurate results

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #201      +/-   ##
==========================================
- Coverage   81.46%   80.96%   -0.50%     
==========================================
  Files          24       24              
  Lines        1219     1219              
==========================================
- Hits          993      987       -6     
- Misses        160      165       +5     
- Partials       66       67       +1     

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

Copy link
Contributor

@iamemilio iamemilio left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks Great!

@chaudharysaket chaudharysaket merged commit d7d37c2 into newrelic:main Mar 20, 2024
2 of 3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants