As the post fields are not deleted, if the previous form was cached and there was an error then as the next request contains the former build id you will get a failure. This causes the last failure in drupalPost -- edit: I mean path -- tests, up until the module_list fix is not in the only way to test this is to change the module_exists call in path.inc to TRUE.

CommentFileSizeAuthor
#1 drupalpost.patch1.66 KBchx
drupalpost.patch1.7 KBchx

Comments

chx’s picture

chx commented
StatusFileSize
new drupalpost.patch1.66 KB

A simpler but better version of the same.

mikejoconnor’s picture

mikejoconnor commented
Status: Needs review » Reviewed & tested by the community

Test Case: modules/path/path.test v1.2

The test failed at line 120. Performing the same tasks manually showed that the error was with the test, and not the path module.

After applying drupalpost_0.patch the test passed. Everything looks good.

dries’s picture

dries commented
Status: Reviewed & tested by the community » Fixed

I've committed this to CVS HEAD. Thanks.

Anonymous’s picture

Anonymous (not verified) commented
Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for two weeks with no activity.