fuzzuf icon indicating copy to clipboard operation
fuzzuf copied to clipboard
verified publisher icon fuzzuf

Add CLI tests for some algorithms

Open Fadis opened this issue 3 years ago • 1 comments

Type of PR

  • [ ] Changes related to the roadmap (e.g., TODO.md) (type: A) -> Create an issue corresponding to the PR in advance, and refer to this PR in the issue.
  • Changes that are not related to the roadmap
    • [ ] Change with multiple possible solutions to the issue (type: B-1) -> Create an issue corresponding to the PR in advance and refer to this PR in the issue.
    • [x] Change with a single solution (type: B-2) -> There is no need to create an issue corresponding to the PR in advance. Please discuss it in this PR.

Related Issue

Importance of PR

  • Importance of the issue
    • [ ] Large (based on several days to weeks of discussion and verification, e.g., this issue is a blocking issue for other issues on the roadmap, etc.)
    • [ ] Medium (based on a few hours to a day of discussion and verification, e.g., this issue is a blocking issue for another minor issue)
    • [x] Small (apparent changes such as build error)
  • Complexity of the solution (code, tests, etc.)
    • [ ] Large (requires several days to several weeks of review)
    • [ ] Medium (requires several hours to a day of review)
    • [x] Small (trivial changes, such as build error)

PR Overview

Concerns (Optional)

  • [ ] Performance
  • [ ] Source Code Quality

The PR author should fill in the following checklist when submitting this PR.

Optional Entries

  • [ ] If this PR is a PR type A/B-1, there is a cross-link between this PR and the related issue.

Mandatory Entries

  • [ ] The PR title is a summary of the changes.
  • [ ] Completed each required field of the PR.

The PR author should fill out the following checklist in the comments to confirm that this PR is ready to be merged

  • [ ] CI is green or confirmed test run results.
  • [ ] All change suggestions from reviewers have been resolved (fixed or foregone).

The maintainer of this repository will set up a reviewer for each PR. PR reviewers should review this PR in terms of the checklist below before moving on to a detailed code review. Please comment on their initial response by filling in the checklist below.

Optional Entries

  • [ ] The reviewer assigned more reviewers if needed.
  • [ ] The reviewer noted that it is necessary to break out some of the changes in this PR into other PRs if needed.
  • [ ] The reviewer noted that the initial response is insufficient if needed.

Mandatory Entries

  • [ ] The title of this PR summarizes the changes made by this PR properly.
  • [ ] The target branch of this PR is as intended.
  • [ ] The reviewer understands the issues in this PR.
  • [ ] The reviewer plans to review with an appropriate workload based on the importance of this PR.

When the PR reviewer concludes that this PR is ready to be merged, please fill in the checklist below by posting it in the comment. If there is more than one reviewer, please do this on your own.

Optional Entries

  • [ ] The reviewer noted that if you believe that new tests are needed to evaluate this PR, they have been noted.
  • [ ] If minor refactorings are not mentioned in the PR, I understand the intent.
  • [ ] If this PR is a PR type A/B-1, we have agreed on this PR's design, direction, and granularity in the related issue.

Mandatory Entries

  • [ ] The reviewer understands how this PR addresses the issue and the specific changes.
  • [ ] This PR uses the best possible issue resolution that the reviewer can think of.
  • [ ] This PR is ready to be merged.

Fadis avatar Jul 29 '22 06:07 Fadis

Since added test for DIE and VUzzer is not working yet, those test are disabled currently.

Fadis avatar Aug 04 '22 07:08 Fadis