Skip to content

Use == instead of EXPECT_EQ in testlib - #52796

Closed
avp wants to merge 1 commit into
react:mainfrom
avp:export-D78823354
Closed

Use == instead of EXPECT_EQ in testlib#52796
avp wants to merge 1 commit into
react:mainfrom
avp:export-D78823354

Conversation

@avp

@avp avp commented Jul 23, 2025

Copy link
Copy Markdown
Contributor

Summary:
X-link: facebook/hermes#1739

The EXPECT_EQ was leading to potentially ambiguous use of << when
gtest tries to print information.

Differential Revision: D78823354

@facebook-github-bot facebook-github-bot added CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. p: Facebook Partner: Facebook Partner labels Jul 23, 2025
@facebook-github-bot

Copy link
Copy Markdown
Contributor

This pull request was exported from Phabricator. Differential Revision: D78823354

avp added a commit to avp/hermes that referenced this pull request Jul 23, 2025
Summary:
X-link: react/react-native#52796



The `EXPECT_EQ` was leading to potentially ambiguous use of `<<` when
gtest tries to print information.

Differential Revision: D78823354
avp added a commit to avp/hermes that referenced this pull request Jul 23, 2025
Summary:
X-link: react/react-native#52796



The `EXPECT_EQ` was leading to potentially ambiguous use of `<<` when
gtest tries to print information.

Differential Revision: D78823354
avp added a commit to avp/hermes that referenced this pull request Jul 23, 2025
Summary:
X-link: react/react-native#52796



The `EXPECT_EQ` was leading to potentially ambiguous use of `<<` when
gtest tries to print information.

Differential Revision: D78823354
avp added a commit to avp/hermes that referenced this pull request Jul 23, 2025
Summary:
X-link: react/react-native#52796



The `EXPECT_EQ` was leading to potentially ambiguous use of `<<` when
gtest tries to print information.

Differential Revision: D78823354
avp added a commit to avp/hermes that referenced this pull request Jul 23, 2025
Summary:
X-link: react/react-native#52796



The `EXPECT_EQ` was leading to potentially ambiguous use of `<<` when
gtest tries to print information.

Differential Revision: D78823354
avp added a commit to avp/hermes that referenced this pull request Jul 23, 2025
Summary:
X-link: react/react-native#52796



The `EXPECT_EQ` was leading to potentially ambiguous use of `<<` when
gtest tries to print information.

Differential Revision: D78823354
avp added a commit to avp/hermes that referenced this pull request Jul 23, 2025
Summary:
X-link: react/react-native#52796



The `EXPECT_EQ` was leading to potentially ambiguous use of `<<` when
gtest tries to print information.

Differential Revision: D78823354
avp added a commit to avp/hermes that referenced this pull request Jul 23, 2025
Summary:
X-link: react/react-native#52796



The `EXPECT_EQ` was leading to potentially ambiguous use of `<<` when
gtest tries to print information.

Differential Revision: D78823354
avp added a commit to avp/hermes that referenced this pull request Jul 23, 2025
Summary:
X-link: react/react-native#52796



The `EXPECT_EQ` was leading to potentially ambiguous use of `<<` when
gtest tries to print information.

Differential Revision: D78823354
avp added a commit to avp/hermes that referenced this pull request Jul 23, 2025
Summary:
X-link: react/react-native#52796



The `EXPECT_EQ` was leading to potentially ambiguous use of `<<` when
gtest tries to print information.

Differential Revision: D78823354
avp added a commit to avp/hermes that referenced this pull request Jul 23, 2025
Summary:
X-link: react/react-native#52796



The `EXPECT_EQ` was leading to potentially ambiguous use of `<<` when
gtest tries to print information.

Differential Revision: D78823354
avp added a commit to avp/hermes that referenced this pull request Jul 23, 2025
Summary:
X-link: react/react-native#52796



The `EXPECT_EQ` was leading to potentially ambiguous use of `<<` when
gtest tries to print information.

Differential Revision: D78823354
avp added a commit to avp/hermes that referenced this pull request Jul 23, 2025
Summary:
X-link: react/react-native#52796



The `EXPECT_EQ` was leading to potentially ambiguous use of `<<` when
gtest tries to print information.

Differential Revision: D78823354
avp added a commit to avp/hermes that referenced this pull request Jul 23, 2025
Summary:
X-link: react/react-native#52796



