go-tuf icon indicating copy to clipboard operation
go-tuf copied to clipboard

test: Fuzzing go 1.18

Open naveensrinivasan opened this issue 2 years ago • 7 comments

This is going to be multiple PR's for fuzzing. After this PR is merged, the goal is to get it integrated with https://github.com/google/oss-fuzz/

Release Notes: Included go 1.18 fuzzing.

Types of changes:

  • [ ] Bug fix (non-breaking change which fixes an issue)
  • [ ] New feature (non-breaking change which adds functionality)
  • [ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)

Description of the changes being introduced by the pull request: Included fuzz tests. Please verify and check that the pull request fulfills the following requirements:

  • [x] Tests have been added for the bug fix or new feature
  • [ ] Docs have been added for the bug fix or new feature

naveensrinivasan avatar Aug 24 '22 23:08 naveensrinivasan

This is great! Can you rebase this on top of #357 soon? Thx!

trishankatdatadog avatar Aug 24 '22 23:08 trishankatdatadog

This is great! Can you rebase this on top of #357 soon? Thx!

I would wait for that to be merged, and I can rebase of master.

naveensrinivasan avatar Aug 25 '22 01:08 naveensrinivasan

I would wait for that to be merged, and I can rebase of master.

Yes that's what I meant thx

trishankatdatadog avatar Aug 25 '22 13:08 trishankatdatadog

This looks great, thanks!

IIUC this is relevant to (but does not fix) #346; just want to make sure the PR and issue link to each other.

#346

The goal is to do multiple PR’s.

naveensrinivasan avatar Aug 29 '22 13:08 naveensrinivasan

https://github.com/theupdateframework/go-tuf/runs/8053810526?check_suite_focus=true

This is failing because action is using version 1.17 instead of 1.19

image

naveensrinivasan avatar Sep 04 '22 20:09 naveensrinivasan

I think that tests will pass if you rebase after #362.

znewman01 avatar Sep 06 '22 16:09 znewman01

I'll take a look early next week!

znewman01 avatar Sep 10 '22 16:09 znewman01

@naveensrinivasan - hey, I've rebased your branch, but it seems that the tests need to be updated.

rdimitrov avatar Jul 03 '23 15:07 rdimitrov

Closing this since it's no longer relevant to the code base, but feel free to rebase and reopen 👍

rdimitrov avatar Jan 31 '24 20:01 rdimitrov