Fix one testcase apparently broken by automatic formatting tools

[plcapi.git] / tests / InvalidHostTest.php

diff --git a/tests/InvalidHostTest.php b/tests/InvalidHostTest.php
index fb89b43..1ac111f 100644 (file)
--- a/tests/InvalidHostTest.php
+++ b/tests/InvalidHostTest.php
@@ -1,62 +1,58 @@
 <?php
 
-include_once(__DIR__.'/../lib/xmlrpc.inc');
+include_once __DIR__ . '/../lib/xmlrpc.inc';
 
-include_once(__DIR__.'/parse_args.php');
+include_once __DIR__ . '/parse_args.php';
 
 class InvalidHostTest extends PHPUnit_Framework_TestCase
 {
-    var $client = null;
-    var $args = array();
+    public $client = null;
+    public $args = array();
 
-    function setUp()
+    public function setUp()
     {
         $this->args = argParser::getArgs();
 
         $this->client = new xmlrpc_client('/NOTEXIST.php', $this->args['LOCALSERVER'], 80);
-        if($this->args['DEBUG'])
-        {
+        if ($this->args['DEBUG']) {
             $this->client->setDebug($this->args['DEBUG']);
         }
     }
 
-    function test404()
+    public function test404()
     {
-        $f = new xmlrpcmsg('examples.echo',array(
-            new xmlrpcval('hello', 'string')
+        $f = new xmlrpcmsg('examples.echo', array(
+            new xmlrpcval('hello', 'string'),
         ));
         $r = $this->client->send($f, 5);
         $this->assertEquals(5, $r->faultCode());
     }
 
-    function testSrvNotFound()
+    public function testSrvNotFound()
     {
-        $f = new xmlrpcmsg('examples.echo',array(
-            new xmlrpcval('hello', 'string')
+        $f = new xmlrpcmsg('examples.echo', array(
+            new xmlrpcval('hello', 'string'),
         ));
         $this->client->server .= 'XXX';
         $r = $this->client->send($f, 5);
         // make sure there's no freaking catchall DNS in effect
         $dnsinfo = dns_get_record($this->client->server);
-        if($dnsinfo)
-        {
+        if ($dnsinfo) {
             $this->markTestSkipped('Seems like there is a catchall DNS in effect: host ' . $this->client->server . ' found');
-        }
-        else
-        {
+        } else {
             $this->assertEquals(5, $r->faultCode());
         }
     }
 
-    function testCurlKAErr()
+    public function testCurlKAErr()
     {
-        if(!function_exists('curl_init'))
-        {
+        if (!function_exists('curl_init')) {
             $this->markTestSkipped('CURL missing: cannot test curl keepalive errors');
+
             return;
         }
-        $f = new xmlrpcmsg('examples.stringecho',array(
-            new xmlrpcval('hello', 'string')
+        $f = new xmlrpcmsg('examples.stringecho', array(
+            new xmlrpcval('hello', 'string'),
         ));
         // test 2 calls w. keepalive: 1st time connection ko, second time ok
         $this->client->server .= 'XXX';
@@ -67,8 +63,7 @@ class InvalidHostTest extends PHPUnit_Framework_TestCase
 
         // now test a successful connection
         $server = explode(':', $this->args['LOCALSERVER']);
-        if(count($server) > 1)
-        {
+        if (count($server) > 1) {
             $this->client->port = $server[1];
         }
         $this->client->server = $server[0];
@@ -77,6 +72,6 @@ class InvalidHostTest extends PHPUnit_Framework_TestCase
         $r = $this->client->send($f, 5, 'http11');
         $this->assertEquals(0, $r->faultCode());
         $ro = $r->value();
-        is_object( $ro ) && $this->assertEquals('hello', $ro->scalarVal());
+        is_object($ro) && $this->assertEquals('hello', $ro->scalarVal());
     }
 }