Merge pull request #18 from adhocore/fix-io-case

Fix io case

Author: adhocore

README.md

@@ -13,7 +13,7 @@ Framework agnostic Command Line Interface utilities and helpers for PHP. Build C
 - Inspired by nodejs [commander](https://github.com/tj/commander.js) (thanks tj)
 - For PHP7 and for good
 
-![Screen Preview](https://i.imgur.com/qIYg9Zn.gif "Preview from adhocore/phalcon-ext which uses this cli package")
+[![Screen Preview](https://i.imgur.com/qIYg9Zn.gif "Preview from adhocore/phalcon-ext which uses this cli package")](https://github.com/adhocore/phalcon-ext/tree/master/example/cli)
 
 #### What's included
 
@@ -198,6 +198,8 @@ $name = $interactor->prompt('Name', null, $nameValidator, 5);
 $interactor->greenBold("The name is: $name", true);
 ```
 
+![Interactive Preview](https://i.imgur.com/qYBNd29.gif "Interactive Preview")
+
 ### IO Components
 
 The interactor is composed of `Ahc\Cli\Input\Reader` and `Ahc\Cli\Output\Writer` while the `Writer` itself is composed of `Ahc\Cli\Output\Color`. All these components can be used standalone.

src/IO/Interactor.php

@@ -205,7 +205,7 @@ protected function isValidChoice($choice, array $choices, bool $case)
             $choices = \array_keys($choices);
         }
 
-        $fn = ['\strcmp', '\strcasecmp'][(int) $case];
+        $fn = ['\strcasecmp', '\strcmp'][(int) $case];
 
         foreach ($choices as $option) {
             if ($fn($choice, $option) == 0) {

tests/IO/InteractorTest.php

@@ -52,6 +52,18 @@ public function test_confirm_more()
         $this->assertFalse($i->confirm('OK?'));
     }
 
+    public function test_case_sensitivity()
+    {
+        $i = $this->newInteractor('A'); // `A` is not `a`
+        $this->assertSame('b', $i->choice('Select one', ['a', 'b', 'c'], 'b', true));
+
+        $i   = $this->newInteractor('B');
+        $cho = $i->choice('Select one', ['a', 'b', 'c'], 'c', false);
+
+        $this->assertNotSame('c', $cho);
+        $this->assertSame('B', $cho);
+    }
+
     public function test_choice()
     {
         $i = $this->newInteractor('a');