Skip to content

Commit

Permalink
Add more YAML tests for Access Control Cluster (#15589)
Browse files Browse the repository at this point in the history
* Add more YAML tests for Access Control Cluster

- Test some invalid entries
- Test some too many entries

Part of #10253

* Fix issue 13075

--When mQueueLength >= mQueueSize happens in CHIPCircularTLVBuffer::GetNewBuffer, if mProcessEvictedElement is set with AlwaysFail, EvictHead would fail when logging Event. We should let EvictHead and Init succeed, then later EnsureSpaceInCircularBuffer
would help get enough space for new event in LogEventPrivate.

add event overflow testing

Co-authored-by: Yunhan Wang <[email protected]>
  • Loading branch information
2 people authored and pull[bot] committed Nov 27, 2023
1 parent 8d6f4af commit 1723812
Show file tree
Hide file tree
Showing 4 changed files with 1,791 additions and 357 deletions.
5 changes: 0 additions & 5 deletions src/access/tests/TestAccessControl.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -146,11 +146,6 @@ constexpr NodeId validPaseSubjects[] = {
NodeIdFromPAKEKeyId(0x0001),
NodeIdFromPAKEKeyId(0xFFFE),
NodeIdFromPAKEKeyId(0xFFFF), // end

// Debatable whether these are valid or not,
// since they have bits in the unused part
// of the range set. Code currently treats
// them as valid (ignoring the unused bits).
};
// clang-format on

Expand Down
Loading

0 comments on commit 1723812

Please sign in to comment.