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.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

socketwench’s picture

Assigned: Unassigned » socketwench
FileSize
6.39 KB

WIP patch.

ChrisKennedy’s picture

Status: Active » Needs work
FileSize
54.06 KB

Oops, 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:

Screenshot

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."

tripper54’s picture

Title: Module has no tests » [meta] Tests for Advanced Poll
Assigned: socketwench » Unassigned
Category: Feature request » Task

Thanks 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!

tripper54’s picture

tripper54’s picture