feat(rum-core): allow sending route-change transactions with no spans

Author: david-luna

packages/rum-core/src/common/config-service.js

@@ -95,7 +95,8 @@ class Config {
       context: {},
       session: false,
       apmRequest: null,
-      sendCredentials: false
+      sendCredentials: false,
+      sendAllRouteChanges: false
     }
 
     this.events = new EventHandler()

packages/rum-core/src/performance-monitoring/performance-monitoring.js

@@ -48,7 +48,8 @@ import {
   OUTCOME_SUCCESS,
   OUTCOME_UNKNOWN,
   QUEUE_ADD_TRANSACTION,
-  TRANSACTION_IGNORE
+  TRANSACTION_IGNORE,
+  ROUTE_CHANGE
 } from '../common/constants'
 import {
   truncateModel,
@@ -362,6 +363,12 @@ export default class PerformanceMonitoring {
       }
 
       if (tr.sampled && tr.spans.length === 0) {
+        if (
+          tr.type === ROUTE_CHANGE &&
+          this._configService.get('sendAllRouteChanges')
+        ) {
+          return true
+        }
         if (__DEV__) {
           this._logginService.debug(
             `transaction(${tr.id}, ${tr.name}) was discarded! Transaction does not have any spans`

packages/rum-core/test/performance-monitoring/performance-monitoring.spec.js

@@ -127,6 +127,20 @@ describe('PerformanceMonitoring', function () {
     )
   })
 
+  it('should allow route-change transactions with no spans if enabled by configuration', () => {
+    spyOn(logger, 'debug').and.callThrough()
+    configService.setConfig({ ...agentConfig, sendAllRouteChanges: true })
+    const transaction1 = new Transaction('/route/:id', 'route-change', {
+      id: 1,
+      startTime: 0,
+      managed: true
+    })
+    transaction1.end(600)
+    expect(transaction1.duration()).toBe(600)
+    expect(performanceMonitoring.filterTransaction(transaction1)).toBe(true)
+    expect(logger.debug).not.toHaveBeenCalled()
+  })
+
   it('should initialize and add transaction to the queue', async () => {
     performanceMonitoring.init()
     spyOn(apmServer, 'addTransaction')