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

Run the fuzzers in the CI #7

Open
sylvestre opened this issue Jan 27, 2024 · 2 comments
Open

Run the fuzzers in the CI #7

sylvestre opened this issue Jan 27, 2024 · 2 comments
Labels
good first issue Good for newcomers

Comments

@sylvestre
Copy link
Collaborator

just like the coreutils, we should run the fuzzers in the CI to make sure we don't regress

https://github.com/uutils/coreutils/blob/main/.github/workflows/fuzzing.yml as example

@hanbings
Copy link
Contributor

hello!
I submitted a fuzzing.yml ci file and made minor modifications to the original code in #29, but I forgot to pick up the issues here in advance, I hope it doesn't disrupt your work.

Here are some other issues I've encountered:

  1. Noticed that both the Windows build and macOS before I forked didn't work.
  2. Although the CI is passing, it doesn't seem to be testing properly.
    here is the fuzzing ci log built on my fork: https://github.com/hanbings/diffutils/actions/runs/8331273334/job/22797824794
    in the Run fuzz_ed for XX secords step, cargo threw an error in the console, but did not interrupt the CI.

Do I need to create some ci action to get full logs from ci? or where should I start to check the fuzzed code?

@hanbings
Copy link
Contributor

Excuse me, I noticed that a recent commit changed the parameters of the diff function, and I have now modified the code of the fuzz test so that it builds properly. PR: #32

@hanbings hanbings mentioned this issue Mar 26, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue Good for newcomers
Projects
Status: No status
Development

No branches or pull requests

2 participants