ref(node): Add propagations to http integration

fix tests

rename name_changes -> changes

remaining nameChanges

ref(node): Add propagations to http integration

fix nextjs tests

Author: AbhiPrasad

packages/nextjs/test/integration/test/server/tracing200.js

@@ -18,6 +18,8 @@ module.exports = async ({ url: urlBase, argv }) => {
       transaction: 'GET /api/users',
       transaction_info: {
         source: 'route',
+        changes: [],
+        propagations: 0,
       },
       type: 'transaction',
       request: {

packages/nextjs/test/integration/test/server/tracing500.js

@@ -17,6 +17,8 @@ module.exports = async ({ url: urlBase, argv }) => {
       transaction: 'GET /api/broken',
       transaction_info: {
         source: 'route',
+        changes: [],
+        propagations: 0,
       },
       type: 'transaction',
       request: {

packages/nextjs/test/integration/test/server/tracingHttp.js

@@ -31,6 +31,8 @@ module.exports = async ({ url: urlBase, argv }) => {
       transaction: 'GET /api/http',
       transaction_info: {
         source: 'route',
+        changes: [],
+        propagations: 1,
       },
       type: 'transaction',
       request: {

packages/nextjs/test/integration/test/server/tracingServerGetInitialProps.js

@@ -16,6 +16,8 @@ module.exports = async ({ url: urlBase, argv }) => {
       transaction: '/[id]/withInitialProps',
       transaction_info: {
         source: 'route',
+        changes: [],
+        propagations: 0,
       },
       type: 'transaction',
       request: {

packages/nextjs/test/integration/test/server/tracingServerGetServerSideProps.js

@@ -16,6 +16,8 @@ module.exports = async ({ url: urlBase, argv }) => {
       transaction: '/[id]/withServerSideProps',
       transaction_info: {
         source: 'route',
+        changes: [],
+        propagations: 0,
       },
       type: 'transaction',
       request: {

packages/node-integration-tests/suites/express/tracing/test.ts

@@ -38,6 +38,8 @@ test('should set a correct transaction name for routes specified in RegEx', asyn
     transaction: 'GET /\\/test\\/regex/',
     transaction_info: {
       source: 'route',
+      changes: [],
+      propagations: 0,
     },
     contexts: {
       trace: {
@@ -66,6 +68,8 @@ test.each([['array1'], ['array5']])(
       transaction: 'GET /test/array1,/\\/test\\/array[2-9]',
       transaction_info: {
         source: 'route',
+        changes: [],
+        propagations: 0,
       },
       contexts: {
         trace: {
@@ -102,6 +106,8 @@ test.each([
     transaction: 'GET /test/arr/:id,/\\/test\\/arr[0-9]*\\/required(path)?(\\/optionalPath)?\\/(lastParam)?',
     transaction_info: {
       source: 'route',
+      changes: [],
+      propagations: 0,
     },
     contexts: {
       trace: {

packages/node/src/integrations/http.ts

@@ -172,6 +172,11 @@ function _createWrappedRequestMethodFactory(
                 `[Tracing] Not adding sentry-trace header to outgoing request (${requestUrl}) due to mismatching tracePropagationTargets option.`,
               );
           }
+
+          const transaction = parentSpan.transaction;
+          if (transaction) {
+            transaction.metadata.propagations += 1;
+          }
         }
       }
 

packages/node/test/integrations/http.test.ts

@@ -172,6 +172,20 @@ describe('tracing', () => {
     expect(baggage).not.toBeDefined();
   });
 
+  it('records outgoing propagations on the transaction', () => {
+    nock('http://dogs.are.great').get('/').reply(200);
+
+    const transaction = createTransactionOnScope();
+
+    expect(transaction.metadata.propagations).toBe(0);
+
+    http.get('http://dogs.are.great/');
+    expect(transaction.metadata.propagations).toBe(1);
+
+    http.get('http://dogs.are.great/');
+    expect(transaction.metadata.propagations).toBe(2);
+  });
+
   describe('tracePropagationTargets option', () => {
     beforeEach(() => {
       // hacky way of restoring monkey patched functions

packages/remix/test/integration/test/server/loader.test.ts

@@ -55,6 +55,8 @@ describe.each(['builtin', 'express'])('Remix API Loaders with adapter = %s', ada
       transaction: 'routes/loader-json-response/$id',
       transaction_info: {
         source: 'route',
+        changes: [],
+        propagations: 0,
       },
       spans: [
         {

packages/tracing/src/transaction.ts

@@ -45,6 +45,8 @@ export class Transaction extends SpanClass implements TransactionInterface {
     this.metadata = {
       ...transactionContext.metadata,
       spanMetadata: {},
+      changes: [],
+      propagations: 0,
     };
 
     this._trimEnd = transactionContext.trimEnd;
@@ -156,6 +158,8 @@ export class Transaction extends SpanClass implements TransactionInterface {
       ...(metadata.source && {
         transaction_info: {
           source: metadata.source,
+          changes: metadata.changes,
+          propagations: metadata.propagations,
         },
       }),
     };