Skip to content

Refactor tests - #61

Closed
lforchini wants to merge 1 commit into
1-ci-apt-updatefrom
2-refactor-tests
Closed

lforchini wants to merge 1 commit into
1-ci-apt-updatefrom
2-refactor-tests

Conversation

@lforchini

Copy link
Copy Markdown
Owner

Renames tests to match the test_ naming convention the project will
adopt moving forward. Introduce proptest which asserts properties beyond
a single case. Remove some tests which were not adding value.

Signed-off-by: Leonardo Forchini leonardo.forchini@nutanix.com


Stack created with GitHub Stacks CLI • Give Feedback 💬

Renames tests to match the `test_` naming convention the project will
adopt moving forward. Introduce proptest which asserts properties beyond
a single case. Remove some tests which were not adding value.

Signed-off-by: Leonardo Forchini <leonardo.forchini@nutanix.com>
@lforchini
lforchini added this pull request to stack #68 September 30, 2026 08:13
@lforchini lforchini closed this Oct 1, 2026
lforchini added a commit that referenced this pull request Oct 1, 2026
It seems the apt repo can get stale and fail, so add an `apt update` to
refresh repos.

Signed-off-by: Leonardo Forchini <leonardo.forchini@nutanix.com>
Reviewed-by: Thanos Makatos <thanos.makatos@nutanix.com>
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.

1 participant