Merge pull request #404 from localheinz/fix/auto-increment

Fix: No need to manually set primary key anymore
zendframework · Feb 20, 2015 · 1270d4c · 1270d4c
2 parents 18f3019 + 7d75fb0
commit 1270d4c
Showing 1 changed file with 0 additions and 8 deletions.
diff --git a/module/ZfModule/test/ZfModuleTest/Integration/Mapper/ModuleTest.php b/module/ZfModule/test/ZfModuleTest/Integration/Mapper/ModuleTest.php
@@ -174,25 +174,17 @@ public function testFindByLikeMatchesEntitiesWhereAllWordsExist()
     }
 
     /**
-     * @todo Id shouldn't need to be set, but currently missing auto_increment
-     * @link https://github.com/zendframework/modules.zendframework.com/pull/340
-     *
      * @return Entity\Module
      */
     private function module()
     {
-        static $id = 1;
-
         $module = new Entity\Module();
 
-        $module->setId($id);
         $module->setName('');
         $module->setOwner('');
         $module->setDescription('');
         $module->setUrl('');
 
-        $id++;
-
         return $module;
     }
 }