-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
Updated unit tests in src/transport/raw/tests/ to use PW instead of NL. #33099
Updated unit tests in src/transport/raw/tests/ to use PW instead of NL. #33099
Conversation
PR #33099: Size comparison from 742e65b to 400d0e5 Full report (40 builds for bl602, bl702, bl702l, cc13x4_26x4, cc32xx, cyw30739, esp32, linux, mbed, nrfconnect, nxp, psoc6, qpg, stm32)
|
PR #33099: Size comparison from 742e65b to 3490e83 Decreases (1 build for efr32)
Full report (71 builds for bl602, bl702, bl702l, cc13x4_26x4, cc32xx, cyw30739, efr32, esp32, linux, mbed, nrfconnect, nxp, psoc6, qpg, stm32, telink)
|
PR #33099: Size comparison from c4699cf to 544888f Decreases (1 build for efr32)
Full report (71 builds for bl602, bl702, bl702l, cc13x4_26x4, cc32xx, cyw30739, efr32, esp32, linux, mbed, nrfconnect, nxp, psoc6, qpg, stm32, telink)
|
…ing MemoryShutdown.
….com/feasel0/connectedhomeip into feature/unittest-transport-raw-tests
@andy31415 I reviewed the changes. I'm starting a holiday, so I might not be able to review them the last time. I think this is the last list of changes. |
PR #33099: Size comparison from 040e5bf to 11cc04c Decreases (2 builds for efr32)
Full report (71 builds for bl602, bl702, bl702l, cc13x4_26x4, cc32xx, cyw30739, efr32, esp32, linux, mbed, nrfconnect, nxp, psoc6, qpg, stm32, telink)
|
PR #33099: Size comparison from 6cdb5da to 4681a06 Decreases (2 builds for efr32)
Full report (71 builds for bl602, bl702, bl702l, cc13x4_26x4, cc32xx, cyw30739, efr32, esp32, linux, mbed, nrfconnect, nxp, psoc6, qpg, stm32, telink)
|
PR #33099: Size comparison from dfe64ee to c13b7aa Decreases (1 build for efr32)
Full report (71 builds for bl602, bl702, bl702l, cc13x4_26x4, cc32xx, cyw30739, efr32, esp32, linux, mbed, nrfconnect, nxp, psoc6, qpg, stm32, telink)
|
These changes are replacing NL unit tests with pigweed unit tests, continuing the efforts of #29342 and others.
I updated the tests in src/transport/raw/tests/
The order which the tests appear in TestTCP.cpp and TestUDP.cpp was changed so that they will be run in the same order that they were run in the old NL version.
The tests in TestTCP.cpp and TestUDP.cpp had fixtures that included an instance of chip::Test::IOContext which needed to go into the test class so that it is available to all the tests that use it.