Conversation
|
vitaliset is a new contributor to projects/pcapplusplus. The PR must be approved by known contributors before it can be merged. The past contributors are: maflcko, egecetin, clementperon, sashashura, DonggeLiu, cvediver, zlowram |
|
I couldn’t quite figure out how to proceed here. Any suggestions would be appreciated! 🙏 |
Does |
Yes. Done that. PTAL. |
| echo "=== Packet++Test ===" | ||
| cd "$ROOT/Tests/Packet++Test" | ||
| # TODO: Skipping failing tests. | ||
| ./Bin/Packet++Test -x "VrrpCreateAndEditTest;TestMacAddress;TestTcpReassemblyRetran" |
There was a problem hiding this comment.
We don't have network atm in our environment for running tests. Should we just disable these if these will fail otherwise?
There was a problem hiding this comment.
Not sure if I understood. They are passing without any -n (no networking) flag. I assume they are ok (?).
There was a problem hiding this comment.
You were right! :) Ran again the ./infra/experimental/chronos/check_tests.sh script adding a --network none to the test docker run and I can confirm that some tests failed. Adding them to the skip list and updating the comment there.
|
Is this ready to merge? |
Yes. CI is green now. :) PTAL at the updated patch you suggested to fix the linking error: |
Adds
run_tests.shto the pcapplusplus project.run_tests.shis used as part of Chronos with cached builds: https://github.com/google/oss-fuzz/tree/master/infra/experimental/chronos#check-testsOutput of ./infra/experimental/chronos/check_tests.sh pcapplusplus c++: