Merge pull request #16 from graze/null-line-ending-support

Add support for null line endings

Author: john-n-smith

src/PromptMatcher.php

@@ -36,7 +36,7 @@ class PromptMatcher implements PromptMatcherInterface
     public function isMatch($prompt, $subject, $lineEnding = null)
     {
         // cheap line ending check before expensive regex
-        if (substr($subject, -1 * strlen($lineEnding)) != $lineEnding) {
+        if (!is_null($lineEnding) && substr($subject, -1 * strlen($lineEnding)) != $lineEnding) {
             return false;
         }
 

tests/unit/PromptMatcherTest.php

@@ -39,6 +39,7 @@ public function dataProviderIsMatch()
     {
         return [
             ['OK', 'this is a response', "\n", false],
+            ['OK', 'this is a response', null, false], // null line ending
             ['OK', "this is a response\nOK\n", "\n", true, ['OK'], 'this is a response'],
             ['(ERROR) ([0-9]{1,3})', "party\r\nERROR 123\r\n", "\r\n", true, ['ERROR 123', 'ERROR', '123'], 'party']
         ];