Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

itest: attempt to reproduce asset balance bug report #679

Closed
wants to merge 2 commits into from
Closed

Conversation

ffranr
Copy link
Contributor

@ffranr ffranr commented Nov 16, 2023

Bug report:
#665

This PR attempts to reproduce the bug reported in the issue above. It does not succeed. These are the rough steps taken in this PR's bug reproduction attempt:

  1. Send an asset to a receiver node.
  2. Assert that the receiver node's asset balance is correct.
  3. Restart the receiver node. Repeat 2.
  4. Mine during receiver node restart. Repeat 2.

This PR is based on the commit corresponding to release tag v0.3.1. I think the receiving node was at release v0.3.1.

This PR shouldn't be merged.

@ffranr
Copy link
Contributor Author

ffranr commented Nov 16, 2023

I've added a commit to this PR. It now reproduces the asset balance bug reported here: #665

The latest commit removes a delay which effectively hid the bug from the itests.

@ffranr
Copy link
Contributor Author

ffranr commented Nov 20, 2023

These commits are now in use in two other PRs that we can potentially merge.

@ffranr ffranr closed this Nov 20, 2023
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.

1 participant