Skip to content
This repository has been archived by the owner on Oct 31, 2024. It is now read-only.

feat: add test for certificate block production #42

Merged
merged 7 commits into from
Dec 14, 2023
Merged

Conversation

atanmarko
Copy link
Member

@atanmarko atanmarko commented Dec 11, 2023

Description

This PR implements test to check if sequencer produces certificates for empty blocks on topos and incal networks.
Also fixes startup sequencer and health checks for edge nodes.

Fixes TP-813

Additions and Changes

Additional script network.sh is implemented to start up/bring down test network, and also to check if running TCE nodes are healthy

PR Checklist:

  • I have performed a self-review of my code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • I have added or updated tests that comprehensively prove my change is effective or that my feature works

@atanmarko atanmarko marked this pull request as ready for review December 12, 2023 12:53
tests/network.sh Show resolved Hide resolved
tests/network.sh Show resolved Hide resolved
tests/test_cert_production.sh Show resolved Hide resolved
tests/test_cert_production.sh Show resolved Hide resolved
tests/test_cert_production.sh Outdated Show resolved Hide resolved
tests/test_cert_production.sh Outdated Show resolved Hide resolved
tests/test_cert_production.sh Outdated Show resolved Hide resolved
tests/test_cert_production.sh Show resolved Hide resolved
Copy link
Member

@Freyskeyd Freyskeyd left a comment

Choose a reason for hiding this comment

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

Approve with some nitpicks

tests/network.sh Outdated Show resolved Hide resolved
subnet-topos.yml Show resolved Hide resolved
scripts/polygon_edge_healthcheck.sh Outdated Show resolved Hide resolved
@atanmarko atanmarko merged commit 5f5aa23 into main Dec 14, 2023
1 check passed
@atanmarko atanmarko deleted the feat/TP-813 branch December 14, 2023 11:44
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants