More comprehensive tests for gates

Open
#7 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Assessment

Difficulty
3/5
Estimated time
1-2 days
Newbie friendliness
45/100
Issue type
Refactor
Clarity
Mostly clear
Activity status
Stale
Tech stack
rust
Domain
testing

Research direction

Start by locating the existing rudimentary gate test in the crossflow repository and read the gate operation and reachability calculations it exercises. Add comprehensive coverage for closed and open gates, including reachability behavior, then run the repository's Rust test suite to confirm the cases pass.

Written by the indexing model from the issue text.

Description

Before proceeding, is there an existing issue or discussion for this?
Description

We support an operation to "gate" a buffer, where a closed gate means that listeners will not be notified when the buffer gets an update. We have a rudimentary test for this operation, but it would be good to have more comprehensive tests, especially tests that make sure that reachability calculations are working as intended.

Dominant language
Rust
Stars
75
Forks
18
Avg merge
12d 12h
Merged PRs (30d)
3

Getting set up

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

More from open-rmf/crossflow

All issues in open-rmf/crossflow

Similar issues

More Rust issues

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.