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

Add Loom tests and verification + ci #5

Open
willothy opened this issue Nov 3, 2024 · 1 comment
Open

Add Loom tests and verification + ci #5

willothy opened this issue Nov 3, 2024 · 1 comment
Labels
enhancement New feature or request help wanted Extra attention is needed

Comments

@willothy
Copy link
Member

willothy commented Nov 3, 2024

Use tokio-rs/loom for safety checking.

@willothy willothy added enhancement New feature or request help wanted Extra attention is needed labels Nov 3, 2024
Copy link

ellipsis-dev bot commented Nov 3, 2024

🚀 This implementation plan was created by @ellipsis-dev

Tag me (@ellipsis-dev) in a comment with guidance if you want me to replan or to implement this plan.

Implementation Plan

Step 1: Add Loom as dev dependency

Path: Cargo.toml
Add loom = "0.5" under [dev-dependencies] in Cargo.toml.

Step 2: Add Loom tests for concurrency

Path: tests/loom_tests.rs
In tests/loom_tests.rs, add Loom tests for ShardMap and ShardSet using loom::model to simulate concurrent operations.

Step 3: Add Loom tests to CI

Path: .github/workflows/rust.yml
In .github/workflows/rust.yml, add a job for Loom tests, mirroring the tests job. Include steps to install dependencies and run cargo test with Loom configurations.


Questions? Check out our documentation. Leave 👍/👎 on this plan to give us feedback.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

1 participant