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

Issue #714 Added the fuzzy test. #1011

Merged
merged 10 commits into from Dec 11, 2019

Conversation

monadbobo
Copy link
Contributor

sumarry:

  1. Added a new parameter "fuzzer" to nebula_add_test(cmake).
    Turn on this parameter indicates that the current test is a fuzzy test.
  2. Currently only a simple fuzzy test has been added for paser module.
  3. The current fuzzy test only supports Clang, so if the compiler was gcc it will skip.

sumarry:

1. Added a new parameter "fuzzer" to nebula_add_test(cmake).
   Turn on this parameter indicates that the current test is a fuzzy test.
2. Currently only a simple fuzzy test has been added for paser module.
3. The current fuzzy test only supports Clang, so if the compiler was gcc it will skip.
@monadbobo monadbobo added the ready-for-testing PR: ready for the CI test label Sep 30, 2019
@nebula-community-bot
Copy link
Member

Unit testing passed.

@nebula-community-bot
Copy link
Member

Unit testing failed.

@monadbobo
Copy link
Contributor Author

Jenkins go!

@nebula-community-bot
Copy link
Member

Unit testing failed.

@nebula-community-bot
Copy link
Member

Unit testing passed.

@dutor
Copy link
Contributor

dutor commented Oct 8, 2019

I didn't find any official documentation to describe how this sanitizer works, did you?

@monadbobo
Copy link
Contributor Author

I didn't find any official documentation to describe how this sanitizer works, did you?

http://llvm.org/docs/LibFuzzer.html

2. Added the AFL dictionary for fuzzy test
@nebula-community-bot
Copy link
Member

Unit testing passed.

@nebula-community-bot
Copy link
Member

Unit testing passed.

Copy link
Contributor

@dutor dutor left a comment

Choose a reason for hiding this comment

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

Great works!

CMakeLists.txt Outdated Show resolved Hide resolved
CMakeLists.txt Show resolved Hide resolved
@nebula-community-bot
Copy link
Member

Unit testing failed.

@nebula-community-bot
Copy link
Member

Unit testing passed.

Copy link
Contributor

@dutor dutor left a comment

Choose a reason for hiding this comment

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

LGTM.

@jude-zhu jude-zhu added this to the R201910_RC2 milestone Oct 28, 2019
Copy link
Contributor

@yixinglu yixinglu left a comment

Choose a reason for hiding this comment

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

LGTM

@nebula-community-bot
Copy link
Member

Unit testing passed.

@dangleptr
Copy link
Contributor

Awesome!

@dangleptr dangleptr merged commit bfdd1bf into vesoft-inc:master Dec 11, 2019
yixinglu pushed a commit to yixinglu/nebula that referenced this pull request Feb 16, 2020
* Issue vesoft-inc#714 Added the fuzzy test.

sumarry:

1. Added a new parameter "fuzzer" to nebula_add_test(cmake).
   Turn on this parameter indicates that the current test is a fuzzy test.
2. Currently only a simple fuzzy test has been added for paser module.
3. The current fuzzy test only supports Clang, so if the compiler was gcc it will skip.

* Move the fuzzer test from the "test" directory to the "fuzzing" directory.

* added the missed cmake file.

* 1. Added a document for fuzz test
2. Added the AFL dictionary for fuzzy test

* Fixed conflict.

* address @dutor's comment.

* Add a new cmake option ENABLE_FUZZ_TEST to control whether to enable the fuzz test.
tong-hao pushed a commit to tong-hao/nebula that referenced this pull request Jun 1, 2021
* Issue vesoft-inc#714 Added the fuzzy test.

sumarry:

1. Added a new parameter "fuzzer" to nebula_add_test(cmake).
   Turn on this parameter indicates that the current test is a fuzzy test.
2. Currently only a simple fuzzy test has been added for paser module.
3. The current fuzzy test only supports Clang, so if the compiler was gcc it will skip.

* Move the fuzzer test from the "test" directory to the "fuzzing" directory.

* added the missed cmake file.

* 1. Added a document for fuzz test
2. Added the AFL dictionary for fuzzy test

* Fixed conflict.

* address @dutor's comment.

* Add a new cmake option ENABLE_FUZZ_TEST to control whether to enable the fuzz test.
yixinglu pushed a commit to yixinglu/nebula that referenced this pull request Jan 31, 2023
* fix name of pr label

* fix conflict

Co-authored-by: Sophie <84560950+Sophie-Xie@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ready-for-testing PR: ready for the CI test
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants