Skip to content
This repository has been archived by the owner on Nov 15, 2023. It is now read-only.

BEEFY: add one/more zombienet tests #11761

Closed
2 tasks done
Tracked by #1632
acatangiu opened this issue Jun 30, 2022 · 2 comments · Fixed by paritytech/polkadot#5855
Closed
2 tasks done
Tracked by #1632

BEEFY: add one/more zombienet tests #11761

acatangiu opened this issue Jun 30, 2022 · 2 comments · Fixed by paritytech/polkadot#5855
Assignees
@acatangiu
Labels
J0-enhancement An additional feature request. U1-asap No need to stop dead in your tracks, however issue should be addressed as soon as possible. Z3-substantial Can be fixed by an experienced coder with a working knowledge of the codebase.

Comments

@acatangiu
Copy link
Contributor

Is there an existing issue?

  • I have searched the existing issues

Experiencing problems? Have you tried our Stack Exchange first?

  • This is not a support question.

Description of bug

BEEFY is currently only tested with simple 4-node unit-tests. It would benefit of a larger integration test most easily done using zombie-net.

Steps to reproduce

No response
@acatangiu acatangiu added J0-enhancement An additional feature request. U3-nice_to_have Issue is worth doing eventually. Z1-easy Can be fixed primarily by duplicating and adapting code by an intermediate coder labels Jun 30, 2022
@acatangiu acatangiu added this to BEEFY Jun 30, 2022
@acatangiu acatangiu moved this to Backlog 🗒 in BEEFY Jun 30, 2022
@acatangiu acatangiu moved this from Backlog 🗒 to Need for Kusama 🗒 in BEEFY Jul 6, 2022
@acatangiu acatangiu added Z4-involved Can be fixed by an expert coder with good knowledge of the codebase. U1-asap No need to stop dead in your tracks, however issue should be addressed as soon as possible. and removed Z1-easy Can be fixed primarily by duplicating and adapting code by an intermediate coder U3-nice_to_have Issue is worth doing eventually. labels Jul 14, 2022
@acatangiu acatangiu self-assigned this Jul 14, 2022
@acatangiu acatangiu moved this from Need for Kusama 🗒 to In Progress 🛠 in BEEFY Aug 1, 2022
@acatangiu
Copy link
Contributor Author

#11761 added a simple BEEFY test, now we build on it

@acatangiu acatangiu mentioned this issue Aug 4, 2022
Merged
@acatangiu acatangiu moved this from In Progress 🛠 to Code in review 🧐 in BEEFY Aug 4, 2022
@acatangiu acatangiu added Z3-substantial Can be fixed by an experienced coder with a working knowledge of the codebase. and removed Z4-involved Can be fixed by an expert coder with good knowledge of the codebase. labels Aug 4, 2022
@acatangiu
Copy link
Contributor Author

added follow-up issue #11972

Repository owner moved this from Code in review 🧐 to Done ✅ in BEEFY Sep 8, 2022
@acatangiu acatangiu moved this to Draft in Parity Roadmap Sep 12, 2022
@acatangiu acatangiu moved this from Draft to Closed in Parity Roadmap Sep 12, 2022
@acatangiu acatangiu mentioned this issue Nov 11, 2022
Closed
14 tasks
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@acatangiu acatangiu

Labels
J0-enhancement An additional feature request. U1-asap No need to stop dead in your tracks, however issue should be addressed as soon as possible. Z3-substantial Can be fixed by an experienced coder with a working knowledge of the codebase.
Projects
No open projects
BEEFY
Status: Done ✅
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

zombienet: add BEEFY justifications import test
1 participant
@acatangiu