As shown in the second list, test states, there is a "Reviewed" column. I plan to review each of the tests, clean them up, and ensure that they test the necessary functionality.
This patch cleans up the code, add comment blocks were necessary and makes the tests more consistent.
This test passed HEAD before I cleaned it and it still does.
If the community also approves that this patch cleans up the test and that the test is good enough to be called final then I will mark it accordingly on the list. This doesn't mean the test won't need updating in the future, but that it is a quality test that can be counted on the detect bugs in core changes.
Comment | File | Size | Author |
---|---|---|---|
#1 | simpletest_review_aggregator.patch | 14.16 KB | boombatower |
simpletest_review_aggregator.patch | 14.16 KB | boombatower | |
Comments
Comment #1
boombatower CreditAttribution: boombatower commentedUpdated patch.
Committed as suggested.
Any comments are still welcome.
Comment #2
Anonymous (not verified) CreditAttribution: Anonymous commentedAutomatically closed -- issue fixed for two weeks with no activity.