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

Implement Watcher Testing #931

Open
4 tasks
frrist opened this issue Apr 20, 2022 · 0 comments
Open
4 tasks

Implement Watcher Testing #931

frrist opened this issue Apr 20, 2022 · 0 comments
Labels
kind/test Kind: Test P1 P1: Must be resolved

Comments

@frrist
Copy link
Member

frrist commented Apr 20, 2022

Description

Test lily watcher package in isolation of its indexer. Test should assert that watcher passes expected tipsets to its indexer, they need not assert it writes the data to the database.

Acceptance criteria

  • Watcher works with confidence of 1
  • Watcher tolerates reorgs
  • Watcher coverage for current, apply, and revert operations
  • Watcher handles failure case when notifier fails unexpectedly

Where to begin

watcher_test.go

@frrist frrist added kind/test Kind: Test P1 P1: Must be resolved labels Apr 20, 2022
@frrist frrist self-assigned this Apr 20, 2022
@frrist frrist added this to Sentinel Apr 20, 2022
@frrist frrist moved this to Todo in Sentinel Apr 20, 2022
@frrist frrist added this to Lily Apr 20, 2022
@frrist frrist moved this to Todo in Lily Apr 20, 2022
@frrist frrist removed their assignment Apr 20, 2022
@kasteph kasteph removed the status in Sentinel Feb 20, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/test Kind: Test P1 P1: Must be resolved
Projects
Status: Todo
Status: No status
Development

No branches or pull requests

1 participant