From e929b83ce9a139d6b734864ac4ea5e0415d38d2d Mon Sep 17 00:00:00 2001 From: Boris Zbarsky Date: Wed, 7 Jun 2023 11:47:00 -0400 Subject: [PATCH] Use a non-chunked attribute for non-chunking tests in TestReadInteraction. (#27126) We were using an attribute that led to multiple reports, in a test that only cares about whether a report is delivered at all. There's no reason to do that, and it can lead to some timing-related failures if we don't spin the event loop long enough to get all the reports. --- src/app/tests/TestReadInteraction.cpp | 15 ++++++++++----- 1 file changed, 10 insertions(+), 5 deletions(-) diff --git a/src/app/tests/TestReadInteraction.cpp b/src/app/tests/TestReadInteraction.cpp index 249e5e991f83fa..57dd270f779dd8 100644 --- a/src/app/tests/TestReadInteraction.cpp +++ b/src/app/tests/TestReadInteraction.cpp @@ -4104,7 +4104,7 @@ void TestReadInteraction::TestSubscribeInvalidateFabric(nlTestSuite * apSuite, v readPrepareParams.mpAttributePathParamsList[0].mEndpointId = Test::kMockEndpoint3; readPrepareParams.mpAttributePathParamsList[0].mClusterId = Test::MockClusterId(2); - readPrepareParams.mpAttributePathParamsList[0].mAttributeId = Test::MockAttributeId(4); + readPrepareParams.mpAttributePathParamsList[0].mAttributeId = Test::MockAttributeId(1); readPrepareParams.mMinIntervalFloorSeconds = 0; readPrepareParams.mMaxIntervalCeilingSeconds = 0; @@ -4163,7 +4163,7 @@ void TestReadInteraction::TestShutdownSubscription(nlTestSuite * apSuite, void * readPrepareParams.mpAttributePathParamsList[0].mEndpointId = Test::kMockEndpoint3; readPrepareParams.mpAttributePathParamsList[0].mClusterId = Test::MockClusterId(2); - readPrepareParams.mpAttributePathParamsList[0].mAttributeId = Test::MockAttributeId(4); + readPrepareParams.mpAttributePathParamsList[0].mAttributeId = Test::MockAttributeId(1); readPrepareParams.mMinIntervalFloorSeconds = 0; readPrepareParams.mMaxIntervalCeilingSeconds = 0; @@ -4210,7 +4210,7 @@ void TestReadInteraction::TestSubscriptionReportWithDefunctSession(nlTestSuite * err = engine->Init(&ctx.GetExchangeManager(), &ctx.GetFabricTable()); NL_TEST_ASSERT(apSuite, err == CHIP_NO_ERROR); - AttributePathParams subscribePath(Test::kMockEndpoint3, Test::MockClusterId(2), Test::MockAttributeId(4)); + AttributePathParams subscribePath(Test::kMockEndpoint3, Test::MockClusterId(2), Test::MockAttributeId(1)); ReadPrepareParams readPrepareParams(ctx.GetSessionBobToAlice()); readPrepareParams.mpAttributePathParamsList = &subscribePath; @@ -4231,6 +4231,7 @@ void TestReadInteraction::TestSubscriptionReportWithDefunctSession(nlTestSuite * ctx.DrainAndServiceIO(); NL_TEST_ASSERT(apSuite, delegate.mGotReport); + NL_TEST_ASSERT(apSuite, delegate.mNumAttributeResponse == 1); NL_TEST_ASSERT(apSuite, engine->GetNumActiveReadHandlers(ReadHandler::InteractionType::Subscribe) == 1); NL_TEST_ASSERT(apSuite, engine->GetNumActiveReadHandlers(ReadHandler::InteractionType::Read) == 0); NL_TEST_ASSERT(apSuite, engine->GetReportingEngine().GetNumReportsInFlight() == 0); @@ -4244,12 +4245,14 @@ void TestReadInteraction::TestSubscriptionReportWithDefunctSession(nlTestSuite * // Test that we send reports as needed. readHandler->SetStateFlag(ReadHandler::ReadHandlerFlags::HoldReport, false); - delegate.mGotReport = false; + delegate.mGotReport = false; + delegate.mNumAttributeResponse = 0; engine->GetReportingEngine().SetDirty(subscribePath); ctx.DrainAndServiceIO(); NL_TEST_ASSERT(apSuite, delegate.mGotReport); + NL_TEST_ASSERT(apSuite, delegate.mNumAttributeResponse == 1); NL_TEST_ASSERT(apSuite, engine->GetNumActiveReadHandlers(ReadHandler::InteractionType::Subscribe) == 1); NL_TEST_ASSERT(apSuite, engine->GetNumActiveReadHandlers(ReadHandler::InteractionType::Read) == 0); NL_TEST_ASSERT(apSuite, engine->GetReportingEngine().GetNumReportsInFlight() == 0); @@ -4258,12 +4261,14 @@ void TestReadInteraction::TestSubscriptionReportWithDefunctSession(nlTestSuite * // up properly. readHandler->GetSession()->MarkAsDefunct(); readHandler->SetStateFlag(ReadHandler::ReadHandlerFlags::HoldReport, false); - delegate.mGotReport = false; + delegate.mGotReport = false; + delegate.mNumAttributeResponse = 0; engine->GetReportingEngine().SetDirty(subscribePath); ctx.DrainAndServiceIO(); NL_TEST_ASSERT(apSuite, !delegate.mGotReport); + NL_TEST_ASSERT(apSuite, delegate.mNumAttributeResponse == 0); NL_TEST_ASSERT(apSuite, engine->GetNumActiveReadHandlers(ReadHandler::InteractionType::Subscribe) == 0); NL_TEST_ASSERT(apSuite, engine->GetNumActiveReadHandlers(ReadHandler::InteractionType::Read) == 0); NL_TEST_ASSERT(apSuite, engine->GetReportingEngine().GetNumReportsInFlight() == 0);