🤖 AI 代码检视意见（回复本评论可解决检视意见，点击被检视代码行左侧的小头像可收起检视意见）

🟡 测试代码中存在重复的设置逻辑

位置: L2111-L2112 | 严重程度: Medium

❓ 问题描述

在 RunQuickMenu_003, RunQuickMenu_004, RunQuickMenu_005, 和 RunQuickMenu_006 四个测试用例中，都添加了相同的两行代码：webPattern->OnModifyDone(); 和 ASSERT_NE(webPattern->delegate_, nullptr);。这种重复违反了DRY（Don't Repeat Yourself）原则。如果未来测试的初始化步骤发生变化（例如，需要调用另一个初始化函数，或需要增加更多的断言），开发者必须记得在所有四个地方进行修改，这很容易导致不一致，并增加了维护负担和引入错误的几率。

💡 修复建议

建议将这两行重复代码提取到一个辅助函数中，或者如果它们适用于所有或大部分 WebSelectOverlayTest 的用例，可以将它们移动到测试夹具的 SetUp() 方法中。这样可以集中管理初始化逻辑，提高代码的可维护性和可读性。

// Example refactoring using a helper function within the test class
RefPtr<WebPattern> WebSelectOverlayTest::CreateInitializedWebPattern(const RefPtr<FrameNode>& frameNode)
{
    auto webPattern = frameNode->GetPattern<WebPattern>();
    ASSERT_NE(webPattern, nullptr);
    webPattern->OnModifyDone();
    ASSERT_NE(webPattern->delegate_, nullptr);
    return webPattern;
}

// Usage in the test
HWTEST_F(WebSelectOverlayTest, RunQuickMenu_003, TestSize.Level1)
{
    // ... create frameNode
    // stack->Push(frameNode);
    auto webPattern = CreateInitializedWebPattern(frameNode); // Replaces the 4 original lines
    MockPipelineContext::SetUp();
    WebSelectOverlay overlay(webPattern);
    // ...
}