Skip to content
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

Implement shadow fail-safe data in FabricTable #19819

Merged
Changes from 1 commit
Commits
Show all changes
58 commits
Select commit Hold shift + click to select a range
54b18f9
Implement shadow fail-safe data in FabricTable
tcarmelveilleux Jun 21, 2022
9ae8c22
Restyled by clang-format
restyled-commits Jun 21, 2022
35c7eeb
Merge remote-tracking branch 'upstream/master' into update-fabrictabl…
tcarmelveilleux Jun 21, 2022
d37dccd
Fixes after merging master
tcarmelveilleux Jun 21, 2022
beff07e
Rename callback of FabricTable::Delegate
tcarmelveilleux Jun 21, 2022
003a59a
Rekick restyle
tcarmelveilleux Jun 21, 2022
a56451d
restyle
tcarmelveilleux Jun 21, 2022
98a7f51
Align last known good time commit / revert to shadow fail-safe approach
msandstedt Jun 21, 2022
6e6122e
Fix CI
tcarmelveilleux Jun 21, 2022
50c67f7
Merge pull request #9 from msandstedt/update-fabrictable-new-opcert-s…
tcarmelveilleux Jun 21, 2022
59d5e58
Fix TV app
tcarmelveilleux Jun 22, 2022
5b4874e
Merge remote-tracking branch 'origin/update-fabrictable-new-opcert-st…
tcarmelveilleux Jun 22, 2022
fa5157c
Reduce flash usage on K32W0 by disabling detail (verbose) logs
tcarmelveilleux Jun 22, 2022
b3a6461
Merge remote-tracking branch 'upstream/master' into update-fabrictabl…
tcarmelveilleux Jun 22, 2022
e94260b
Merge remote-tracking branch 'upstream/master' into update-fabrictabl…
tcarmelveilleux Jun 22, 2022
db03f18
Disable progress logging in TI platform lock example to save flash
tcarmelveilleux Jun 22, 2022
4835323
Disable progress logging in TI platform shell example to save flash
tcarmelveilleux Jun 22, 2022
8ea7610
Reduce stack usage of TestEventLogging
tcarmelveilleux Jun 22, 2022
45550c6
Apply review comments from @msandstedt
tcarmelveilleux Jun 22, 2022
52699c4
Save stack for Nordic build
tcarmelveilleux Jun 22, 2022
e62391e
Save stack for Nordic build, some more
tcarmelveilleux Jun 22, 2022
56c610d
Added unit tests for all basic operations
tcarmelveilleux Jun 23, 2022
ef143c6
Merge remote-tracking branch 'upstream/master' into update-fabrictabl…
tcarmelveilleux Jun 23, 2022
626e738
Fix merge conflict
tcarmelveilleux Jun 23, 2022
ed6db09
Restyle
tcarmelveilleux Jun 23, 2022
6962ec1
Merge remote-tracking branch 'upstream/master' into update-fabrictabl…
tcarmelveilleux Jun 23, 2022
5d019c8
Change nrf connect stack limit up by 512 bytes for fake unit test com…
tcarmelveilleux Jun 23, 2022
3decd98
Revert "Save stack for Nordic build"
tcarmelveilleux Jun 23, 2022
33ef943
Revert "Save stack for Nordic build, some more"
tcarmelveilleux Jun 23, 2022
5dd808c
Fix UpdateLabel
tcarmelveilleux Jun 23, 2022
3eb3a16
Apply review comments, add ACL fabric removal
tcarmelveilleux Jun 23, 2022
84a871c
Apply review comments
tcarmelveilleux Jun 23, 2022
2dda34d
per bzbarsky, clear in-memory Last Known Good Time if revert fails
msandstedt Jun 23, 2022
c3ebba5
Merge pull request #10 from msandstedt/update-fabrictable-new-opcert-…
tcarmelveilleux Jun 23, 2022
efa0c5f
Apply fixes from @bzbarsky-apple
tcarmelveilleux Jun 23, 2022
f4d541b
Merge remote-tracking branch 'origin/update-fabrictable-new-opcert-st…
tcarmelveilleux Jun 23, 2022
684d8e8
Apply more review comments
tcarmelveilleux Jun 23, 2022
d619d56
Restyled
tcarmelveilleux Jun 23, 2022
225ab8f
Fix tests
tcarmelveilleux Jun 23, 2022
0e33917
Add unit test for iterator
tcarmelveilleux Jun 23, 2022
f09d11f
Iterator now works
tcarmelveilleux Jun 23, 2022
05cc156
Merge remote-tracking branch 'upstream/master' into update-fabrictabl…
tcarmelveilleux Jun 23, 2022
eea33f0
Fix semantic merge conflict
tcarmelveilleux Jun 23, 2022
2b17b3e
Restyled
tcarmelveilleux Jun 23, 2022
4935173
Merge remote-tracking branch 'upstream/master' into update-fabrictabl…
tcarmelveilleux Jun 23, 2022
cbbf425
Fix predicates for storage presence temporarily
tcarmelveilleux Jun 23, 2022
5691e4a
Fix Darwin build
bzbarsky-apple Jun 23, 2022
92d340e
Merge pull request #11 from bzbarsky-apple/testing
tcarmelveilleux Jun 23, 2022
29f01dd
Add missing docs, move methods to private that shouldn't be public
tcarmelveilleux Jun 24, 2022
c182cfe
Merge remote-tracking branch 'upstream/master' into update-fabrictabl…
tcarmelveilleux Jun 24, 2022
3efe420
Change stack warning temporarily to pass on nRFConnect
tcarmelveilleux Jun 24, 2022
50818c4
Make MatterControllerFactory use const FabricInfo
tcarmelveilleux Jun 24, 2022
9f0b34e
Restyle
tcarmelveilleux Jun 24, 2022
54376dd
Merge remote-tracking branch 'upstream/master' into update-fabrictabl…
tcarmelveilleux Jun 24, 2022
d8864e3
Fix semantic conflict on SessionManager
tcarmelveilleux Jun 24, 2022
096a76a
Fix an init ordering issue in TestSessionManager.cpp
tcarmelveilleux Jun 24, 2022
21ab76b
Fix SessionManager shutdown
tcarmelveilleux Jun 24, 2022
444cbd5
Restyled
tcarmelveilleux Jun 24, 2022
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Reduce stack usage of TestEventLogging
tcarmelveilleux committed Jun 22, 2022

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.
commit 8ea7610ecb9c18b739cd7814faf0454f31de5677
14 changes: 9 additions & 5 deletions src/app/tests/TestEventLogging.cpp
Original file line number Diff line number Diff line change
@@ -36,6 +36,7 @@
#include <lib/support/CHIPCounter.h>
#include <lib/support/EnforceFormat.h>
#include <lib/support/ErrorStr.h>
#include <lib/support/ScopedBuffer.h>
#include <lib/support/UnitTestRegistration.h>
#include <lib/support/logging/Constants.h>
#include <messaging/ExchangeContext.h>
@@ -148,21 +149,24 @@ static void CheckLogReadOut(nlTestSuite * apSuite, chip::app::EventManagement &
chip::TLV::TLVReader reader;
chip::TLV::TLVWriter writer;
size_t eventCount = 0;
uint8_t backingStore[1024];

chip::Platform::ScopedMemoryBuffer<uint8_t> backingStore;
VerifyOrDie(backingStore.Alloc(1024));

size_t totalNumElements;
writer.Init(backingStore, 1024);
writer.Init(backingStore.Get(), 1024);
err = alogMgmt.FetchEventsSince(writer, clusterInfo, startingEventNumber, eventCount, chip::Access::SubjectDescriptor{});
NL_TEST_ASSERT(apSuite, err == CHIP_NO_ERROR || err == CHIP_END_OF_TLV);

reader.Init(backingStore, writer.GetLengthWritten());
reader.Init(backingStore.Get(), writer.GetLengthWritten());

err = chip::TLV::Utilities::Count(reader, totalNumElements, false);
NL_TEST_ASSERT(apSuite, err == CHIP_NO_ERROR);

printf("totalNumElements vs expectedNumEvents vs eventCount : %u vs %u vs %u \n", static_cast<unsigned int>(totalNumElements),
static_cast<unsigned int>(expectedNumEvents), static_cast<unsigned int>(eventCount));
NL_TEST_ASSERT(apSuite, totalNumElements == expectedNumEvents && totalNumElements == eventCount);
reader.Init(backingStore, writer.GetLengthWritten());
reader.Init(backingStore.Get(), writer.GetLengthWritten());
chip::TLV::Debug::Dump(reader, SimpleDumpWriter);
}

@@ -328,7 +332,7 @@ nlTestSuite sSuite =

int TestEventLogging()
{
TestContext gContext;
static TestContext gContext;
nlTestRunner(&sSuite, &gContext);
return (nlTestRunnerStats(&sSuite));
}