-
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/tests/ to use PW instead of NL. #33249
Updated unit tests in src/transport/tests/ to use PW instead of NL. #33249
Conversation
PR #33249: Size comparison from 335ae19 to 08fa55e 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)
|
@feasel0 this seems to need merge conflict fixing and restyle |
…nto feature/unittest--transport-tests
…m/feasel0/connectedhomeip into feature/unittest--transport-tests
PR #33249: Size comparison from 062e063 to e3afaa9 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 #33249: Size comparison from be19fc9 to 3674380 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 #33249: Size comparison from 3c5b92c to 10b7a72 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)
|
…nto feature/unittest--transport-tests
…instead of dynamically. Changed all tests to use XTestSuite instead of per-test SetUp/TearDown.
…m/feasel0/connectedhomeip into feature/unittest--transport-tests
PR #33249: Size comparison from c4e37c2 to 6bcbcb5 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)
|
… each test case is run.
…m/feasel0/connectedhomeip into feature/unittest--transport-tests
PR #33249: Size comparison from 16e220b to cbb528e 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 #33249: Size comparison from d75f73c to 2456239 Decreases (2 builds for efr32)
Full report (70 builds for bl602, bl702, bl702l, cc13x4_26x4, cc32xx, cyw30739, efr32, esp32, linux, nrfconnect, nxp, psoc6, qpg, stm32, telink)
|
PR #33249: Size comparison from 3b1c60a to 827022b Decreases (2 builds for efr32)
Full report (59 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. I updated the tests in src/transport/tests/
This PR replaces a previous PR #33155 which is now closed. The previous PR had some files accidentally included in it which were not supposed to be part of it.
All suggestions from that PR have been addressed.
Note that in
TestSecureSessionTable.cpp
, the test functionValidateSessionSorting
has to call another functionTestSecureSessionTable::ValidateSessionSorting
because the tests make reference toSecureSession::State
which is private. Putting the tests inside the scopeTestSecureSessionTable
allowsState
to be accessed sinceSecureSession
explicitly namesTestSecureSessionTable
as a friend class.