🐛 Fix update view name api in context history

packages/rum-core/src/domain/contexts/viewContexts.spec.ts

@@ -3,7 +3,7 @@ import { relativeToClocks, CLEAR_OLD_VALUES_INTERVAL } from '@datadog/browser-co
 import type { TestSetupBuilder } from '../../../test'
 import { setup } from '../../../test'
 import { LifeCycleEventType } from '../lifeCycle'
-import type { ViewCreatedEvent } from '../view/trackViews'
+import type { ViewCreatedEvent, ViewEvent } from '../view/trackViews'
 import type { ViewContexts } from './viewContexts'
 import { startViewContexts, VIEW_CONTEXT_TIME_OUT_DELAY } from './viewContexts'
 
@@ -105,6 +105,16 @@ describe('viewContexts', () => {
       lifeCycle.notify(LifeCycleEventType.BEFORE_VIEW_CREATED, buildViewCreatedEvent({ name: 'Fake name' }))
       expect(viewContexts.findView()!.name).toBe('Fake name')
     })
+
+    it('should update the view name for the current context', () => {
+      const { lifeCycle } = setupBuilder.build()
+      lifeCycle.notify(LifeCycleEventType.BEFORE_VIEW_CREATED, buildViewCreatedEvent({ name: 'foo' }))
+      lifeCycle.notify(LifeCycleEventType.VIEW_UPDATED, {
+        startClocks,
+        name: 'Fake Name',
+      } as ViewEvent)
+      expect(viewContexts.findView()!.name).toBe('Fake Name')
+    })
   })
 
   describe('history contexts', () => {

packages/rum-core/src/domain/contexts/viewContexts.ts

@@ -2,7 +2,7 @@ import type { RelativeTime, ClocksState } from '@datadog/browser-core'
 import { SESSION_TIME_OUT_DELAY, ValueHistory } from '@datadog/browser-core'
 import type { LifeCycle } from '../lifeCycle'
 import { LifeCycleEventType } from '../lifeCycle'
-import type { ViewCreatedEvent } from '../view/trackViews'
+import type { ViewCreatedEvent, ViewEvent } from '../view/trackViews'
 
 export const VIEW_CONTEXT_TIME_OUT_DELAY = SESSION_TIME_OUT_DELAY
 
@@ -30,6 +30,13 @@ export function startViewContexts(lifeCycle: LifeCycle): ViewContexts {
     viewContextHistory.closeActive(endClocks.relative)
   })
 
+  lifeCycle.subscribe(LifeCycleEventType.VIEW_UPDATED, (viewUpdate: ViewEvent) => {
+    const currentView = viewContextHistory.find(viewUpdate.startClocks.relative)
+    if (currentView && viewUpdate.name) {
+      currentView.name = viewUpdate.name
+    }
+  })
+
   lifeCycle.subscribe(LifeCycleEventType.SESSION_RENEWED, () => {
     viewContextHistory.reset()
   })

test/e2e/scenario/rum/init.scenario.ts

@@ -117,7 +117,7 @@ describe('API calls and events around init', () => {
         intakeRegistry,
         { name: 'before init', viewId: initialView.view.id },
         { name: 'before manual view', viewId: initialView.view.id },
-        { name: 'after manual view', viewId: initialView.view.id }
+        { name: 'after manual view', viewId: initialView.view.id, viewName: 'after manual view' }
       )
     })
 })
@@ -173,12 +173,18 @@ function expectToHaveErrors(events: IntakeRegistry, ...errors: Array<{ message:
   }
 }
 
-function expectToHaveActions(events: IntakeRegistry, ...actions: Array<{ name: string; viewId: string }>) {
+function expectToHaveActions(
+  events: IntakeRegistry,
+  ...actions: Array<{ name: string; viewId: string; viewName?: string }>
+) {
   expect(events.rumActionEvents.length).toBe(actions.length)
   for (let i = 0; i < actions.length; i++) {
     const registryAction = events.rumActionEvents[i]
     const expectedAction = actions[i]
     expect(registryAction.action.target!.name).toBe(expectedAction.name)
     expect(registryAction.view.id).toBe(expectedAction.viewId)
+    if (i === 0 && expectedAction.viewName) {
+      expect(registryAction.view.name).toBe(expectedAction.viewName)
+    }
   }
 }