TestingParty08: Filter tips - short tips
cwgordon7 - August 16, 2008 - 02:18
| Project: | Drupal |
| Version: | 7.x-dev |
| Component: | filter.module |
| Category: | task |
| Priority: | normal |
| Assigned: | Unassigned |
| Status: | needs work |
Description
We need to test the functionality of Drupal's filter tips. This issue focuses on the "short" filter tips - the ones that appear in an unordered list on the node/add page.
Suggested assertions:
- Create a filter with all of the core filters enabled, make sure the proper tips show up on the node/add page.
- Create several filters, and make sure each has its proper tips show up on the node/add page.
- Make sure the link to the longer tips shows up on the node/add page.
#1
hi folks,
we just are onto it - expect a patch in the next days ;)
#2
hi folks,
so here i got some test that does what is described above.
and i found a bug as well - "Correct broken HTML" does not have a help tip.
will file a bug report as well.
#3
well, ok - just found out i was a little bit too rigid.
so here is now a complete patch for the test class, which ignores missing help texts.
cu
#4
Test runs fine and passes but there's a few code style issues - tabs instead of spaces, munging together of variable names (should be $foo_bar), and code comments should read as full sentences.
#5
Few code style corrections.
#6
#7
The last submitted patch failed testing.
#8
Re-roll.
Fail :
$this->assertEqual(htmlentities((string)$fields[0]->ul->li[$filter->delta - $missingHelpCount]), $tips[$filter->delta], t('Filter help text match !filter', array('!filter' => $filter->name)));#9
#10
#11
The last submitted patch failed testing.
#12
HEAD is broken : http://testing.drupal.org/node/35
#13
The last submitted patch failed testing.
#14
Moving this out of the critical bugs queue - see #607038: Meta issue: fix gaps in code coverage.