From 40e6427cf1c4a244126bae7251c3b645b58c2498 Mon Sep 17 00:00:00 2001 From: Robert Hafner Date: Wed, 24 Sep 2014 18:45:01 -0700 Subject: [PATCH 1/2] Updated phpunit --- composer.json | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/composer.json b/composer.json index 36808e0..a5b86b9 100644 --- a/composer.json +++ b/composer.json @@ -17,8 +17,8 @@ }, "require-dev": { "tedivm/dovecottesting": "1.2.3", - "phpunit/phpunit": "4.0.*", "fabpot/php-cs-fixer": "0.4.0", + "phpunit/phpunit": "4.2.*", "satooshi/php-coveralls": "dev-master" }, From e8ae09423f5a0b85b32e6502f854d41d638da7ec Mon Sep 17 00:00:00 2001 From: Robert Hafner Date: Wed, 24 Sep 2014 18:46:49 -0700 Subject: [PATCH 2/2] Updated php-cs-fixer --- composer.json | 2 +- tests/Fetch/Test/ServerTest.php | 28 ++++++++++++++-------------- 2 files changed, 15 insertions(+), 15 deletions(-) diff --git a/composer.json b/composer.json index a5b86b9..43ca4a1 100644 --- a/composer.json +++ b/composer.json @@ -17,8 +17,8 @@ }, "require-dev": { "tedivm/dovecottesting": "1.2.3", - "fabpot/php-cs-fixer": "0.4.0", "phpunit/phpunit": "4.2.*", + "fabpot/php-cs-fixer": "0.5.*", "satooshi/php-coveralls": "dev-master" }, diff --git a/tests/Fetch/Test/ServerTest.php b/tests/Fetch/Test/ServerTest.php index 128e424..6044c4b 100644 --- a/tests/Fetch/Test/ServerTest.php +++ b/tests/Fetch/Test/ServerTest.php @@ -19,7 +19,7 @@ use Fetch\Server; */ class ServerTest extends \PHPUnit_Framework_TestCase { - static $num_messages_inbox = 12; + public static $num_messages_inbox = 12; /** * @dataProvider flagsDataProvider @@ -40,7 +40,7 @@ class ServerTest extends \PHPUnit_Framework_TestCase public function testFlagOverwrite() { - $server = Static::getServer(); + $server = static::getServer(); $server->setFlag('TestFlag', 'true'); $this->assertAttributeContains('TestFlag=true', 'flags', $server); @@ -97,14 +97,14 @@ class ServerTest extends \PHPUnit_Framework_TestCase public function testNumMessages() { - $server = Static::getServer(); + $server = static::getServer(); $numMessages = $server->numMessages(); $this->assertEquals(self::$num_messages_inbox, $numMessages); } public function testGetMessages() { - $server = Static::getServer(); + $server = static::getServer(); $messages = $server->getMessages(5); $this->assertCount(5, $messages, 'Five messages returned'); @@ -115,7 +115,7 @@ class ServerTest extends \PHPUnit_Framework_TestCase public function testGetMessagesOrderedByDateAsc() { - $server = Static::getServer(); + $server = static::getServer(); $messages = $server->getOrdered(SORTDATE, false, 2); $this->assertCount(2, $messages, 'Two messages returned'); @@ -124,7 +124,7 @@ class ServerTest extends \PHPUnit_Framework_TestCase public function testGetMessagesOrderedByDateDesc() { - $server = Static::getServer(); + $server = static::getServer(); $messages = $server->getOrdered(SORTDATE, true, 2); $this->assertCount(2, $messages, 'Two messages returned'); @@ -133,7 +133,7 @@ class ServerTest extends \PHPUnit_Framework_TestCase public function testGetMailBox() { - $server = Static::getServer(); + $server = static::getServer(); $this->assertEquals('', $server->getMailBox()); $this->assertTrue($server->setMailBox('Sent')); $this->assertEquals('Sent', $server->getMailBox()); @@ -141,7 +141,7 @@ class ServerTest extends \PHPUnit_Framework_TestCase public function testSetMailBox() { - $server = Static::getServer(); + $server = static::getServer(); $this->assertTrue($server->setMailBox('Sent')); $this->assertEquals('Sent', $server->getMailBox()); @@ -157,7 +157,7 @@ class ServerTest extends \PHPUnit_Framework_TestCase public function testHasMailBox() { - $server = Static::getServer(); + $server = static::getServer(); $this->assertTrue($server->hasMailBox('Sent'), 'Has mailbox "Sent"'); $this->assertTrue($server->hasMailBox('Flagged Email'), 'Has mailbox "Flagged Email"'); @@ -166,7 +166,7 @@ class ServerTest extends \PHPUnit_Framework_TestCase public function testListMailbox() { - $server = Static::getServer(); + $server = static::getServer(); $spec = sprintf('{%s:143/novalidate-cert}', TESTING_SERVER_HOST); $list = $server->listMailbox('*'); @@ -176,7 +176,7 @@ class ServerTest extends \PHPUnit_Framework_TestCase public function testCreateMailbox() { - $server = Static::getServer(); + $server = static::getServer(); $this->assertFalse($server->hasMailBox('Cheese'), 'Does not have mailbox "Cheese"'); $this->assertTrue($server->createMailBox('Cheese'), 'createMailbox returns true.'); @@ -188,20 +188,20 @@ class ServerTest extends \PHPUnit_Framework_TestCase */ public function testSetOptionsException() { - $server = Static::getServer(); + $server = static::getServer(); $server->setOptions('purple'); } public function testSetOptions() { - $server = Static::getServer(); + $server = static::getServer(); $server->setOptions(5); $this->assertAttributeEquals(5, 'options', $server); } public function testExpunge() { - $server = Static::getServer(); + $server = static::getServer(); $message = $server->getMessageByUid(12); $this->assertInstanceOf('\Fetch\Message', $message, 'Message exists');