Standardize there names and meanings, and add prefixes to make them easier to document which plugins (or all) that they apply.
simpletest.tests
coder.reviews
global.directory
| Comment | File | Size | Author |
|---|---|---|---|
| #9 | 696434-standard-arguments.patch | 30.69 KB | boombatower |
| #8 | 696434-standard-arguments.patch | 34.21 KB | boombatower |
| #6 | 696434-standard-arguments.patch | 34.11 KB | boombatower |
| #5 | 696434-standard-arguments.patch | 30.14 KB | boombatower |
| #4 | 696434-standard-arguments.patch | 30.31 KB | boombatower |
Comments
Comment #1
boombatower commentedCurrent arguments:
Obviously latter two override pifr_drupal base...and database arugment needs to be removed.
Comment #2
boombatower commentedLets shoot for something like:
EDIT: Updated several times.
Comment #3
boombatower commentedThis patch should do it, but I'm looking into more generic implementations of test.* in client.inc instead of drupal/coder/simpletest clients.
Comment #4
boombatower commentedShift code around to fit new argument design, added some new useful arguments, replaced some class variables with arguments, updated debug test arrays, added update path, and fixed a few two bugs in code related to these arguments.
Comment #5
boombatower commentedUpdated the client plugin tests, which do not run on but...thus no fail until I ran them locally.
Comment #6
boombatower commentedAnother test fix.
Comment #8
boombatower commentedComment #9
boombatower commentedFinal tweaks to test.
Comment #10
boombatower commentedCommitted.