Selenium点击函数未触发失败，测试用例无论元素存在与否均通过求方案

我来帮你拆解下问题所在，以及对应的解决方案：

• 异常捕获类型不对：你在click方法里捕获的是AssertionError，但Selenium找不到元素时抛出的是NoSuchElementException（属于WebDriver异常体系），这个异常根本不会被当前的catch块处理。所以当元素不存在时，异常会直接向外抛出，但后续测试用例的处理又出了问题。
• 测试用例未标记失败：你的测试方法accept_cookies_dialog里，即使捕获到异常，也只是记录日志，没有告诉JUnit这个测试用例失败了。JUnit的逻辑是：只要测试方法没有抛出未被捕获的异常，就会判定为通过。
• click方法未传递异常：就算你捕获到正确的异常，当前click方法只是打印信息后return，上层测试用例根本感知不到元素查找失败的情况。

1. 修正click方法的异常处理 #

把捕获的异常类型改成正确的NoSuchElementException，并且重新抛出异常让上层测试用例处理，这样失败信息能传递到测试层：

import org.openqa.selenium.NoSuchElementException;

public void click(By element) {
    try {
        driver.findElement(element).click();
    } catch (NoSuchElementException e) {
        String errorMsg = "Element " + element + " not found on page";
        System.out.println(errorMsg);
        // 重新抛出异常，让测试用例捕获并标记失败
        throw new NoSuchElementException(errorMsg, e);
    }
}

2. 修改测试用例，主动标记失败 #

在测试方法的catch块中，除了记录日志，还要用Assert.fail()明确让JUnit知道这个测试用例失败了：

import org.junit.Assert;
import org.openqa.selenium.NoSuchElementException;

@Test(priority = 1)
public void accept_cookies_dialog() throws Exception {
    try {
        click(By.id(propObjctRepo.getProperty("id_cookieCta")));
    } catch (NoSuchElementException e) {
        addErrorlogs(e, "Not found accept cookie dialog.");
        // 主动触发测试失败，告诉JUnit这个用例不通过
        Assert.fail("Failed to accept cookies: " + e.getMessage());
    }
}

3. 可选优化：加入显式等待（推荐） #

为了避免因为元素加载延迟导致的误判，建议在click方法中加入显式等待，确保元素可点击后再执行点击操作：

import org.openqa.selenium.support.ui.WebDriverWait;
import org.openqa.selenium.support.ui.ExpectedConditions;
import java.time.Duration;
import org.openqa.selenium.NoSuchElementException;
import org.openqa.selenium.TimeoutException;

public void click(By element) {
    try {
        // 等待10秒，直到元素可点击
        WebDriverWait wait = new WebDriverWait(driver, Duration.ofSeconds(10));
        wait.until(ExpectedConditions.elementToBeClickable(element)).click();
    } catch (NoSuchElementException | TimeoutException e) {
        String errorMsg = "Element " + element + " not found or not clickable within timeout";
        System.out.println(errorMsg);
        throw new RuntimeException(errorMsg, e);
    }
}

这样你的测试用例就能准确捕获元素不存在或不可点击的情况，并正确标记为失败了。

内容的提问来源于stack exchange，提问作者Bhupendra Singh Rautela