pretty-format: Omit unnecessary symbol filter for object keys

[pedrottimark]

Summary

Goals:

• Remove filter for symbol keys because Object.keys returns properties with string keys, see https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Object/keys#Return_value
• Follow up on suggestion to improve cohesion of code pretty-format: Omit non-enumerable symbol properties #7448 (comment)

This dubious filter began in pretty-format 1.1.0 to remove string keys whose values look like a symbol description:

• jamiebuilds/pretty-format@85643d3#diff-328ef508b38c647e0a11e9981383022cR7
• jamiebuilds/pretty-format@85643d3#diff-328ef508b38c647e0a11e9981383022cR290

However, code change was unnecessary because the test added in 1.1.0 passes with 1.0.0 :(

I decided to leave if (Object.getOwnPropertySymbols) { … } for now, because method isn’t in IE:

https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Object/getOwnPropertySymbols#Browser_compatibility

Test plan

Existing test prints an object with properties and symbols still passes (for example, properties with symbol keys are not duplicated)

[SimenB]

#trustthetests

[github-actions]

This pull request has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.
Please note this issue tracker is not a help forum. We recommend using StackOverflow or our discord channel for questions.