apache / apache/iggy

Fix the 4 ignored tests covering persisted durability fault coverage

Open
#4,139 0 comments 0 reactions 0 assignees View on GitHub
server test
Dominant language
Rust
Stars
4.9k
Forks
432
Avg merge
2d 10h
Merged PRs (30d)
173

Description

**After #4132 is merged.**

Each test below carries its reason in the `#[ignore]` attribute and its doc comment.

| Test | Location |
|---|---|
| `given_a_completed_write_when_the_process_is_sigkilled_then_the_bytes_should_not_survive` | `core/integration/tests/cluster/crash_durability.rs:878` |
| `given_the_cluster_simulator_when_seeding_a_topic_then_a_persisted_policy_should_be_expressible` | `core/simulator/src/lib.rs:7723` |
| `given_a_retired_writer_when_reacquiring_then_the_drain_wait_should_be_executor_agnostic` | `core/simulator/src/storage/tests.rs:2741` |
| `deleting_and_recreating_a_partition_fences_an_old_writer_completion` | `core/simulator/src/storage/tests.rs:852` (pre-existing) |

```
cargo test -p simulator --lib -- --ignored
cargo test -p integration given_a_completed_write -- --ignored
```

Remove each `#[ignore]` once it passes.

Contributor guide

Open the contributing guide

Research direction

After #4132 is merged, start with the four ignored tests and their #[ignore] reasons and doc comments: core/integration/tests/cluster/crash_durability.rs:878, core/simulator/src/lib.rs:7723, and core/simulator/src/storage/tests.rs:2741 and :852. Run the two cargo test commands in the issue, then address each test's stated failure. Done means all four tests pass and their #[ignore] attributes are removed.

Written by the indexing model from the issue text.

Assessment

Tech stack
rust
Domain
distributed-systems, testing
Issue type
Bug
Difficulty
4/5
Estimated time
3-5 days
Activity status
Active
Clarity
Mostly clear
Newbie friendliness
65/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.