test getObjectUsers() and fix a bug found by the tests

git-svn-id: https://semanticscuttle.svn.sourceforge.net/svnroot/semanticscuttle/trunk@484 b3834d28-1941-0410-a4f8-b48e95affb8f

src/SemanticScuttle/Service/User.php

@@ -155,6 +155,7 @@ class SemanticScuttle_Service_User extends SemanticScuttle_DbService
             return false;
         }
 
+        $users = array();
         while ($row = & $this->db->sql_fetchrow($dbresult)) {
             $users[] = new SemanticScuttle_Model_User(
                 $row[$this->getFieldName('primary')],

tests/UserTest.php

@@ -53,6 +53,7 @@ class UserTest extends TestBase
     protected function setUp()
     {
         $this->us = SemanticScuttle_Service_Factory::get('User');
+        $this->us->deleteAll();
     }
 
 
@@ -118,6 +119,51 @@ class UserTest extends TestBase
         );
     }
 
+
+
+    /**
+     * Check if getObjectUsers() without any user works
+     *
+     * @return void
+     */
+    public function testGetObjectUsersNone()
+    {
+        $users = $this->us->getObjectUsers();
+        $this->assertEquals(0, count($users));
+    }
+
+
+
+    /**
+     * Check if getObjectUsers() with a single user works
+     *
+     * @return void
+     */
+    public function testGetObjectUsersSingle()
+    {
+        $uid = $this->addUser();
+        $users = $this->us->getObjectUsers();
+        $this->assertEquals(1, count($users));
+        $this->assertType('SemanticScuttle_Model_User', reset($users));
+    }
+
+
+
+    /**
+     * Check if getObjectUsers() with a several users works
+     *
+     * @return void
+     */
+    public function testGetObjectUsersMultiple()
+    {
+        $uid = $this->addUser();
+        $uid2 = $this->addUser();
+        $uid3 = $this->addUser();
+        $users = $this->us->getObjectUsers();
+        $this->assertEquals(3, count($users));
+        $this->assertType('SemanticScuttle_Model_User', reset($users));
+    }
+
 }