Нет сомнений в том, что мы всегда должны выбирать css selector вместо xpath.
Но селектор CSS, который вы используете: .tv-button.tv-button--no-border-radius.tv-button--size_large.tv-button--primary_ghost.tv-button--loader
выглядит очень нестабильно.
Для просто кнопки входа, вы можете использовать:
xpath: //span[contains(text(),'Log In')]/parent::button
Причина, по которой вам следует избегать css-селектора, заключается в том, что (в данном случае) этот css-селектор является комбинацией имени класса, поэтому в случае изменения имени класса вам придется изменить локатор.
Вероятность изменения имени класса в этом случае весьма вероятна. Потому что это сделано из 5 классов.
Надеюсь, это поможет.