Commit 2d5e7421 authored by Vaclav Brozek's avatar Vaclav Brozek Committed by Commit Bot

Fix FormParserTest.CVC

The test was meant to check what happens if a password field has a
CVC-like name. Instead, it tested what happens if a text field has
this name.

This CL fixes that mistake. The test thankfully still passes.

This is a yak-shaving for adding the fallback pass, hence the below
bug association.

Bug: 906584
Change-Id: I9228c0252c97397fb5486e8a62c4cfb6a3f14cf6
Reviewed-on: https://chromium-review.googlesource.com/c/1343269Reviewed-by: default avatarVadym Doroshenko <dvadym@chromium.org>
Commit-Queue: Vaclav Brozek <vabr@chromium.org>
Cr-Commit-Position: refs/heads/master@{#609710}
parent 40525a18
...@@ -1358,7 +1358,7 @@ TEST(FormParserTest, CVC) { ...@@ -1358,7 +1358,7 @@ TEST(FormParserTest, CVC) {
"Name of 'verification_type' matches the CVC pattern.", "Name of 'verification_type' matches the CVC pattern.",
{ {
{.role = ElementRole::USERNAME, .form_control_type = "text"}, {.role = ElementRole::USERNAME, .form_control_type = "text"},
{.form_control_type = "text", .name = "verification_type"}, {.form_control_type = "password", .name = "verification_type"},
{.role = ElementRole::CURRENT_PASSWORD, {.role = ElementRole::CURRENT_PASSWORD,
.form_control_type = "password"}, .form_control_type = "password"},
}, },
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment