Updated test cases for addMany() fix.

agiledivider · Jan 2, 2012 · 3086047 · 3086047
1 parent 66b1235
commit 3086047
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 3 deletions.
diff --git a/library/Zend/Mail/AddressList.php b/library/Zend/Mail/AddressList.php
@@ -84,7 +84,7 @@ public function addMany(array $addresses)
             if (is_int($key) || is_numeric($key)) {
                 $this->add($value);
             } elseif (is_string($key)) {
-                $this->add($key, value);
+                $this->add($key, $value);
             } else {
                 throw new Exception\RuntimeException(sprintf(
                     'Invalid key type in provided addresses array ("%s")',

diff --git a/tests/Zend/Mail/AddressListTest.php b/tests/Zend/Mail/AddressListTest.php
@@ -98,7 +98,7 @@ public function testCanAddManyAddressesAtOnce()
     {
         $addresses = array(
             '[email protected]',
-            'ZF Contributors List' => '[email protected]',
+            '[email protected]' => 'ZF Contributors List',
             new Address('[email protected]', 'ZF Announce List'),
         );
         $this->list->addMany($addresses);
@@ -112,7 +112,6 @@ public function testDoesNotStoreDuplicatesAndFirstWins()
     {
         $addresses = array(
             '[email protected]',
-            'ZF DevTeam' => '[email protected]',
             new Address('[email protected]', 'ZF DevTeam'),
         );
         $this->list->addMany($addresses);