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

Ddfbra 222 update webform module #1808

Open
wants to merge 4 commits into
base: develop
Choose a base branch
from

Conversation

Dresse
Copy link
Contributor

@Dresse Dresse commented Nov 28, 2024

Link to issue

https://reload.atlassian.net/browse/DDFBRA-222

Description

This PR does the following:

  • Adds 5 new Cypress tests for testing out the webform, honeypot and antibot modules by testing the default contact webform found at /kontakt.

  • Updates the current webform module from version 6.2.2 to newest stable version 6.2.8

Testing

To test the upgraded version, I have relied on the added Cypress tests (added them before upgrading), and then ran them after upgrading to newest version.

Furthermore I went over the changelogs from 6.2.2 -> 6.2.8.
6.2.3
6.2.4
6.2.5
6.2.8

6.2.6 and 6.2.7 are not relevant, as they solve an issue with a missing drupal version compatibility declaration in composer.json in earlier versions.

The list is quite big, so I did not dive into every single issue, but instead I have tested the core functionality of how we are using the webform module:

  • Creating new forms.
  • Adding new elements to forms.
  • Editing existing forms.
  • Adding new email handlers.
  • Submitting forms.
  • Checking anti-spam modules.
  • Checking that our hooks in dpl_webform still works as intended (hiding certain fields, setting custom permissions on some fields, setting default values on some fields etc.)

elements.

Added 5 new Cypress tests for testing webform, honeypot and antibot
functionality.

These tests will be used for testing the modules after updating them.
need to wait for the honeypot timer to run out before submitting the form.
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

Successfully merging this pull request may close these issues.

6 participants