Топ-10 ошибок в автотестах на Java которые я вижу каждую неделю

1// Плохо: логика дублируется в каждом тесте
2@Test
3void testCheckout() {
4    open("https://shop.example.com/login");
5    $("#email").setValue("user@test.com");
6    $("#password").setValue("pass123");
7    $("button[type=submit]").click();
8    // ... 30 строк логики покупки
9}
10
11@Test
12void testOrderHistory() {
13    open("https://shop.example.com/login");
14    $("#email").setValue("user@test.com");  // дублирование
15    $("#password").setValue("pass123");     // дублирование
16    $("button[type=submit]").click();        // дублирование
17}

1// Хорошо: логика в одном месте, тесты читаемые
2public class LoginPage {
3    private final SelenideElement email    = $("#email");
4    private final SelenideElement password = $("#password");
5    private final SelenideElement submit   = $("button[type=submit]");
6
7    public MainPage loginAs(String email, String password) {
8        this.email.setValue(email);
9        this.password.setValue(password);
10        submit.click();
11        return new MainPage();
12    }
13}
14
15@Test
16void testCheckout() {
17    new LoginPage().loginAs("user@test.com", "pass123")
18        .goToCart()
19        .checkout();
20}

1// Плохо: хрупкие локаторы
2$("div.sc-bdXxxt.fKBQlc > span:nth-child(2)").click();  // XPath по структуре
3$(".btn-primary-large-rounded-shadow").click();          // CSS по визуальному классу
4$("//div[3]/button[1]").click();                         // XPath по позиции
5
6// Хорошо: стабильные локаторы
7$("[data-testid='submit-button']").click();   // атрибут для тестов
8$("[data-qa='login-form']").click();          // явный QA-атрибут
9$(byText("Оформить заказ")).click();           // по тексту (если текст стабилен)
10$("#checkout-submit").click();                // по ID

1// Плохо: не знает Stream API
2List<String> activeUsers = new ArrayList<>();
3for (User user : users) {
4    if (user.isActive()) {
5        activeUsers.add(user.getEmail());
6    }
7}
8
9// Хорошо: Stream API
10List<String> activeUsers = users.stream()
11    .filter(User::isActive)
12    .map(User::getEmail)
13    .toList();
14
15// Плохо: сравнение строк через ==
16if (status == "ACTIVE") { ... }
17
18// Хорошо: null-safe сравнение
19if (Objects.equals(status, "ACTIVE")) { ... }

1// Плохо: три одинаковых теста
2@Test void loginWithGmail()     { loginPage.loginAs("user@gmail.com",   "pass"); }
3@Test void loginWithCorporate() { loginPage.loginAs("user@company.ru",  "pass"); }
4@Test void loginWithSubdomain() { loginPage.loginAs("u@mail.example.com","pass"); }
5
6// Хорошо: @ParameterizedTest
7@ParameterizedTest
8@ValueSource(strings = {"user@gmail.com", "user@company.ru", "u@mail.example.com"})
9void loginWithValidEmail(String email) {
10    loginPage.loginAs(email, "pass");
11    assertThat(mainPage.isLoaded()).isTrue();
12}
13
14// Для сложных данных — @MethodSource
15@ParameterizedTest
16@MethodSource("validCredentials")
17void loginWithValidCredentials(String email, String password, String expectedRole) {
18    loginPage.loginAs(email, password);
19    assertThat(userProfile.getRole()).isEqualTo(expectedRole);
20}
21
22static Stream<Arguments> validCredentials() {
23    return Stream.of(
24        Arguments.of("admin@test.com",  "admin123",  "ADMIN"),
25        Arguments.of("user@test.com",   "user123",   "USER"),
26        Arguments.of("mentor@test.com", "mentor123", "MENTOR")
27    );
28}

1// Хорошо: Extension для автоматических скриншотов при падении
2public class ScreenshotExtension implements TestWatcher {
3    @Override
4    public void testFailed(ExtensionContext context, Throwable cause) {
5        String name = context.getDisplayName();
6        Selenide.screenshot("screenshots/" + name);
7        Allure.addAttachment("Screenshot", new FileInputStream("screenshots/" + name + ".png"));
8    }
9}
10
11// Хорошо: Extension для управления браузером
12public class BrowserExtension implements BeforeEachCallback, AfterEachCallback {
13    @Override
14    public void beforeEach(ExtensionContext ctx) {
15        Configuration.browser = "chrome";
16        Configuration.headless = true;
17        Configuration.baseUrl = System.getProperty("base.url", "https://app.example.com");
18    }
19    @Override
20    public void afterEach(ExtensionContext ctx) { closeWebDriver(); }
21}
22
23// Использование — чистые тесты без setup/teardown
24@ExtendWith({BrowserExtension.class, ScreenshotExtension.class})
25class LoginTests {
26    @Test
27    void loginTest() { /* только бизнес-логика */ }
28}

1// Плохо
2$("#submit").click();
3Thread.sleep(3000);  // ждём 3 секунды вслепую
4$("#success").shouldBe(visible);
5
6// Хорошо: Selenide ждёт автоматически (до 4 сек по умолчанию)
7$("#submit").click();
8$("#success").shouldBe(visible);
9
10// Хорошо: кастомное время ожидания если нужно
11$("#heavy-report").shouldBe(visible, Duration.ofSeconds(30));
12
13// Хорошо: для чистого Selenium — WebDriverWait
14new WebDriverWait(driver, Duration.ofSeconds(10))
15    .until(ExpectedConditions.visibilityOfElementLocated(By.id("success")));

1// Плохо: тест зависит от состояния после предыдущего
2@Test @Order(1)
3void createUser() { /* создаёт пользователя в БД */ }
4
5@Test @Order(2)
6void updateUser() { /* упадёт если createUser не запускался */ }
7
8// Хорошо: каждый тест самодостаточен
9@Test
10void updateUser() {
11    // создаём данные прямо здесь через API
12    User user = userApi.create(UserFactory.defaultUser());
13    userApi.update(user.getId(), UpdateRequest.builder().name("New Name").build());
14    assertThat(userApi.get(user.getId()).getName()).isEqualTo("New Name");
15}

1// Плохо: всё через UI, включая подготовку данных (5 минут на тест)
2@Test
3void testOrderHistory() {
4    loginPage.loginAs("user", "pass");
5    catalogPage.open().addToCart("Product A");
6    cartPage.checkout().fillAddress(address).pay();
7    orderHistoryPage.open().assertOrderExists("Product A");
8}
9
10// Хорошо: подготовка через API, UI только для проверки отображения (30 секунд)
11@Test
12void testOrderHistory() {
13    String token = authApi.login("user", "pass");
14    Order order  = orderApi.create(token, OrderRequest.of("Product A"));
15
16    open("/orders");
17    $("[data-testid='order-" + order.getId() + "']").shouldBe(visible);
18}