Tests needed: user.module

I will do my hardest best to complete this test within the next two weeks.

Is this issue still active? The above URL gives me an error. By the way, if it is, it would still need some testing.

The user role and permissions tests are being added in #300993: User roles and permissions API.

From the list in the initial issue:

4) Submitting a username with an invalid character

Coverage seems to have been started, but all illegal characters do not seem to be tested. It only seems that the forward slash is tested for.

9) Signatures in the issue list above:

There appears to be no tests for existence or validity of signature in user.test

A test might include:
1) enabling of signatures
2) creation of new user
3) creation of valid and invalid signatures for user
4) enable comment module if not so
5) create comment to check that signature appears
6) remove comment and created user

Missing "Who's New" block test. (#6 on the list above)

No user search test committed yet. (#5 on the list above)

Test missing for creating a new role (for example: "test" user role), and assigning user(s) to that role. Also, multiple role needs to be accounted for.

It would be expected that the following would be in the test:
- Create new user.
- Create new role(s).
- Apply core permission(s) to the new role.
- ...
- revoke role from user
- remove role from site

Albeit really needed, this is not critical.

Tagging.

subscribe

the attached patch has tests for the "who's new" block and for signatures...

There is one weirdness on signatures: The test I'm using works (user with signature can leave a comment and the comment will have the correct sig), but for some (inexplicable) reason I cannot seem to verify that the user has properly created a signature when filling out the profile form. To see what I'm referring to, please see the @todo in this patch. Un commenting that chunk *should* work, but it does not.

Any ideas on that? Once I get past that roadblock I will try to make a few more user tests

ooops ... forgot patch

obviously this will fail testing ... I still need someone to review though

Ignore comment #15 .. clearly I was drunk (aka meant to post that on another "Tests" issue I was working on)

looking at this error

Turns out that this test found an error when using signatures: #699594: Comments throw error when user has a signature

Once that issue is fixed, the test in #13 *should* be working

This patch adds one more small test for signatures ... again, #699594: Comments throw error when user has a signature needs to be fixed before this patch will pass

Ok ... patch in #18 is waiting for #520760: SA-CORE-2009-007 user signature format to land before testbot will be happy with it

sub

Added a SimpleTest that creates a user and role from the api using user_save and user_role save, as well as removing the role with user_role_delete. As far as I understand what's supposed to be happening, the code in the test should pass, but two of the tests fail. These tests relate to the Role ID being returned from the database not returning as an int, as expected from user_role_delete. Additional eyes are welcome.

Had to modify the patch as it had git-styled dates and was messing up the patching process. When I run this test it passes all.

subscribing

passed all tests

#23: 276582-additionalusertests-v2.patch queued for re-testing.

All 924 user tests passed for 276582-additionalusertests-v2.patch.

Hm. I can't get this to apply cleanly. Can someone give it a re-roll?

I could apply it, problem possible caused by the /no newline at the end of the file.
Anyway: reroll

+++ modules/user/user.test	23 Apr 2010 08:43:03 -0000
@@ -1585,3 +1585,67 @@
+    ¶

white space issues

+++ modules/user/user.test	23 Apr 2010 08:43:03 -0000
@@ -1585,3 +1585,67 @@
+    $this->assertFalse(user_access('administer nodes', $myTestUser), t('User does not have permission to administer content.'));   ¶

more white space

+++ modules/user/user.test	23 Apr 2010 08:43:03 -0000
@@ -1585,3 +1585,67 @@
+    $perm = db_query('SELECT rid FROM {role_permission} WHERE rid = :rid AND permission = :permission',

I'm sure this is just something that I'm unfamiliar with, but why are we using ":" here and not "%"?

95 critical left. Go review some!

The ':uid' markup is what's in the database abstraction layer documentation.

http://drupal.org/node/310072

I understand that's how it was in D6, and if % signs are the correct way to go, I'll modify the code.

dont change it ... I just hadnt seen that syntax before. Based on the link you posted, it certainly seems correct. Still have white space issues thought ;)

Details, details... ;-)

Patch takes care of the white space issues in the new test. I'd like to run it through code review, but it screams when I try to enable in on HEAD.

here testbot ... here boy

bump, easy review for somebody...

Looked over the patch at comment #35. The patch logic seems fine to me and test passes. The patch line numbers did not match up for me but was able to put in the changes. Although the variable names are camel case when I think they should be lower case with underscores. See the Drupal coding standard:

http://drupal.org/coding-standards#naming

This is a little confusing since objects are camel case in SimpleTest. Should this be changed? Looks like other test are using lower case with underscores for variable names.

Stumbled upon this old issue. AFAICS, all the tests listed in the original post can be found in user.test in D7. I assume the work was done and committed somewhere else, so I set this issue fixed.