Posted by boombatower on March 30, 2008 at 11:37pm
2 followers
Jump to:
| Project: | SimpleTest |
| Component: | Code |
| Category: | task |
| Priority: | normal |
| Assigned: | boombatower |
| Status: | closed (fixed) |
Issue Summary
Filter module review.
This patch cleans up the code, adds comment blocks were necessary.
The test previously passed HEAD and passes after the patch. The patched removed allot of unnecessary regular expressions that have been outdated due to the new internal browser's support for assertField.
Included in this patch is a fix for the assertFieldByXPath method to handle multiple fields when checking values correctly. This is necessary on the filter page with the radio inputs.
| Attachment | Size | Status | Test result | Operations |
|---|---|---|---|---|
| simpletest_review_filter.patch | 11.13 KB | Ignored: Check issue status. | None | None |
Comments
#1
The modification to the
assertNoFieldByXPathmethod causes some issues with comment. I'm holding off till I can figure them out.#2
Marking as a patch.
#3
Committed.
Determined that the change to the
assertNoFieldByXPathworks correctly and there is an issue with the comment test.#4
Automatically closed -- issue fixed for two weeks with no activity.