fix: make promises call scope.end() in measureFn

[andresusanto]

Hi @maraisr, I've found a little issue where .end() is not being called by measureFn when we're measuring promises. It makes spans missing the end time and hence duration.

[maraisr]

hi @andresusanto 👋🏻 thanks for this! Wonder if we should write a test for this, probably should have caught it sooner.

[andresusanto]

hi @andresusanto 👋🏻 thanks for this! Wonder if we should write a test for this, probably should have caught it sooner.

sounds good @maraisr, I will add some additional tests to catch this behaviour

[maraisr]

if this is blocking ya'll, ill merge and release, and loop back to tests after? What say you?

[andresusanto]

if this is blocking ya'll, ill merge and release, and loop back to tests after? What say you?

I've added the tests @maraisr, should be able to cover the edge case, cheers 🙌

[andresusanto]

thanks for merging and releasing @maraisr !! 🙌