Skip to content

Fuzz test fixes #577

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 2 commits into from
Jun 28, 2024
Merged

Fuzz test fixes #577

merged 2 commits into from
Jun 28, 2024

Conversation

lplewa
Copy link
Contributor

@lplewa lplewa commented Jun 27, 2024 •
edited
Loading

Fixes in fuzz test - details on commits messages.

@lplewa lplewa requested a review from a team as a code owner June 27, 2024 12:47
@lplewa lplewa changed the title Fuzz Fuzz test fixes Jun 27, 2024
@PatKamin
Copy link
Contributor

remote -> remove in commit msgs.

Why are you removing all these prints? How can you determine the scenario (generated API calls) on a test fail with all prints removed?

@lplewa
Copy link
Contributor Author

lplewa commented Jun 27, 2024 •
edited
Loading

remote -> remove in commit msgs.

Why are you removing all these prints? How can you determine the scenario (generated API calls) on a test fail with all prints removed?

read seed from log, and reproduce by using it.

Also Log is too long to attach it to the raport

@bratpiorka
Copy link
Contributor

remote -> remove in commit msgs.

Why are you removing all these prints? How can you determine the scenario (generated API calls) on a test fail with all prints removed?

because the logs produced by this test hangs chrome :) also, the zip with logs > 50 MB

@lplewa
remove hardcoded seed from fuzz test
be3ac99 
fuzztest prints seed in the log. Hard coding seed in fuzz test is
counter productive.
@PatKamin
Copy link
Contributor

remote -> remove in commit msgs.
Why are you removing all these prints? How can you determine the scenario (generated API calls) on a test fail with all prints removed?

read seed from log, and reproduce by using it.

Should work, ok

@lukaszstolarczuk lukaszstolarczuk merged commit 96d8978 into oneapi-src:main Jun 28, 2024
71 checks passed
@KFilipek KFilipek mentioned this pull request Jul 23, 2024
Open
6 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@bratpiorka bratpiorka bratpiorka approved these changes

@lukaszstolarczuk lukaszstolarczuk lukaszstolarczuk approved these changes

@PatKamin PatKamin PatKamin approved these changes

@kswiecicki kswiecicki kswiecicki approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@lplewa @PatKamin @bratpiorka @lukaszstolarczuk @kswiecicki