Revert "Fix circular references in iterable equality"

[scotthovestadt]

Summary

Before re-merging the PR, we need to ensure the following test is added and passes:

test('returns false when given inequal set within a map', () => {
  expect(
    iterableEquality(
      new Map([['one', new Set([2])]]),
      new Map([['one', new Set([1, 2])]]),
    ),
  ).toBe(false);
});

I was able to isolate the cause of issues we are seeing to the following simple test case above. I see that the iterableEquality function itself is returning false, but it's getting swallowed somewhere in jasmineUtils eq and returning true regardless.

Hopefully can re-merge for the next release.

FYI @mattphillips

Test plan

• Revert PR and all related tests. Ensured revert solved the issue seen above.

[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.