Make a node type, set some node type specific comment variables, delete the node type, check the variables have gone too.

Comments

joshuajabbour’s picture

joshuajabbour commented
Component: tests » comment.module
Assigned: Unassigned » joshuajabbour
joshuajabbour’s picture

joshuajabbour commented

The tests for comment.module appear to need some more work.

There's a bug in the setCommentsPerPage function on line 146 (it appends 'article' to the variable name twice), but this doesn't cause any of the tests to fail. I'm not sure why

Also a lot of the strings in assertions aren't run through t(), so I'm guessing they'd fail on non-English sites. I'll look into these issues more and create a patch to fix.

joshuajabbour’s picture

joshuajabbour commented
Assigned: joshuajabbour » Unassigned

Removing myself for now... won't have time to work on this for a few weeks.

catch’s picture

catch
he/him
Primary language English
commented
Category: bug » task
Priority: Critical » Normal

Moving this out of the critical bugs queue - see #607038: Meta issue: fix gaps in code coverage.

Status: Active » Closed (outdated)

Automatically closed because Drupal 7 security and bugfix support has ended as of 5 January 2025. If the issue verifiably applies to later versions, please reopen with details and update the version.