Add more php unit test cases and fix php solution (blakeembrey#177)

Author: caoglish

solutions/php/reverse-words-in-string.php

@@ -2,7 +2,5 @@
 
 function reverseWordsInStrings($string)
 {
-    return implode(array_reverse(preg_split('/\s+/', $string)), ' ');
+    return trim(implode(array_reverse(preg_split('/\s+/', $string)), ' '));
 }
-
-?>

solutions/php/sum-of-array-plus-one.php

@@ -1,11 +1,10 @@
 <?php
 
-function SumOfIntegers ($integerArray) {
-  $sum = 0;
-  foreach ($integerArray as $valor) {
-    $sum += $valor;
-  }
-  return $sum + count($integerArray);
+function SumOfArrayPlusOne($integerArray)
+{
+    $sum = 0;
+    foreach ($integerArray as $valor) {
+        $sum += $valor;
+    }
+    return $sum + count($integerArray);
 }
-
-?>

tests/php/factorialTest.php

@@ -1,17 +1,16 @@
 <?php
 use PHPUnit\Framework\TestCase;
 
-require_once  'solutions/php/factorial.php';
+require_once 'solutions/php/factorial.php';
 
-class StackTest extends TestCase
+class FactorialTest extends TestCase
 {
     public function test_factorial()
     {
-        $this->assertEquals(1,factorial(1) );
-        $this->assertEquals(2,factorial(2) );
-        $this->assertEquals(6,factorial(3) );
-        $this->assertEquals(120,factorial(5) );
-        $this->assertEquals(2432902008176640000,factorial(20) );
+        $this->assertEquals(1, factorial(1));
+        $this->assertEquals(2, factorial(2));
+        $this->assertEquals(6, factorial(3));
+        $this->assertEquals(120, factorial(5));
+        $this->assertEquals(2432902008176640000, factorial(20));
     }
 }
-?>

tests/php/reverseWordsInStringTest.php

@@ -0,0 +1,16 @@
+<?php
+use PHPUnit\Framework\TestCase;
+
+require_once 'solutions/php/reverse-words-in-string.php';
+
+class reverseWordsInStringsTest extends TestCase
+{
+    public function test_reverseWordsInStrings()
+    {
+        $this->assertEquals("awesome! are Interviews", reverseWordsInStrings("Interviews are awesome!"));
+        $this->assertEquals("degree CS", reverseWordsInStrings(" CS degree"));
+        $this->assertEquals("degree CS", reverseWordsInStrings("CS degree"));
+        $this->assertEquals("degree CS", reverseWordsInStrings("CS degree "));
+        $this->assertEquals("degree CS", reverseWordsInStrings(" CS degree "));
+    }
+}

tests/php/sumOfArrayPlusOneTest.php

@@ -0,0 +1,13 @@
+<?php
+use PHPUnit\Framework\TestCase;
+
+require_once 'solutions/php/sum-of-array-plus-one.php';
+
+class sumOfArrayPlusOneTest extends TestCase
+{
+    public function test_SumOfArrayPlusOne()
+    {
+        $this->assertEquals(14, SumOfArrayPlusOne([1, 2, 3, 4]));
+
+    }
+}