Skip to content

refactor(tests): Extract tests related logic into raw::tests for reuse#3420

Merged
Xuanwo merged 2 commits intomainfrom
fuzzer-checker
Oct 30, 2023
Merged

refactor(tests): Extract tests related logic into raw::tests for reuse#3420
Xuanwo merged 2 commits intomainfrom
fuzzer-checker

Conversation

@Xuanwo
Copy link
Copy Markdown
Member

@Xuanwo Xuanwo commented Oct 30, 2023

This extract tests related logci into raw::tests for better reusing. raw::tests is hiden under tests features that users won't touch that unexpected.

This change makes it much easier to reproduce our fuzzing tests:

pub async fn test_fuzz_issue_2717(op: Operator) -> Result<()> {
    let actions = [ReadAction::Seek(SeekFrom::End(-2))];

    test_fuzz_read(op, 2, .., &actions).await
}

Signed-off-by: Xuanwo <github@xuanwo.io>
@Xuanwo Xuanwo marked this pull request as ready for review October 30, 2023 06:20
@Xuanwo Xuanwo requested a review from PsiACE as a code owner October 30, 2023 06:20
@github-actions github-actions Bot added the releases-note/refactor The PR does a refactor on code or has a title that begins with "refactor" label Oct 30, 2023
Copy link
Copy Markdown
Member

@suyanhanx suyanhanx left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@Xuanwo Xuanwo merged commit 00fdfe0 into main Oct 30, 2023
@Xuanwo Xuanwo deleted the fuzzer-checker branch October 30, 2023 06:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

releases-note/refactor The PR does a refactor on code or has a title that begins with "refactor"

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants