Role Expire - 7.x-1.x | Drupal Quality Assurance

Latest changes

So Long and Thanks for all the Tests

1 week 2 days ago

Support

In case of troubles

check the FAQ. Please make sure you've tried the test locally.
search known testbot issues and then create a new issue if yours isn't there already. Please give a link to the test.

Test client logs

No clients to display.

Role Expire - 7.x-1.x

Test status

Status	Result
Last received	Mon, 04/09/2012 - 16:03:41
Last requested	Mon, 04/09/2012 - 16:04:03
Last tested	Mon, 04/09/2012 - 16:04:49
Test count	7

Environment status

MySQL	fail
Code review	fail

Overall Result	fail

Branch information

Identifier	7.x-1.x
Project	Role Expire
Link	Project information

Event logView entire log

ID	Message	Timestamp
3551959	Result retrieved by project client.	3 years 46 weeks ago
3551954	Result retrieved by project client.	3 years 46 weeks ago
3551949	Result received from test client #.	3 years 46 weeks ago
3551944	Result received from test client #659.	3 years 46 weeks ago
3551939	Requested by test client #659.	3 years 46 weeks ago
3551934	Requested by test client #.	3 years 46 weeks ago
3551929	Test request received.	3 years 46 weeks ago
3551909	Result retrieved by project client.	3 years 46 weeks ago
3551899	Result retrieved by project client.	3 years 46 weeks ago
3551894	Result received from test client #659.	3 years 46 weeks ago

SummaryFAILED: [[Coder]]: [Code review] 26 minor(s), 0 critical(s), and 64 normal(s); [[SimpleTest]]: [MySQL] 80 pass(es), 9 fail(s), and 1 exception(s).

Details

MySQL
Code review

MySQL

Setup environment
Detect an invalid patch URL
Detect invalid repository information
Detect invalid patch format
Detect a non-applicable patch
Detect invalid PHP syntax
Detect a Drupal installation failure
Detect a test run failure
Detect a failing test
Review the test failures below then if necessary run them locally until the problem has been fixed.
Complete test suite with all tests passing

80 pass(es), 9 fail(s), and 1 exception(s)

Non-pass

Test name

Pass

Fail

Exception

Role Expire web tests (RoleExpireWebTestCase) [Role Expire]

Message	Group	Filename	Line	Function
Making sure role checkbox is still on page.	Other	role_expire.test	119	RoleExpireWebTestCase->testRoleExpire_tests()
Testing aborted before completion: checkbox missing.	Other	role_expire.test	121	RoleExpireWebTestCase->testRoleExpire_tests()
Failed to set field roles[2] to 2	Other	role_expire.test	171	RoleExpireWebTestCase->testAssignAndRemoveRole()
Found the requested form fields at user/4/edit	Other	role_expire.test	171	RoleExpireWebTestCase->testAssignAndRemoveRole()
"The changes have been saved." found	Other	role_expire.test	172	RoleExpireWebTestCase->testAssignAndRemoveRole()
Role is assigned.	Browser	role_expire.test	173	RoleExpireWebTestCase->testAssignAndRemoveRole()
Failed to set field roles[2] to	Other	role_expire.test	176	RoleExpireWebTestCase->testAssignAndRemoveRole()
Found the requested form fields at user/4/edit	Other	role_expire.test	176	RoleExpireWebTestCase->testAssignAndRemoveRole()
"The changes have been saved." found	Other	role_expire.test	177	RoleExpireWebTestCase->testAssignAndRemoveRole()
Role is removed from user.	Browser	role_expire.test	178	RoleExpireWebTestCase->testAssignAndRemoveRole()

All

Test name

Pass

Fail

Exception

Role Expire web tests (RoleExpireWebTestCase) [Role Expire]

Message	Group	Filename	Line	Function
Making sure role checkbox is still on page.	Other	role_expire.test	119	RoleExpireWebTestCase->testRoleExpire_tests()
Testing aborted before completion: checkbox missing.	Other	role_expire.test	121	RoleExpireWebTestCase->testRoleExpire_tests()
Failed to set field roles[2] to 2	Other	role_expire.test	171	RoleExpireWebTestCase->testAssignAndRemoveRole()
Found the requested form fields at user/4/edit	Other	role_expire.test	171	RoleExpireWebTestCase->testAssignAndRemoveRole()
"The changes have been saved." found	Other	role_expire.test	172	RoleExpireWebTestCase->testAssignAndRemoveRole()
Role is assigned.	Browser	role_expire.test	173	RoleExpireWebTestCase->testAssignAndRemoveRole()
Failed to set field roles[2] to	Other	role_expire.test	176	RoleExpireWebTestCase->testAssignAndRemoveRole()
Found the requested form fields at user/4/edit	Other	role_expire.test	176	RoleExpireWebTestCase->testAssignAndRemoveRole()
"The changes have been saved." found	Other	role_expire.test	177	RoleExpireWebTestCase->testAssignAndRemoveRole()
Role is removed from user.	Browser	role_expire.test	178	RoleExpireWebTestCase->testAssignAndRemoveRole()

