Skip to content

op-node: Reset bug unit test#3751

Merged
mergify[bot] merged 2 commits intodevelopfrom
jg/unit_test_for_reset_bug
Oct 20, 2022
Merged

op-node: Reset bug unit test#3751
mergify[bot] merged 2 commits intodevelopfrom
jg/unit_test_for_reset_bug

Conversation

@trianglesphere
Copy link
Contributor

Description

This adds a unit test for a bug that was fixed in #3694. I have confirmed that this unit test would have found that bug by temporarily reverting that commit & running this test.

Metadata

  • Fixes ENG-2909

@changeset-bot
Copy link

changeset-bot bot commented Oct 20, 2022

⚠️ No Changeset found

Latest commit: eb9b6f5

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@mergify
Copy link
Contributor

mergify bot commented Oct 20, 2022

This PR changes implementation code, but doesn't include a changeset. Did you forget to add one?

@trianglesphere trianglesphere force-pushed the jg/unit_test_for_reset_bug branch from b9896ea to b2d1c77 Compare October 20, 2022 21:39
@trianglesphere trianglesphere requested a review from tynes October 20, 2022 21:39
Copy link
Contributor

@tynes tynes left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Logically looks good

@mergify
Copy link
Contributor

mergify bot commented Oct 20, 2022

This PR has been added to the merge queue, and will be merged soon.

@mergify mergify bot merged commit 7037cc6 into develop Oct 20, 2022
@mergify mergify bot deleted the jg/unit_test_for_reset_bug branch October 20, 2022 22:12
@mergify
Copy link
Contributor

mergify bot commented Oct 20, 2022

This PR is next in line to be merged, and will be merged as soon as checks pass.

@mergify mergify bot removed the on-merge-train label Oct 20, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants