Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Consolidate / DRY up spam trap tests into a single spec #53

Open
lorenyu opened this issue Jan 6, 2025 · 0 comments
Open

Consolidate / DRY up spam trap tests into a single spec #53

lorenyu opened this issue Jan 6, 2025 · 0 comments

Comments

@lorenyu
Copy link

lorenyu commented Jan 6, 2025

from the following discussion

@lorenyu non-blocking thought: i see a lot of "handles submissions by bots" tests in this PR, i wonder if it makes sense to DRY that up somehow. maybe not but just wondering

@doshitan Yea not sure, just from the upstream changes: c7e83fa

@lorenyu yeah, thinking about it more, it seems like it should be a separate spec along the lines of

# psuedocode
describe "non-logged in forms"
  parametrize on { form: ["registration", "password reset", "login"] }
  it "includes honeypot fields and logic to catch bots that spam all fields"
    navigate to form
    spam all fields
    submit
    expect response to be 422
  end
end
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant