Proxy - master

Test status
StatusResult
Last receivedThu, 03/22/2012 - 14:43:37
Last requestedThu, 03/22/2012 - 14:44:03
Last testedThu, 03/22/2012 - 14:44:28
Test count53
Environment status
MySQLfail
Code reviewfail
Overall Resultfail
Branch information
Identifiermaster
ProjectProxy
LinkProject information
SummaryFAILED: [[Coder]]: [Code review] 0 minor(s), 0 critical(s), and 12 normal(s); [[SimpleTest]]: [MySQL] 14 pass(es), 2 fail(s), and 1 exception(s).
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
    Review the test failures below then if necessary run them locally until the problem has been fixed.
  10. Complete test suite with all tests passing
14 pass(es), 2 fail(s), and 1 exception(s)

Non-pass

Test namePassFailException
ExpandProxyTest1421
MessageGroupFilenameLineFunctionStatus
Trying to get property of non-objectNoticeproxy.test44ProxyTest->testBasicProxy()
Ensure that the correct content is returned from drupal.org.Otherproxy.test44ProxyTest->testBasicProxy()
Ensure that the correct text shows on the page.Otherproxy.test63ProxyTest->testRouterProxy()

All

Test namePassFailException
ExpandProxyTest1421
MessageGroupFilenameLineFunctionStatus
Trying to get property of non-objectNoticeproxy.test44ProxyTest->testBasicProxy()
Ensure that the correct content is returned from drupal.org.Otherproxy.test44ProxyTest->testBasicProxy()
Ensure that the correct text shows on the page.Otherproxy.test63ProxyTest->testRouterProxy()

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
0 minor(s), 0 critical(s), and 12 normal(s)

Non-pass

Test nameMinorCriticalNormal
Expandincludes/proxy.admin.inc008
MessageGroupFilenameLineFunctionStatus
Use sentence case, not title case, for end-user strings. (Wikipedia)Otherproxy.admin.inc30N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otherproxy.admin.inc35N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otherproxy.admin.inc44N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otherproxy.admin.inc49N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otherproxy.admin.inc55N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otherproxy.admin.inc63N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otherproxy.admin.inc68N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otherproxy.admin.inc74N/A
Expandprocessors/proxy.processors.inc001
MessageGroupFilenameLineFunctionStatus
Use sentence case, not title case, for end-user strings. (Wikipedia)Otherproxy.processors.inc19N/A
Expandproxy.module001
MessageGroupFilenameLineFunctionStatus
Use the matching Drupal theme functions, not raw HTML.Otherproxy.module82N/A
Expandproxy.test002
MessageGroupFilenameLineFunctionStatus
Use the matching Drupal theme functions, not raw HTML.Otherproxy.test38N/A
Use the matching Drupal theme functions, not raw HTML.Otherproxy.test58N/A

All

Test nameMinorCriticalNormal
Expandincludes/proxy.admin.inc008
MessageGroupFilenameLineFunctionStatus
Use sentence case, not title case, for end-user strings. (Wikipedia)Otherproxy.admin.inc30N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otherproxy.admin.inc35N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otherproxy.admin.inc44N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otherproxy.admin.inc49N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otherproxy.admin.inc55N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otherproxy.admin.inc63N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otherproxy.admin.inc68N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otherproxy.admin.inc74N/A
Leafincludes/proxy.processor.inc000
Leafincludes/proxy.proxy.inc000
Leafincludes/proxy.router.inc000
Expandprocessors/proxy.processors.inc001
MessageGroupFilenameLineFunctionStatus
Use sentence case, not title case, for end-user strings. (Wikipedia)Otherproxy.processors.inc19N/A
Leafprocessors/proxy_processor_proxify_urls.inc000
Leafproxy.install000
Expandproxy.module001
MessageGroupFilenameLineFunctionStatus
Use the matching Drupal theme functions, not raw HTML.Otherproxy.module82N/A
Expandproxy.test002
MessageGroupFilenameLineFunctionStatus
Use the matching Drupal theme functions, not raw HTML.Otherproxy.test38N/A
Use the matching Drupal theme functions, not raw HTML.Otherproxy.test58N/A