История вопроса.
В PageObject довольно удобно оформлять весь форумляр отдельным классом.
Соответственно в Java (эх, в Котлине это всё гораздо проще сделать) если разные тесты не всегда заполняют все поля - то надо или оверлоудить кучу методов для заполнения,
или строить билдеры (а значит тоже  - куча методов, иногда меньшая чем в оверлоуде, но это не точно), или (я так делаю, потому что практично) - передавать для полей, которые не надо заполнять (и не надо стирать) - null,  а в PO писать конструкцию типа if (value!=null) field.setValue(value); к полям которые можно игнорировать.

Вариант "игнорировать" - позволяет просто писать field.setValue(value) во всех случаях.

Вариант игнорировать очень злой, выльётся в часы траблшутинга на проектах

ну теперешний вариант - тоже 🙂

вы можете добавить default метод отдельный

по моему представлении об ООП, setValue(null) - выглядит странно, поэтому должен кидать эксепшн 🙂

И новый метод можно построить на базе старого, а вот старый на новом - нет

этот вариант как раз позволит избежать долгого траблшутинга

Блин, с вами придётся свечку ставить за backward compatibility макаронному монстру

да, этот вариант позволит по крайней мере не ломать жизнь существующие тесты разработчикам

как раз даже именно в данной редакции - более логично решается вопрос с а что если $("element-not-found").setValue(null);

По мне довольно типичный кейс, когда в формуляре некоторые поля даже исчезают, поэтому и искать не надо, сразу игнор.

Тесты становится правда fillPersonalData("John", null, null, "01.04.1984");  - менее читаемы в блокноте, но IntelliJ Hints же у всех включены, правда?

🤔

по правильному - кинуть бы эксепшн
но! селениум вроде как тоже позволяет передавать null

Мне эта идея кажется очень сомнительной.
Написать $.setValueIfPresent(foo) не проще, чем if (foo != null) $.setValue(foo).

Да, скорее уж я бы кидал ошибку в случае $.setValue(null).

и, вопрос - должны ли мы в нашем случае переопределять стандартное поведение?

То, что люди городят сложные конструкции с куче null параметров - на их совести. Нам-то зачем в это соваться.
Для этого в котлине, например, есть foo?.let {  $.setValue(foo)  }

отбой, про очистку значения в чистом селениуме