Review log

Code review

Setup environment
Detect an invalid patch URL
Detect invalid repository information
Detect invalid patch format
Detect a non-applicable patch
Detect invalid PHP syntax
Detect a review run failure
Detect coder flags
Ensure that your code follows the Drupal standard and passes a coder review.
Complete coder review with all passed

26 minor(s), 0 critical(s), and 64 normal(s)

Non-pass

Test name Minor Critical Normal

role_expire.module 18 0 23

Message	Group	Filename	Line	Function
put a space between the asterisk and the comment text	Other	role_expire.module	13	N/A
Function summaries should be one line only. (Drupal Docs)	Other	role_expire.module	16	N/A
If you define a @param or @return, you should document it as well.	Other	role_expire.module	21	N/A
Function summaries should be one line only. (Drupal Docs)	Other	role_expire.module	30	N/A
If you define a @param or @return, you should document it as well.	Other	role_expire.module	33	N/A
If you define a @param or @return, you should document it as well.	Other	role_expire.module	52	N/A
If you define a @param or @return, you should document it as well.	Other	role_expire.module	65	N/A
Use an indent of 2 spaces, with no tabs	Other	role_expire.module	88	N/A
Use an indent of 2 spaces, with no tabs	Other	role_expire.module	90	N/A
Use an indent of 2 spaces, with no tabs	Other	role_expire.module	91	N/A
Use an indent of 2 spaces, with no tabs	Other	role_expire.module	92	N/A
Use an indent of 2 spaces, with no tabs	Other	role_expire.module	93	N/A
Function summaries should be one line only. (Drupal Docs)	Other	role_expire.module	108	N/A
If you define a @param or @return, you should document it as well.	Other	role_expire.module	111	N/A
Function documentation should be less than 80 characters per line.	Other	role_expire.module	112	N/A
Use an indent of 2 spaces, with no tabs	Other	role_expire.module	132	N/A
Use an indent of 2 spaces, with no tabs	Other	role_expire.module	134	N/A
Use an indent of 2 spaces, with no tabs	Other	role_expire.module	135	N/A
Use an indent of 2 spaces, with no tabs	Other	role_expire.module	136	N/A
Function summaries should be one line only. (Drupal Docs)	Other	role_expire.module	150	N/A
put a space between the asterisk and the comment text	Other	role_expire.module	181	N/A
Function summaries should be one line only. (Drupal Docs)	Other	role_expire.module	253	N/A
in most cases, replace the string function with the drupal_ equivalent string functions	Other	role_expire.module	257	N/A
Function summaries should be one line only. (Drupal Docs)	Other	role_expire.module	269	N/A
The $message argument to drupal_set_message() should be enclosed within t() so that it is translatable.	Other	role_expire.module	283	N/A
in most cases, replace the string function with the drupal_ equivalent string functions	Other	role_expire.module	301	N/A
There should be no trailing spaces	Other	role_expire.module	372	N/A
Doxygen uses @todo and @bug to markup things to be done.	Other	role_expire.module	373	N/A
in most cases, replace the string function with the drupal_ equivalent string functions	Other	role_expire.module	392	N/A
Doxygen uses @todo and @bug to markup things to be done.	Other	role_expire.module	418	N/A
Format should be `* Implementation of hook_foo().`	Other	role_expire.module	427	N/A
'Implementation' should be at the start of the sentence and begin with a capitialized letter	Other	role_expire.module	427	N/A
missing space after comma	Other	role_expire.module	445	N/A
Use Drupal's format_date(), not PHP's default date().	Other	role_expire.module	479	N/A
put a space between the asterisk and the comment text	Other	role_expire.module	492	N/A
Function summaries should be one line only. (Drupal Docs)	Other	role_expire.module	495	N/A
Function summaries should be one line only. (Drupal Docs)	Other	role_expire.module	514	N/A
If you define a @param or @return, you should document it as well.	Other	role_expire.module	515	N/A
curly braces { should end a line, not start one	Other	role_expire.module	534	N/A
Function summaries should be one line only. (Drupal Docs)	Other	role_expire.module	547	N/A
Use Drupal's format_date(), not PHP's default date().	Other	role_expire.module	562	N/A

role_expire.rules.inc 2 0 2

Message	Group	Filename	Line	Function
There should be no trailing spaces	Other	role_expire.rules.inc	106	N/A
Function summaries should be one line only. (Drupal Docs)	Other	role_expire.rules.inc	106	N/A
Control statements should have one space between the control keyword and opening parenthesis	Other	role_expire.rules.inc	112	N/A
There should be no trailing spaces	Other	role_expire.rules.inc	113	N/A

role_expire.views.inc 2 0 3

Message	Group	Filename	Line	Function
Function documentation should be less than 80 characters per line.	Other	role_expire.views.inc	11	N/A
Function documentation should be less than 80 characters per line.	Other	role_expire.views.inc	16	N/A
@see should always be followed by a filename, a URL, class/interface name (optionally including method), or a function name including ().	Other	role_expire.views.inc	19	N/A
@see references should be separated by "," followed by a single space and with no trailing punctuation	Other	role_expire.views.inc	19	N/A
Doxygen uses @todo and @bug to markup things to be done.	Other	role_expire.views.inc	112	N/A

tests/role_expire.test 2 0 36

Message	Group	Filename	Line	Function
@file block missing (Drupal Docs)	Other	role_expire.test	-1	N/A
string concatenation should be formatted with a space separating the operators (dot .) and non-quote terms	Other	role_expire.test	36	N/A
Arrays should be formatted with a space separating each element and assignment operator	Other	role_expire.test	40	N/A
missing space after comma	Other	role_expire.test	40	N/A
Arrays should be formatted with a space separating each element and assignment operator	Other	role_expire.test	42	N/A
missing space after comma	Other	role_expire.test	42	N/A
missing space after comma	Other	role_expire.test	65	N/A
string concatenation should be formatted with a space separating the operators (dot .) and non-quote terms	Other	role_expire.test	69	N/A
missing space after comma	Other	role_expire.test	69	N/A
string concatenation should be formatted with a space separating the operators (dot .) and non-quote terms	Other	role_expire.test	70	N/A
missing space after comma	Other	role_expire.test	71	N/A
Use an indent of 2 spaces, with no tabs	Other	role_expire.test	75	N/A
Use an indent of 2 spaces, with no tabs	Other	role_expire.test	76	N/A
string concatenation should be formatted with a space separating the operators (dot .) and non-quote terms	Other	role_expire.test	81	N/A
string concatenation should be formatted with a space separating the operators (dot .) and non-quote terms	Other	role_expire.test	82	N/A
missing space after comma	Other	role_expire.test	85	N/A
Use Drupal's format_date(), not PHP's default date().	Other	role_expire.test	85	N/A
missing space after comma	Other	role_expire.test	87	N/A
missing space after comma	Other	role_expire.test	88	N/A
missing space after comma	Other	role_expire.test	93	N/A
missing space after comma	Other	role_expire.test	94	N/A
missing space after comma	Other	role_expire.test	99	N/A
missing space after comma	Other	role_expire.test	100	N/A
missing space after comma	Other	role_expire.test	103	N/A
Use Drupal's format_date(), not PHP's default date().	Other	role_expire.test	103	N/A
missing space after comma	Other	role_expire.test	105	N/A
missing space after comma	Other	role_expire.test	106	N/A
missing space after comma	Other	role_expire.test	107	N/A
Doxygen uses @todo and @bug to markup things to be done.	Other	role_expire.test	114	N/A
missing space after comma	Other	role_expire.test	118	N/A
missing space after comma	Other	role_expire.test	119	N/A
missing space after comma	Other	role_expire.test	124	N/A
missing space after comma	Other	role_expire.test	134	N/A
Arrays should be formatted with a space separating each element and assignment operator	Other	role_expire.test	135	N/A
missing space after comma	Other	role_expire.test	135	N/A
missing space after comma	Other	role_expire.test	136	N/A
missing space after comma	Other	role_expire.test	150	N/A
missing space after comma	Other	role_expire.test	159	N/A

All