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

Cleanup tests #213

Merged
merged 5 commits into from
May 28, 2024
Merged

Cleanup tests #213

merged 5 commits into from
May 28, 2024

Conversation

Nadrieril
Copy link
Member

@Nadrieril Nadrieril commented May 27, 2024

This PR:

  • Merges hashmap and hashmap_main;
  • Renames betree_main to betree;
  • Adds a way to specify options to the test runner for crates, in crate/aeneas-test-options;
  • Adds some tests we wanted to add.

Fixes #208, #153, #195, #191.

@Nadrieril Nadrieril force-pushed the cleanup-tests branch 3 times, most recently from a295eb1 to 65e07e6 Compare May 27, 2024 15:54
tests/src/infinite-loop.rs Outdated Show resolved Hide resolved
@Nadrieril Nadrieril requested a review from sonmarcho May 27, 2024 16:00
tests/src/hashmap.rs Outdated Show resolved Hide resolved
tests/src/betree/aeneas-test-options Show resolved Hide resolved
@sonmarcho
Copy link
Member

This is great! I would like to talk about the betree before you merge however (and we can decide whether we do the change now or in a later PR).

@Nadrieril Nadrieril merged commit ef7792c into main May 28, 2024
6 checks passed
@Nadrieril Nadrieril deleted the cleanup-tests branch May 28, 2024 09:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Cleanup remaining tests
2 participants