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] Fix the flaky panic: leveldb: closed test #803

Open
5 tasks
Olshansk opened this issue Sep 9, 2024 · 0 comments
Open
5 tasks

[Testing] Fix the flaky panic: leveldb: closed test #803

Olshansk opened this issue Sep 9, 2024 · 0 comments
Assignees
Labels
testing Test (or test utils) additions, fixes, improvements or other

Comments

@Olshansk
Copy link
Member

Olshansk commented Sep 9, 2024

Objective

Make unit tests in CI more reliable

Origin Document

An example of a failed CI run that only modified documentation: https://github.com/pokt-network/poktroll/actions/runs/10746831666/job/29808368953?pr=801

Screenshot 2024-09-09 at 10 50 06 AM

Goals

  • Remove flaky tests
  • Increase developer productivity (remove context switching)

Deliverables

  • Identify and fix the panic: leveldb: closed test

Non-goals / Non-deliverables

General deliverables

  • Comments: Add/update TODOs and comments alongside the source code so it is easier to follow.
  • Testing: Add new tests (unit and/or E2E) to the test suite.
  • Makefile: Add new targets to the Makefile to make the new functionality easier to use.
  • Documentation: Update architectural or development READMEs; use mermaid diagrams where appropriate.

Creator: @Olshansk
Co-Owners: @bryanchriswhite

@Olshansk Olshansk added the testing Test (or test utils) additions, fixes, improvements or other label Sep 9, 2024
@Olshansk Olshansk self-assigned this Sep 9, 2024
@Olshansk Olshansk added this to Shannon Sep 9, 2024
@Olshansk Olshansk moved this to 🔖 Ready in Shannon Sep 9, 2024
@Olshansk Olshansk assigned bryanchriswhite and unassigned Olshansk Sep 24, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
testing Test (or test utils) additions, fixes, improvements or other
Projects
Status: 🔖 Ready
Development

No branches or pull requests

2 participants