diff --git a/src/elements/content-sidebar/versions/VersionsSidebarContainer.js b/src/elements/content-sidebar/versions/VersionsSidebarContainer.js index 175a773d11..c8d7596a95 100644 --- a/src/elements/content-sidebar/versions/VersionsSidebarContainer.js +++ b/src/elements/content-sidebar/versions/VersionsSidebarContainer.js @@ -95,11 +95,6 @@ class VersionsSidebarContainer extends React.Component { } } - componentWillUnmount() { - // Reset the current version id since the wrapping route is no longer active - this.props.onVersionChange(null); - } - handleActionDelete = (versionId: string): Promise => { this.setState({ isLoading: true }); diff --git a/src/elements/content-sidebar/versions/__tests__/VersionsSidebarContainer.test.js b/src/elements/content-sidebar/versions/__tests__/VersionsSidebarContainer.test.js index 6d7f2c26d5..6178d9efad 100644 --- a/src/elements/content-sidebar/versions/__tests__/VersionsSidebarContainer.test.js +++ b/src/elements/content-sidebar/versions/__tests__/VersionsSidebarContainer.test.js @@ -58,17 +58,6 @@ describe('elements/content-sidebar/versions/VersionsSidebarContainer', () => { }); }); - describe('componentWillUnmount', () => { - test('should forward verison id reset to the parent component', () => { - const onVersionChange = jest.fn(); - const wrapper = getWrapper({ onVersionChange }); - - wrapper.instance().componentWillUnmount(); - - expect(onVersionChange).toBeCalledWith(null); - }); - }); - describe('componentDidMount', () => { test('should call onLoad after a successful fetchData() call', async () => { const onLoad = jest.fn();