lordgamez commented on PR #1793: URL: https://github.com/apache/nifi-minifi-cpp/pull/1793#issuecomment-2317025229
> Hi @lordgamez @szaszm > > I fixed the LoggerTests fmt formatting issue "[7def7a0](https://github.com/apache/nifi-minifi-cpp/pull/1793/commits/7def7a081390fa991302e5d4882a0eff494dbfa3)": > > > For conan, we install fmt 10.2.1 prebuilt conan package to not conflict with spdlog 1.14.0 prebuilt conan package. > > However, when we switched to a newer version of fmt, that caused the LoggerTest to fail, which didn't happen > > when we were using fmt 10.1.0 that was version installed when we used the standalone CMake approach. So, we updated > > our TEST_CASE for 'fmt formatting works with the logger' to explicity create the chrono minutes duration, > > followed by fmt format and then we pass our formatted minutes duration to our logger->log_critical, so we get > > the expected fmt formatting to be '1m'. We didn't have to do this explicitly when we used fmt 10.1.0 since even > > if we passed our logger->log_critical '1min', it would auto format to '1m'. For fmt 10.2.1 to have backward compatibility > > with fmt 10.1.0 and the result we expected from this TEST_CASE in LoggerTests, we made this update. > > I checked that when I build MiNiFi using conan and then build MiNiFi using standalone CMake, their LoggerTest PASSES. > > Please let me know if these updates to LoggerTests.cpp are sufficient. Then when you get another chance, can you review my PR and approve if you think its ready? Thanks for the fix @james94 ! It works now and all the tests pass (besides ControllerTests which is expected at this point). The only problem I found was that some of the HTTP tests seem to work differently and are taking way too much time to finish, especially ListenHTTPTests. Could this be due to different curl versions maybe? Test run when building from source: 201/203 Test #68: VerifyInvokeHTTPPostTest ..................... Passed 18.38 sec 199/203 Test #64: InvokeHTTPTests .............................. Passed 15.90 sec 198/203 Test #1: ListenHTTPTests .............................. Passed 13.93 sec Test run using conan: 217/219 Test #85: VerifyInvokeHTTPPostTest ..................... Passed 33.06 sec 218/219 Test #81: InvokeHTTPTests .............................. Passed 38.32 sec 219/219 Test #9: ListenHTTPTests .............................. Passed 102.88 sec -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
