Field Permissions - 7.x-1.x

Test status
StatusResult
Last receivedTue, 01/10/2012 - 21:07:37
Last requestedTue, 01/10/2012 - 21:08:03
Last testedTue, 01/10/2012 - 21:09:00
Test count5
Environment status
MySQLpass
Code reviewfail
Overall Resultpass
Branch information
Identifier7.x-1.x
ProjectField Permissions
LinkProject information
SummaryPASSED: [[Coder]]: [Code review] 4 minor(s), 0 critical(s), and 2 normal(s); [[SimpleTest]]: [MySQL] 296 pass(es).
Details

MySQL

  1. Setup environment
  2. Detect an invalid patch URL
  3. Detect invalid repository information
  4. Detect invalid patch format
  5. Detect a non-applicable patch
  6. Detect invalid PHP syntax
  7. Detect a Drupal installation failure
  8. Detect a test run failure
  9. Detect a failing test
  10. Complete test suite with all tests passing
296 pass(es), 0 fail(s), and 0 exception(s)
Test namePassFailException
LeafField permissions functionality (FieldPermissionsTestCase) [Field permissions]29600

Code review

  1. Setup environment
  2. Detect an invalid patch URL
  3. Detect invalid repository information
  4. Detect invalid patch format
  5. Detect a non-applicable patch
  6. Detect invalid PHP syntax
  7. Detect a review run failure
  8. Detect coder flags
    Ensure that your code follows the Drupal standard and passes a coder review.
  9. Complete coder review with all passed
4 minor(s), 0 critical(s), and 2 normal(s)

Non-pass

Test nameMinorCriticalNormal
Expandfield_permissions.admin.inc001
MessageGroupFilenameLineFunctionStatus
The $text argument to l() should be enclosed within t() so that it is translatable.Otherfield_permissions.admin.inc384N/A
Expandfield_permissions.install001
MessageGroupFilenameLineFunctionStatus
Function documentation should be less than 80 characters per line.Otherfield_permissions.install20N/A

All

Test nameMinorCriticalNormal
Expandfield_permissions.admin.inc001
MessageGroupFilenameLineFunctionStatus
The $text argument to l() should be enclosed within t() so that it is translatable.Otherfield_permissions.admin.inc384N/A
Expandfield_permissions.install001
MessageGroupFilenameLineFunctionStatus
Function documentation should be less than 80 characters per line.Otherfield_permissions.install20N/A
Expandfield_permissions.module300
MessageGroupFilenameLineFunctionStatus
Format should be * Implementation of hook_foo().Otherfield_permissions.module74N/A
Format should be * Implementation of hook_foo().Otherfield_permissions.module101N/A
'Implementation' should be at the start of the sentence and begin with a capitialized letterOtherfield_permissions.module101N/A
Expandfield_permissions.test100
MessageGroupFilenameLineFunctionStatus
Use an indent of 2 spaces, with no tabsOtherfield_permissions.test304N/A