Include dev deps & fix Server::findroute countable warning (#39)

* Add phpunit as dev dep

* update tests with PHPUnit\Framework\Testcase

* force params with ? in request URL

* return empty array instead of null to satisfy countable

* Include 5.3 precise test runner

* Use earlier version of phpunit

.gitignore

@@ -0,0 +1 @@
+vendor/

.travis.yml

@@ -1,11 +1,15 @@
 language: php
 
 php:
-  - 5.3
   - 5.4
   - 5.5
   - 5.6
 
+matrix:
+  include:
+    - php: 5.3
+      dist: precise
+
 before_script:
   # Composer
   - wget http://getcomposer.org/composer.phar

RestService/Server.php

@@ -1181,9 +1181,9 @@ public function argumentName($pName)
     public function findRoute($pUri, $pMethod = '_all_')
     {
         if (isset($this->routes[$pUri][$pMethod]) && $method = $this->routes[$pUri][$pMethod]) {
-            return array($method, null, $pMethod, $pUri);
+            return array($method, array(), $pMethod, $pUri);
         } elseif ($pMethod != '_all_' && isset($this->routes[$pUri]['_all_']) && $method = $this->routes[$pUri]['_all_']) {
-            return array($method, null, $pMethod, $pUri);
+            return array($method, array(), $pMethod, $pUri);
         } else {
             //maybe we have a regex uri
             foreach ($this->routes as $routeUri => $routeMethods) {

Test/Basic/BasicTest.php

@@ -5,7 +5,7 @@
 use RestService\Server;
 use Test\Controller\MyRoutes;
 
-class BasicTest extends \PHPUnit_Framework_TestCase
+class BasicTest extends \PHPUnit\Framework\TestCase
 {
     public function testCustomUrl()
     {
@@ -20,4 +20,4 @@ public function testCustomUrl()
 }', $response);
 
     }
-}
+}

Test/Synthetic/CollectTest.php

@@ -5,7 +5,7 @@
 use RestService\Server;
 use Test\Controller\MyRoutes;
 
-class CollectTest extends \PHPUnit_Framework_TestCase
+class CollectTest extends \PHPUnit\Framework\TestCase
 {
     /**
      * @var Server

Test/Synthetic/CustomRoutesTest.php

@@ -5,7 +5,7 @@
 use RestService\Server;
 use Test\Controller\MyRoutes;
 
-class CustomRoutesTest extends \PHPUnit_Framework_TestCase
+class CustomRoutesTest extends \PHPUnit\Framework\TestCase
 {
 
     public function testOwnController()
@@ -14,7 +14,7 @@ public function testOwnController()
             ->setClient('RestService\\InternalClient')
             ->addPostRoute('login', 'postLogin');
 
-        $response = $restService->simulateCall('/login', 'post');
+        $response = $restService->simulateCall('/login?', 'post');
 
         $this->assertEquals('{
     "status": 400,

Test/Synthetic/RouteTest.php

@@ -4,7 +4,7 @@
 
 use RestService\Server;
 
-class RouteTest extends \PHPUnit_Framework_TestCase
+class RouteTest extends \PHPUnit\Framework\TestCase
 {
 
     public function testAllRoutesClosures()

composer.json

@@ -17,5 +17,8 @@
         "psr-0": {
             "RestService": ""
         }
+    },
+    "require-dev": {
+        "phpunit/phpunit": "^4"
     }
-}
+}