diff --git a/packages/react/src/profiler.tsx b/packages/react/src/profiler.tsx index 9647d34f0fb4..b8ffe2f1708a 100644 --- a/packages/react/src/profiler.tsx +++ b/packages/react/src/profiler.tsx @@ -63,6 +63,7 @@ class Profiler extends React.Component { description: `<${name}>`, op: REACT_MOUNT_OP, origin: 'auto.ui.react.profiler', + data: { 'ui.component_name': name }, }); } } @@ -87,6 +88,7 @@ class Profiler extends React.Component { this._updateSpan = this._mountSpan.startChild({ data: { changedProps, + 'ui.component_name': this.props.name, }, description: `<${this.props.name}>`, op: REACT_UPDATE_OP, @@ -120,6 +122,7 @@ class Profiler extends React.Component { op: REACT_RENDER_OP, origin: 'auto.ui.react.profiler', startTimestamp: this._mountSpan.endTimestamp, + data: { 'ui.component_name': name }, }); } } @@ -184,6 +187,7 @@ function useProfiler( description: `<${name}>`, op: REACT_MOUNT_OP, origin: 'auto.ui.react.profiler', + data: { 'ui.component_name': name }, }); } @@ -203,6 +207,7 @@ function useProfiler( op: REACT_RENDER_OP, origin: 'auto.ui.react.profiler', startTimestamp: mountSpan.endTimestamp, + data: { 'ui.component_name': name }, }); } }; diff --git a/packages/react/test/profiler.test.tsx b/packages/react/test/profiler.test.tsx index 70eaff2d2c8b..fa629434cdf3 100644 --- a/packages/react/test/profiler.test.tsx +++ b/packages/react/test/profiler.test.tsx @@ -80,6 +80,7 @@ describe('withProfiler', () => { description: `<${UNKNOWN_COMPONENT}>`, op: REACT_MOUNT_OP, origin: 'auto.ui.react.profiler', + data: { 'ui.component_name': 'unknown' }, }); }); }); @@ -99,6 +100,7 @@ describe('withProfiler', () => { op: REACT_RENDER_OP, origin: 'auto.ui.react.profiler', startTimestamp: undefined, + data: { 'ui.component_name': 'unknown' }, }); }); @@ -114,7 +116,6 @@ describe('withProfiler', () => { expect(mockStartChild).toHaveBeenCalledTimes(1); }); }); - describe('update span', () => { it('is created when component is updated', () => { const ProfiledComponent = withProfiler((props: { num: number }) =>
{props.num}
); @@ -126,7 +127,7 @@ describe('withProfiler', () => { rerender(); expect(mockStartChild).toHaveBeenCalledTimes(2); expect(mockStartChild).toHaveBeenLastCalledWith({ - data: { changedProps: ['num'] }, + data: { changedProps: ['num'], 'ui.component_name': 'unknown' }, description: `<${UNKNOWN_COMPONENT}>`, op: REACT_UPDATE_OP, origin: 'auto.ui.react.profiler', @@ -137,7 +138,7 @@ describe('withProfiler', () => { rerender(); expect(mockStartChild).toHaveBeenCalledTimes(3); expect(mockStartChild).toHaveBeenLastCalledWith({ - data: { changedProps: ['num'] }, + data: { changedProps: ['num'], 'ui.component_name': 'unknown' }, description: `<${UNKNOWN_COMPONENT}>`, op: REACT_UPDATE_OP, origin: 'auto.ui.react.profiler', @@ -180,6 +181,7 @@ describe('useProfiler()', () => { description: '', op: REACT_MOUNT_OP, origin: 'auto.ui.react.profiler', + data: { 'ui.component_name': 'Example' }, }); }); }); @@ -203,6 +205,7 @@ describe('useProfiler()', () => { description: '', op: REACT_RENDER_OP, origin: 'auto.ui.react.profiler', + data: { 'ui.component_name': 'Example' }, }), ); });