Closed NeilJed closed 1 year ago
Thank you, @NeilJed 🙏
Tests are succeeding - don't worry about the integration test failing, it seems to be a secrets configuration error on my side.
Will have a closer look later today, but I think this is ready to be merged 🚀
@alexcasalboni Were you going to merge this or did you want to solve the Node16 test run first?
Apologies, it took me a while to jump on this 😃
Merging now 🚀
This is now published on SAR as version 4.3.1
👌
Updates
extractDuration
to use the Billed Duration value rather than Duration from logs for cost calculations. Billed duration is at 1ms resolution and expressed as an integer.Change reflects the fact that for certain packages/runtimes the init phase is charged for and thus the Billed Duration reflects the true cost of the lambda. See #197
Unit tests updated to reflect the change and modified to use realistic values for given execution times.