Skip to content

Draft: Enable JS to test new Vue password input - 2

Eduardo Sanz García requested to merge eduardosanz/vue-password-tests-3 into master

What does this MR do and why?

In the HAML, we replace the password field for a text field placeholder. This forces specs to use JavaScript execution to test the Vue password component as real users do.

On the second commit there are 7 specs that are still broken:

  • ee/spec/features/admin/admin_settings_spec.rb
  • spec/features/admin/admin_appearance_spec.rb
  • spec/features/admin/admin_projects_spec.rb
  • spec/features/admin/admin_runners_spec.rb
  • spec/features/admin/admin_settings_spec.rb
  • spec/features/admin/users/users_spec.rb
  • spec/features/profiles/active_sessions_spec.rb
  • spec/features/projects/features_visibility_spec.rb
  • spec/features/usage_stats_consent_spec.rb

EE: true

Screenshots or screen recordings

No impact

How to set up and validate locally

N/A

MR acceptance checklist

This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.

Merge request reports