#
f0b59af4 |
| 20-Mar-2024 |
Ed Tanous <ed@tanous.net> |
Add misc-include-cleaner
And fix the includes that are wrong.
Note, there is a very large ignore list included in the .clang-tidy configcfile. These are things that clang-tidy doesn't yet handle w
Add misc-include-cleaner
And fix the includes that are wrong.
Note, there is a very large ignore list included in the .clang-tidy configcfile. These are things that clang-tidy doesn't yet handle well, like knowing about a details include.
Change-Id: Ie3744f2c8cba68a8700b406449d6c2018a736952 Signed-off-by: Ed Tanous <ed@tanous.net>
show more ...
|
#
6fb96ce3 |
| 28-Jan-2024 |
Ed Tanous <ed@tanous.net> |
Make tests not require body interaction
The muitipart test interacts with some significant details of the response class. This was largely only done because Request lacked an addHeader method that
Make tests not require body interaction
The muitipart test interacts with some significant details of the response class. This was largely only done because Request lacked an addHeader method that Request already had.
Add addHeader() method to the Request class, and adapt multipart unit tests to use it.
Tested: Unit tests pass. Unit test only changes.
Change-Id: Icb3b92dce6d17011ae0063a962678173b1b01a87 Signed-off-by: Ed Tanous <ed@tanous.net>
show more ...
|
#
77665bda |
| 12-Oct-2022 |
Nan Zhou <nanzhoumails@gmail.com> |
header cleanups
This commit fixed several places (but not all) where wrong include directory is specified and prevent the clean up in the chidren changes.
Signed-off-by: Nan Zhou <nanzhoumails@gmai
header cleanups
This commit fixed several places (but not all) where wrong include directory is specified and prevent the clean up in the chidren changes.
Signed-off-by: Nan Zhou <nanzhoumails@gmail.com> Change-Id: Ibbba62e2c0cfe3583a65f1befa1b233bd3eebf19
show more ...
|
#
18e3f7fb |
| 24-Aug-2022 |
Krzysztof Grobelny <krzysztof.grobelny@intel.com> |
Fixed issues with multipart parser
- Index was not checked against size before dereference. Which cased to override memory. - Header without colon could put parser into invalid state. Now it will
Fixed issues with multipart parser
- Index was not checked against size before dereference. Which cased to override memory. - Header without colon could put parser into invalid state. Now it will return with error. - Content after boundary was not correctly discarded. - Parser did not check body for final boudary. Now missing final boundary will return with error.
Tested: - Tested that payload with header without colon doesn't cause memory corruption anymore.
Signed-off-by: Krzysztof Grobelny <krzysztof.grobelny@intel.com> Change-Id: I12f496ab5f53e6c088cdfdf2e96be636d66f7c7f
show more ...
|
#
c33a039b |
| 10-Sep-2022 |
Nan Zhou <nanzhoumails@gmail.com> |
treewide: reorganize unit tests
Like other C++ projects, unit tests normally are in a separate repo and respect the folder structure of the file under test.
This commit deleted all "ut" folder and
treewide: reorganize unit tests
Like other C++ projects, unit tests normally are in a separate repo and respect the folder structure of the file under test.
This commit deleted all "ut" folder and move tests to a "test" folder. The test folder also has similar structure as the main folder.
This commit also made neccessary include changes to make codes compile. Unused tests are untouched.
Tested: unit test passed.
Reference: [1] https://github.com/grpc/grpc/tree/master/test [2] https://github.com/boostorg/core/tree/414dfb466878af427d33b36e6ccf84d21c0e081b/test [3] Many other OpenBMC repos: https://github.com/openbmc/entity-manager/tree/master/test [4] https://stackoverflow.com/questions/2360734/whats-a-good-directory-structure-for-larger-c-projects-using-makefile
Signed-off-by: Nan Zhou <nanzhoumails@gmail.com> Change-Id: I4521c7ef5fa03c47cca5c146d322bbb51365ee96
show more ...
|