Review: User Module Test
boombatower - April 10, 2008 - 02:06
| Project: | SimpleTest |
| Version: | 7.x-1.x-dev |
| Component: | Code |
| Category: | task |
| Priority: | normal |
| Assigned: | boombatower |
| Status: | closed |
Jump to:
Description
The user module tests didn't pass before this review for several reasons. After review the tests now passes all except for default timezone check.
<?php
$this->assertEqual($user->timezone, variable_get('date_default_timezone', NULL), t('Correct timezone field.'));
?>After a look at the code this appears to be a bug.
Other than that this test has been cleaned up quiet a bit. I removed the access rules test since that is no longer in core.
As a note UserValidationTestCase is full of unit tests. Once unit testing is implemented they should be moved.
| Attachment | Size |
|---|---|
| simpletest_review_user.patch | 17.76 KB |
#1
Was going to open an issue for access rules and you've beaten me to it!
#2
Committed.
#3
Automatically closed -- issue fixed for two weeks with no activity.