The `EXPECT_EQ` was leading to potentially ambiguous use of `<<` when
gtest tries to print information.

Differential Revision: D78823354
avp added a commit to avp/hermes that referenced this pull request Jul 23, 2025
Summary:
X-link: react/react-native#52796



The `EXPECT_EQ` was leading to potentially ambiguous use of `<<` when
gtest tries to print information.

Differential Revision: D78823354
avp added a commit to avp/hermes that referenced this pull request Jul 23, 2025
Summary:
X-link: react/react-native#52796



The `EXPECT_EQ` was leading to potentially ambiguous use of `<<` when
gtest tries to print information.

Differential Revision: D78823354
avp added a commit to avp/hermes that referenced this pull request Jul 23, 2025
Summary:
X-link: react/react-native#52796



The `EXPECT_EQ` was leading to potentially ambiguous use of `<<` when
gtest tries to print information.

Differential Revision: D78823354
avp added a commit to avp/hermes that referenced this pull request Jul 23, 2025
Summary:
X-link: react/react-native#52796



The `EXPECT_EQ` was leading to potentially ambiguous use of `<<` when
gtest tries to print information.

Differential Revision: D78823354
avp added a commit to avp/hermes that referenced this pull request Jul 23, 2025
Summary:
X-link: react/react-native#52796



The `EXPECT_EQ` was leading to potentially ambiguous use of `<<` when
gtest tries to print information.

Differential Revision: D78823354
avp added a commit to avp/hermes that referenced this pull request Jul 23, 2025
Summary:
X-link: react/react-native#52796



The `EXPECT_EQ` was leading to potentially ambiguous use of `<<` when
gtest tries to print information.

Differential Revision: D78823354
avp added a commit to avp/hermes that referenced this pull request Jul 23, 2025
Summary:
X-link: react/react-native#52796



The `EXPECT_EQ` was leading to potentially ambiguous use of `<<` when
gtest tries to print information.

Differential Revision: D78823354
avp added a commit to avp/hermes that referenced this pull request Jul 23, 2025
Summary:
X-link: react/react-native#52796



The `EXPECT_EQ` was leading to potentially ambiguous use of `<<` when
gtest tries to print information.

Differential Revision: D78823354
avp added a commit to avp/hermes that referenced this pull request Jul 23, 2025
Summary:
X-link: react/react-native#52796



The `EXPECT_EQ` was leading to potentially ambiguous use of `<<` when
gtest tries to print information.

Differential Revision: D78823354
avp added a commit to avp/hermes that referenced this pull request Jul 23, 2025
Summary:
X-link: react/react-native#52796



The `EXPECT_EQ` was leading to potentially ambiguous use of `<<` when
gtest tries to print information.

Differential Revision: D78823354
avp added a commit to avp/hermes that referenced this pull request Jul 23, 2025
Summary:
X-link: react/react-native#52796



The `EXPECT_EQ` was leading to potentially ambiguous use of `<<` when
gtest tries to print information.

Differential Revision: D78823354
@avp
avp force-pushed the export-D78823354 branch from d74e0b5 to e9abb3e Compare July 25, 2025 18:13
avp added a commit to avp/hermes that referenced this pull request Jul 25, 2025
Summary:
X-link: react/react-native#52796



The `EXPECT_EQ` was leading to potentially ambiguous use of `<<` when
gtest tries to print information.

Differential Revision: D78823354
avp added a commit to avp/hermes that referenced this pull request Jul 25, 2025
Summary:
X-link: react/react-native#52796

Pull Request resolved: facebook#1740

Pull Request resolved: facebook#1739

The `EXPECT_EQ` was leading to potentially ambiguous use of `<<` when
gtest tries to print information.

Differential Revision: D78823354
@facebook-github-bot

Copy link
Copy Markdown
Contributor

This pull request was exported from Phabricator. Differential Revision: D78823354

@avp
avp force-pushed the export-D78823354 branch from e9abb3e to 513a37c Compare July 25, 2025 18:18
avp added a commit to avp/hermes that referenced this pull request Jul 25, 2025
Summary:
X-link: react/react-native#52796

Pull Request resolved: facebook#1740

Pull Request resolved: facebook#1739

The `EXPECT_EQ` was leading to potentially ambiguous use of `<<` when
gtest tries to print information.

Differential Revision: D78823354
avp added a commit to avp/hermes that referenced this pull request Jul 25, 2025
Summary:
X-link: react/react-native#52796



