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

test(consensus): fix foreign_block_distribution test #958

Draft
wants to merge 2 commits into
base: development
Choose a base branch
from

Conversation

sdbondi
Copy link
Member

@sdbondi sdbondi commented Mar 6, 2024

Description

Motivation and Context

How Has This Been Tested?

What process can a PR reviewer use to test or verify this change?

Breaking Changes

  • None
  • Requires data directory to be deleted
  • Other - Please specify

@sdbondi sdbondi force-pushed the consensus-fix-foreign-block-distro-test branch from 97f94af to 95f6487 Compare March 6, 2024 10:56
Copy link

github-actions bot commented Mar 6, 2024

Test Results (CI)

498 tests  +4   488 ✅  -  6   1h 51m 11s ⏱️ + 3m 57s
 60 suites ±0     0 💤 ± 0 
  2 files   ±0    10 ❌ +10 

For more details on these failures, see this check.

Results for commit 2398bfb. ± Comparison against base commit 0deb6cd.

This pull request removes 2 and adds 6 tests. Note that renamed tests count towards both.
consensus_tests ‑ consensus::foreign_block_distribution
tari_transaction ‑ id_provider::tests::it_fails_if_generating_more_ids_than_the_max
consensus_tests ‑ consensus::foreign_shard_decides_to_abort
tari_engine_types ‑ id_provider::tests::get_random_bytes
tari_engine_types ‑ id_provider::tests::it_fails_if_generating_more_ids_than_the_max
tari_template_lib ‑ models::entity_id::export_bindings_componentkey
tari_template_lib ‑ models::entity_id::export_bindings_entityid
tari_transaction ‑ transaction::export_bindings_versionedsubstate

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

Successfully merging this pull request may close these issues.

None yet

2 participants