feat: Added transaction ID to ErrorTrace event

[svetlanabrennan]

Description

We need to add the transaction id to ErrorTrace event to link Transactions to Traces.

Added transaction.id as the last field of the Error Trace JSON payload if the error occurs within a transaction.

Example error trace payload:

[
  "BSr1H3OtQKJhADnLvCAdVaVlqrHUAAgBAAAnIQEAAMwQAgQgHVy5AwAGMTEuOYWZ0ZXItMw",
  [
    0,
    "WebTransaction/Expressjs/GET//error-stack",
    "Expected ',' or '}' after property value in JSON at position 47",
    "SyntaxError",
    {
      "userAttributes": {},
      "agentAttributes": {
        "spanId": "4c6c496ad0492950",
        "request.headers.host": "localhost:3000",
        "request.headers.userAgent": "curl/8.4.0",
        "request.headers.accept": "*/*",
        "request.method": "GET",
        "request.uri": "/error-stack",
        "http.statusCode": "500",
        "http.statusText": "Internal Server Error",
        "response.headers.contentSecurityPolicy": "default-src 'none'",
        "response.headers.contentType": "text/html; charset=utf-8",
        "response.headers.contentLength": 1228
      },
      "intrinsics": {
        "totalTime": 0.009906432625,
        "error.expected": false
      },
      "stack_trace": [ // snipped ]
    },
    "80e1fa9afe5439fb" // Transaction ID
  ]
]

How to Test

Updated some unit tests. Added unit tests that confirm transaction id is added to error trace json when distributed tracking is both turned off and turned on.

Related Issues

Closes #1951

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (60e57a1) 97.03% compared to head (73c52fc) 97.03%.
Report is 2 commits behind head on main.

❗ Current head 73c52fc differs from pull request most recent head 4f4bf29. Consider uploading reports for the commit 4f4bf29 to get more accurate results

Additional details and impacted files

@@           Coverage Diff           @@
##             main    #1954   +/-   ##
=======================================
  Coverage   97.03%   97.03%           
=======================================
  Files         215      215           
  Lines       40372    40373    +1     
=======================================
+ Hits        39176    39177    +1     
  Misses       1196     1196           

Flag	Coverage Δ
integration-tests-16.x	78.88% <100.00%> (+<0.01%)	⬆️
integration-tests-18.x	79.16% <100.00%> (+0.01%)	⬆️
integration-tests-20.x	79.15% <100.00%> (-0.03%)	⬇️
unit-tests-16.x	91.12% <100.00%> (+<0.01%)	⬆️
unit-tests-18.x	91.11% <100.00%> (+<0.01%)	⬆️
unit-tests-20.x	91.11% <100.00%> (+<0.01%)	⬆️
versioned-tests-16.x	74.00% <100.00%> (-0.03%)	⬇️
versioned-tests-18.x	74.02% <100.00%> (-0.03%)	⬇️
versioned-tests-20.x	74.03% <100.00%> (-0.03%)	⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

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

[svetlanabrennan]

@bizob2828 I've fixed the tests checking the transaction id is present as you suggested since I have access to the transaction.id in the tests.

[bizob2828]

🚢 🇮🇹