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

[Testing] Organize Pocket Repo TestUtils #796

Open
10 tasks
Olshansk opened this issue May 31, 2023 · 0 comments · May be fixed by #797
Open
10 tasks

[Testing] Organize Pocket Repo TestUtils #796

Olshansk opened this issue May 31, 2023 · 0 comments · May be fixed by #797
Assignees
Labels
core Core infrastructure - protocol related testing Defining, adding, automating or modifying tests

Comments

@Olshansk
Copy link
Member

Olshansk commented May 31, 2023

Objective

Design and organize all test tooling used throughout the V1 Pocket code base.

Origin Document

This is an umbrella ticket used to capture the work that will be split out of the TestUtils document prepared by @bryanchriswhite and @dylanlott: https://www.notion.so/pocketnetwork/Testutils-9cba9010e18447248e9daa8a3b87e3f2?pvs=4

Goals

  1. Consolidate the best practices we’ve been building for testing purposes
  2. Determine a location for mocks, helpers and testing related documentation

Deliverable

  • GitHub Issues for each component/subcomponent in the origin document to track the design & implementation of the actual work.

Non-goals / Non-deliverables

  • Business logic changes unrelated to testing, but everything else (CLI, RPC, E2E, unit tests, etc is fair game)

General issue deliverables

  • Update the appropriate CHANGELOG(s)
  • Update any relevant local/global README(s)
  • Update relevant source code tree explanations
  • Add or update any relevant or supporting mermaid diagrams

Testing Methodology

  • Task specific tests or benchmarks: make ...
  • New tests or benchmarks: make ...
  • All tests: make test_all
  • LocalNet: verify a LocalNet is still functioning correctly by following the instructions at docs/development/README.md
  • k8s LocalNet: verify a k8s LocalNet is still functioning correctly by following the instructions here

Creators: @bryanchriswhite @dylanlott (creators of the origin document)
Co-Owners: Olshansk

@Olshansk Olshansk added core Core infrastructure - protocol related testing Defining, adding, automating or modifying tests labels May 31, 2023
@bryanchriswhite bryanchriswhite moved this to In Progress in V1 Dashboard Jun 1, 2023
@bryanchriswhite bryanchriswhite linked a pull request Jun 1, 2023 that will close this issue
20 tasks
@Olshansk Olshansk removed their assignment Jun 14, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
core Core infrastructure - protocol related testing Defining, adding, automating or modifying tests
Projects
Status: In Progress
Development

Successfully merging a pull request may close this issue.

3 participants