Connect Component's displayName is not working properly

[myNameIsDu]

Wrong name for the component that was packaged with Connect in React - Developer-Tools

I think that's the problem

// If we're in "pure" mode, ensure our wrapper component only re-renders when incoming props have changed.
    const Connect = pure ? React.memo(ConnectFunction) : ConnectFunction

    Connect.WrappedComponent = WrappedComponent
    Connect.displayName = displayName

Displayname should be a property in the connectFunction
Like this

    ConnectFunction.displayName=displayName

[timdorr]

Weird. This does look like a legit issue going back to 7.0.0, but I can't figure it out.

That definitely fixes the issue for React Dev Tools, but we have tests for the displayName that do pass:

react-redux/test/components/connect.spec.js

Lines 1695 to 1730 in 1df5622

	it('should set the displayName correctly', () => {
	expect(
	connect((state) => state)(
	class Foo extends Component {
	render() {
	return <div />
	}
	}
	).displayName
	).toBe('Connect(Foo)')
	expect(
	connect((state) => state)(
	createClass({
	displayName: 'Bar',
	render() {
	return <div />
	},
	})
	).displayName
	).toBe('Connect(Bar)')
	expect(
	connect((state) => state)(
	// eslint: In this case, we don't want to specify a displayName because we're testing what
	// happens when one isn't defined.
	/* eslint-disable react/display-name */
	createClass({
	render() {
	return <div />
	},
	})
	/* eslint-enable react/display-name */
	).displayName
	).toBe('Connect(Component)')
	})

I'm not sure why that isn't translating over to RDT. Perhaps Babel is mangling something? It looks like facebook/react#18026 is related.

[myNameIsDu]

Thanks for your answer, I think it should be React - Developer-Tools don't support 'memoize Components' displayname, like this

let Test =  (props) => {
  const dispatch = useDispatch()

    return (
        <div onClick={ () => dispatch({type:'TEST'})}>
            dispatch test------
        </div>
    )
}
const MemoizeTest = React.memo(Test);
MemoizeTest.displayName='memoize(test)'

But the displayname for 'Memoize Components' does exist, so your test case passed

[myNameIsDu]

Can I submit a fix for this issue?

[timdorr]

Sure, a PR is always appreciated.