Only redraw() when necessary

Author: clue

src/Readline.php

@@ -65,6 +65,10 @@ public function __construct($output)
      */
     public function setPrompt($prompt)
     {
+        if ($prompt === $this->prompt) {
+            return $this;
+        }
+
         $this->prompt = $prompt;
 
         return $this->redraw();
@@ -83,9 +87,18 @@ public function setPrompt($prompt)
      */
     public function setEcho($echo)
     {
+        if ($echo === $this->echo) {
+            return $this;
+        }
+
         $this->echo = !!$echo;
 
-        return $this->redraw();
+        // only redraw if there is any input
+        if ($this->linebuffer !== '') {
+            $this->redraw();
+        }
+
+        return $this;
     }
 
     /**
@@ -102,9 +115,7 @@ public function setMove($move)
     {
         $this->move = !!$move;
 
-        $this->linepos = $this->strlen($this->linebuffer);
-
-        return $this->redraw();
+        return $this->moveCursorTo($this->strlen($this->linebuffer));
     }
 
     /**
@@ -134,10 +145,19 @@ public function getCursorPosition()
      */
     public function setInput($input)
     {
+        if ($this->linebuffer === $input) {
+            return $this;
+        }
+
         $this->linebuffer = $input;
         $this->linepos = $this->strlen($this->linebuffer);
 
-        return $this->redraw();
+        // only redraw if input should be echo'ed (i.e. is not hidden anyway)
+        if ($this->echo) {
+            $this->redraw();
+        }
+
+        return $this;
     }
 
     /**
@@ -362,12 +382,17 @@ public function moveCursorBy($n)
     public function moveCursorTo($n)
     {
         if ($n < 0 || $n === $this->linepos || $n > $this->strlen($this->linebuffer)) {
-            return;
+            return $this;
         }
 
         $this->linepos = $n;
 
-        return $this->redraw();
+        // only redraw if cursor is actually visible
+        if ($this->echo) {
+            $this->redraw();
+        }
+
+        return $this;
     }
 
     /**

tests/ReadlineTest.php

@@ -55,4 +55,73 @@ public function testRedrawingReadlineWritesToOutputOnce()
         $this->output->expects($this->once())->method('write')->with($this->equalTo("\r\033[K" . "> " . "test" . "\x08\x08"));
         $this->assertSame($this->readline, $this->readline->redraw());
     }
+
+    public function testSettingSameSettingsDoesNotNeedToRedraw()
+    {
+        $this->readline->setPrompt('> ');
+        $this->readline->setInput('test');
+        $this->readline->moveCursorBy(-2);
+
+        $this->output->expects($this->never())->method('write');
+
+        $this->assertSame($this->readline, $this->readline->setPrompt('> '));
+        $this->assertSame($this->readline, $this->readline->setInput('test'));
+        $this->assertSame($this->readline, $this->readline->moveCursorTo(2));
+    }
+
+    public function testSettingEchoOnWithInputDoesRedraw()
+    {
+        $this->readline->setEcho(false);
+        $this->readline->setPrompt('> ');
+        $this->readline->setInput('test');
+
+        $this->output->expects($this->once())->method('write')->with($this->equalTo("\r\033[K" . "> " . "test"));
+
+        $this->assertSame($this->readline, $this->readline->setEcho(true));
+    }
+
+    public function testSettingEchoOffWithInputDoesRedraw()
+    {
+        $this->readline->setEcho(true);
+        $this->readline->setPrompt('> ');
+        $this->readline->setInput('test');
+
+        $this->output->expects($this->once())->method('write')->with($this->equalTo("\r\033[K" . "> "));
+
+        $this->assertSame($this->readline, $this->readline->setEcho(false));
+    }
+
+    public function testSettingEchoWithoutInputDoesNotNeedToRedraw()
+    {
+        $this->output->expects($this->never())->method('write');
+
+        $this->assertSame($this->readline, $this->readline->setEcho(false));
+        $this->assertSame($this->readline, $this->readline->setEcho(true));
+    }
+
+    public function testSettingInputDoesRedraw()
+    {
+        $this->output->expects($this->once())->method('write');
+        $this->assertSame($this->readline, $this->readline->setInput('test'));
+    }
+
+    public function testSettingInputWithoutEchoDoesNotNeedToRedraw()
+    {
+        $this->readline->setEcho(false);
+
+        $this->output->expects($this->never())->method('write');
+
+        $this->assertSame($this->readline, $this->readline->setInput('test'));
+    }
+
+    public function testMovingCursorWithoutEchoDoesNotNeedToRedraw()
+    {
+        $this->readline->setEcho(false);
+        $this->readline->setInput('test');
+
+        $this->output->expects($this->never())->method('write');
+
+        $this->assertSame($this->readline, $this->readline->moveCursorTo(0));
+        $this->assertSame($this->readline, $this->readline->moveCursorBy(2));
+    }
 }