Repository navigation
Support CTest - #1468
Support CTest#1468
Conversation
Add HTTPLIB_TEST option Downlaod GoogleTest source using FetchContent module Generate cert files to test httplib with OpenSSL
|
Testing with sudo apt install -y libssl-dev libbrotli-dev zlib1g-dev |
|
For some reason, generating PS. tests in CI sometimes fail due to network issues. |
|
@jimmy-park, thanks for the pull request. I found some embedded strings, 1) physical paths such as I actually don't maintain any CMake related files. @sum01 kindly takes care of those files, so that I don't need to worry about it. Could you willingly take a responsibility to maintain Thanks for your understanding! |
|
@yhirose Thanks for replying.
Since
We can choose the latest main branch using I saw comments and understand what you are concerned about. I'm willing to maintain Thanks for the useful library 😄 |
|
@jimmy-park thanks for the answers, and accept maintaining it! As you mentioned, could you try to use Google Test from the latest main branch? |
|
@yhirose Sure! |
|
@jimmy-park, thanks for the fine contribution! |
Tachi107
left a comment
There was a problem hiding this comment.
Sorry for my late review, but I wasn't notified about this patch until the 0.12.0 release.
Mostly minor issues/suggestions
|
@jimmy-park could you replay to @Tachi107's comments and suggestions? |
|
@jimmy-park yes, please go ahead to make another pull request for the adjustments. Thanks! |
* Add test/CMakeLists.txt to enable testing with CTest Add HTTPLIB_TEST option Downlaod GoogleTest source using FetchContent module Generate cert files to test httplib with OpenSSL * Generate cert2.pem with a new certificate request * Use the latest GoogleTest library
I added
test/CMakeLists.txtbecause the current testing workflow is separate from the CMake build system.Now we can do this for testing in a cross-platform way.
I also tested
httplibwithOpenSSLas shown intest/Makefile.This requires me to generate some cert files in the build directory.
Here is results on my machine (Windows 10)
It may test with
Brotliandzlibas well if libraries are installed correctly.