The `EXPECT_EQ` was leading to potentially ambiguous use of `<<` when
gtest tries to print information.

Differential Revision: D78823354
avp added a commit to avp/hermes that referenced this pull request Jul 25, 2025
Summary:
X-link: react/react-native#52796



The `EXPECT_EQ` was leading to potentially ambiguous use of `<<` when
gtest tries to print information.

Differential Revision: D78823354
@avp
avp force-pushed the export-D78823354 branch from 513a37c to 837f5d1 Compare July 25, 2025 21:30
avp added a commit to avp/hermes that referenced this pull request Jul 25, 2025
Summary:
X-link: react/react-native#52796

Pull Request resolved: facebook#1740

Pull Request resolved: facebook#1739

The `EXPECT_EQ` was leading to potentially ambiguous use of `<<` when
gtest tries to print information.

Differential Revision: D78823354
@facebook-github-bot

Copy link
Copy Markdown
Contributor

This pull request was exported from Phabricator. Differential Revision: D78823354

@avp
avp force-pushed the export-D78823354 branch from 837f5d1 to a838a2d Compare July 25, 2025 21:36
avp added a commit to avp/hermes that referenced this pull request Jul 25, 2025
Summary:
X-link: react/react-native#52796

Pull Request resolved: facebook#1740

Pull Request resolved: facebook#1739

The `EXPECT_EQ` was leading to potentially ambiguous use of `<<` when
gtest tries to print information.

Reviewed By: tsaichien

Differential Revision: D78823354
avp added a commit to avp/hermes that referenced this pull request Jul 28, 2025
Summary:
X-link: react/react-native#52796



The `EXPECT_EQ` was leading to potentially ambiguous use of `<<` when
gtest tries to print information.

Reviewed By: tsaichien

Differential Revision: D78823354
avp added a commit to avp/hermes that referenced this pull request Jul 28, 2025
Summary:
X-link: react/react-native#52796



The `EXPECT_EQ` was leading to potentially ambiguous use of `<<` when
gtest tries to print information.

Reviewed By: tsaichien

Differential Revision: D78823354
avp added a commit to avp/hermes that referenced this pull request Jul 28, 2025
Summary:
X-link: react/react-native#52796



The `EXPECT_EQ` was leading to potentially ambiguous use of `<<` when
gtest tries to print information.

Reviewed By: tsaichien

Differential Revision: D78823354
@avp
avp force-pushed the export-D78823354 branch from a838a2d to 7455cf4 Compare July 28, 2025 16:05
avp added a commit to avp/hermes that referenced this pull request Jul 28, 2025
Summary:
X-link: react/react-native#52796

Pull Request resolved: facebook#1740

Pull Request resolved: facebook#1739

The `EXPECT_EQ` was leading to potentially ambiguous use of `<<` when
gtest tries to print information.

Reviewed By: tsaichien

Differential Revision: D78823354
Summary:
Pull Request resolved: react#52796

X-link: facebook/hermes#1740

X-link: facebook/hermes#1739

The `EXPECT_EQ` was leading to potentially ambiguous use of `<<` when
gtest tries to print information.

Reviewed By: tsaichien

Differential Revision: D78823354
@facebook-github-bot

Copy link
Copy Markdown
Contributor

This pull request was exported from Phabricator. Differential Revision: D78823354

@avp
avp force-pushed the export-D78823354 branch from 7455cf4 to e4b8b35 Compare July 28, 2025 16:15
avp added a commit to avp/hermes that referenced this pull request Jul 28, 2025
Summary:
X-link: react/react-native#52796

Pull Request resolved: facebook#1740

Pull Request resolved: facebook#1739

The `EXPECT_EQ` was leading to potentially ambiguous use of `<<` when
gtest tries to print information.

Reviewed By: tsaichien

Differential Revision: D78823354
@react-native-bot

Copy link
Copy Markdown
Collaborator

This PR is stale because it has been open for 180 days with no activity. It will be closed in 7 days unless you comment on it or remove the "Stale" label.

@react-native-bot react-native-bot added the Stale There has been a lack of activity on this issue and it may be closed soon. label Jan 25, 2026
@react-native-bot

Copy link
Copy Markdown
Collaborator

This PR was closed because it has been stalled for 7 days with no activity.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. fb-exported p: Facebook Partner: Facebook Partner

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants