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

switch RlpBlockImporter tests to Junit5 #5704

Closed
wants to merge 10 commits into from

Conversation

0xBlockPay
Copy link

Migrate RlpBlockImporterTest from JUnit 4.0 to JUnit 5.0 #5571

Fixed Issue(s)

#5571

@github-actions
Copy link

  • I thought about documentation and added the doc-change-required label to this PR if updates are required.
  • I thought about the changelog and included a changelog update if required.
  • If my PR includes database changes (e.g. KeyValueSegmentIdentifier) I have thought about compatibility and performed forwards and backwards compatibility tests

@macfarla
Copy link
Contributor

hi @0xBlockPay you'll need to add signoff to each commit to make the DCO happy - you should be able to follow the instructions here https://github.com/hyperledger/besu/pull/5704/checks?check_run_id=15037098648

@0xBlockPay 0xBlockPay force-pushed the fix/5571 branch 2 times, most recently from 663a380 to 63e6bb1 Compare July 14, 2023 07:57
@0xBlockPay
Copy link
Author

Can I migrate to Junit5 other test files in besu module ?

fab-10 and others added 5 commits July 17, 2023 14:09
Signed-off-by: Fabio Di Fabio <fabio.difabio@consensys.net>
Signed-off-by: 0xBlockPay <0xblockpay@gmail.com>
Signed-off-by: Karim TAAM <karim.t2am@gmail.com>
Signed-off-by: 0xBlockPay <0xblockpay@gmail.com>
Signed-off-by: 0xBlockPay <jaguzik@e-science.pl>
Signed-off-by: 0xBlockPay <0xblockpay@gmail.com>
Signed-off-by: 0xBlockPay <0xblockpay@gmail.com>
Signed-off-by: 0xBlockPay <0xblockpay@gmail.com>
Signed-off-by: 0xBlockPay <0xblockpay@gmail.com>
Signed-off-by: 0xBlockPay <0xblockpay@gmail.com>
Signed-off-by: 0xBlockPay <0xblockpay@gmail.com>
Signed-off-by: 0xBlockPay <0xblockpay@gmail.com>
Signed-off-by: 0xBlockPay <0xblockpay@gmail.com>
@macfarla
Copy link
Contributor

Can I migrate to Junit5 other test files in besu module ?

happy for you to use your judgement - as long as PRs are less than 100 files!

@macfarla
Copy link
Contributor

closing this one - due to conflicts with main branch it's prob easier to start from scratch

@macfarla macfarla closed this Sep 13, 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.

4 participants