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

swamp: Full Node sync tests #441

Merged
merged 4 commits into from
Mar 14, 2022
Merged

Conversation

Bidon15
Copy link
Member

@Bidon15 Bidon15 commented Feb 15, 2022

Introduction of Full Node to the swamp tests

  1. Creation of the full node
  2. Sync test for full <-> bridge and light ->full -> bridge

Based on #416 #428 #438
Resolves #417

@Bidon15 Bidon15 added the swamp Related to Integration tests label Feb 15, 2022
@Bidon15 Bidon15 self-assigned this Feb 15, 2022
@Wondertan Wondertan deleted the branch celestiaorg:main March 9, 2022 14:43
@Wondertan Wondertan closed this Mar 9, 2022
@Bidon15 Bidon15 reopened this Mar 9, 2022
@Bidon15 Bidon15 changed the base branch from hlib/full-node to main March 9, 2022 14:46
@Bidon15 Bidon15 force-pushed the full_node_with_sync branch from de7e597 to ad6e7b6 Compare March 9, 2022 15:48
@Bidon15
Copy link
Member Author

Bidon15 commented Mar 9, 2022

After this PR, the main focus will be pointed to refactoring the swamp code-base #454 and covering possible cases that can de derived or even be fully covered here celestiaorg/test-infra#21

@Bidon15 Bidon15 marked this pull request as ready for review March 9, 2022 16:13
node/tests/sync_test.go Outdated Show resolved Hide resolved
Copy link
Member

@renaynay renaynay left a comment

Choose a reason for hiding this comment

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

LGTM - love the light node sync against full node test ❤️

Copy link
Member

@liamsi liamsi left a comment

Choose a reason for hiding this comment

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

👏🏼

@Bidon15 Bidon15 merged commit 06c8fbf into celestiaorg:main Mar 14, 2022
@Bidon15 Bidon15 deleted the full_node_with_sync branch March 14, 2022 13:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
swamp Related to Integration tests
Projects
No open projects
Archived in project
Development

Successfully merging this pull request may close these issues.

swamp/tests/sync: Include Full Node type
4 participants