Closed (fixed)
Project:
Drupal core
Version:
7.x-dev
Component:
tests
Priority:
Critical
Category:
Bug report
Assigned:
Reporter:
Created:
4 Jun 2008 at 10:26 UTC
Updated:
20 Jun 2008 at 13:12 UTC
Jump to comment: Most recent file
Found the requested form at admin/settings/filters/add [Other] Fail
Found the Save configuration button [Other] Fail
Failed to set field name to simpletest_w93x [Other] Fail
Failed to set field roles[2] to 1 [Other] Fail
Failed to set field filters[filter/2] to 1
There are other failures which are due to core bugs rather than the test, RTBC patch here: http://drupal.org/node/161217
| Comment | File | Size | Author |
|---|---|---|---|
| #2 | filter.test.patch | 832 bytes | boombatower |
Comments
Comment #1
catchIssue is with the createFormat() function which just looks like it's not building the $edit array properly (compared to the filter admin test which passes). I guess it'd be useful to have a reusable function so we could test different combinations of filters and how they interact, but it'll need some work. Definitely a bug in the test rather than anything else though.
Comment #2
boombatower commentedThe array looks fine.
This was the culprit.
should be
Very simple to check just place the following code after a
drupalGetordrupalPost.I received an access error, so I check the permissions...
Placed that same code after
The assertions are working as they should. The page output contains the links that the code is asserting shouldn't be there. So either the assertions are wrong or the filter is broken.
I would like someone else to take a look at that since I'm not familiar with how the filter should work.
Comment #3
catchThe filter's a bug in the code and there's an RTBC patch here: http://drupal.org/node/161217
This one's good.
Comment #4
dries commentedCommitted to CVS HEAD. Is there a way we can check a permission actually exists? That might prevent these errors.
Comment #5
boombatower commentedDoesn't seem like this got committed.
Comment #6
boombatower commentedAlso patch for checking permissions: #267308: Simpletest: check permissions to make sure they are valid.
Comment #7
catchI got committed as collateral here so missed getting it's own message: http://drupal.org/cvs?commit=119893
Comment #8
Anonymous (not verified) commentedAutomatically closed -- issue fixed for two weeks with no activity.