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 Integration Tests for WirePayForMessage #39

Closed
evan-forbes opened this issue Feb 24, 2021 · 1 comment
Closed

Implement Integration Tests for WirePayForMessage #39

evan-forbes opened this issue Feb 24, 2021 · 1 comment
Assignees

Comments

@evan-forbes
Copy link
Member

evan-forbes commented Feb 24, 2021

The latest cli addition still needs integration tests. This should include

  • booting up a single node devnet and ipfs daemon
  • posting the data via the cli
  • checking that the appropriate fees were burned (which could be 0 depending on which stage we're at)
  • check for the data posted via ipfs
@evan-forbes evan-forbes changed the title Implement Integration Tests for the cli Implement Integration Tests for WirePayForMessage Feb 24, 2021
@github-actions github-actions bot added the Stale label Aug 6, 2021
@evan-forbes evan-forbes reopened this Aug 13, 2021
@evan-forbes evan-forbes removed the Stale label Aug 13, 2021
@evan-forbes evan-forbes mentioned this issue Nov 29, 2021
34 tasks
@liamsi liamsi moved this to TODO in Celestia Node Dec 8, 2021
@evan-forbes evan-forbes moved this from TODO to In Review in Celestia Node Jan 13, 2022
@evan-forbes evan-forbes self-assigned this Jan 13, 2022
@evan-forbes
Copy link
Member Author

closed with #171

Repository owner moved this from In Review to Done in Celestia Node Jan 14, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
No open projects
Archived in project
Development

No branches or pull requests

1 participant