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 <yunhanw@google.com>
  • Loading branch information
mlepage-google and yunhanw-google authored Mar 2, 2022
1 parent 5924088 commit c2a5710
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 c2a5710

Please sign in to comment.