Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
Problem/Motivation
Currently, Advanced Poll does not have any automated tests.
Proposed resolution
Write automated tests.
Remaining tasks
Write patch.
User interface changes
None.
API changes
None.
Comment | File | Size | Author |
---|---|---|---|
#1 | 2416957.1.noTests.patch | 6.39 KB | socketwench |
Comments
Comment #1
socketwench CreditAttribution: socketwench commentedWIP patch.
Comment #2
ChrisKennedy CreditAttribution: ChrisKennedy commentedOops, responding to the correct issue this time. Thanks for spearheading this - looking good.
The one test that didn't pass for me was the UI test. I get this result:
Does it work on your end? I'm wondering if the UI test needs to login or something to that effect, as the verbose message shows a Drupal page that displays "Access denied. You are not authorized to access this page."
Comment #3
tripper54 CreditAttribution: tripper54 as a volunteer commentedThanks for your work on this @socketwrench.
We badly need test coverage for this module.
The supplied patch no longer applies.
I'm going to split this issue up into a number of child issues with a single test for each. That way we can get stuff committed a bit more quickly. I figure any test coverage is better than none at all!
Comment #4
tripper54 CreditAttribution: tripper54 as a volunteer commentedComment #5
tripper54 CreditAttribution: tripper54 as a volunteer commented