Add deprecations in Console package.

src/Console/ConsoleIo.php

@@ -372,6 +372,7 @@ public function setOutputAs($mode)
      */
     public function outputAs($mode)
     {
+        deprecationWarning('ConsoleIo::outputAs() is deprecated. Use ConsoleIo::setOutputAs() instead.');
         $this->_out->setOutputAs($mode);
     }
 

src/Console/ConsoleOptionParser.php

@@ -310,6 +310,10 @@ public function getCommand()
      */
     public function command($text = null)
     {
+        deprecationWarning(
+            'ConsoleOptionParser::command() is deprecated. ' .
+            'Use ConsoleOptionParser::setCommand()/getCommand() instead.'
+        );
         if ($text !== null) {
             return $this->setCommand($text);
         }
@@ -354,6 +358,10 @@ public function getDescription()
      */
     public function description($text = null)
     {
+        deprecationWarning(
+            'ConsoleOptionParser::description() is deprecated. ' .
+            'Use ConsoleOptionParser::setDescription()/getDescription() instead.'
+        );
         if ($text !== null) {
             return $this->setDescription($text);
         }
@@ -400,6 +408,10 @@ public function getEpilog()
      */
     public function epilog($text = null)
     {
+        deprecationWarning(
+            'ConsoleOptionParser::epliog() is deprecated. ' .
+            'Use ConsoleOptionParser::setEpilog()/getEpilog() instead.'
+        );
         if ($text !== null) {
             return $this->setEpilog($text);
         }
@@ -785,6 +797,10 @@ public function help($subcommand = null, $format = 'text', $width = 72)
      */
     public function setHelpAlias($alias)
     {
+        deprecationWarning(
+            'ConsoleOptionParser::setHelpAlias() is deprecated. ' .
+            'Use ConsoleOptionParser::setRootName() instead.'
+        );
         $this->rootName = $alias;
     }
 

src/Console/ConsoleOutput.php

@@ -339,6 +339,10 @@ public function setOutputAs($type)
      */
     public function outputAs($type = null)
     {
+        deprecationWarning(
+            'ConsoleOutput::outputAs() is deprecated. ' .
+            'Use ConsoleOutput::setOutputAs()/getOutputAs() instead.'
+        );
         if ($type === null) {
             return $this->_outputAs;
         }

src/Console/Shell.php

@@ -243,6 +243,10 @@ public function setIo(ConsoleIo $io)
      */
     public function io(ConsoleIo $io = null)
     {
+        deprecationWarning(
+            'Shell::io() is deprecated. ' .
+            'Use Shell::setIo()/getIo() instead.'
+        );
         if ($io !== null) {
             $this->_io = $io;
         }
@@ -783,6 +787,10 @@ public function success($message = null, $newlines = 1, $level = Shell::NORMAL)
      */
     protected function wrapMessageWithType($messageType, $message)
     {
+        deprecationWarning(
+            'Shell::wrapMessageWithType() is deprecated. ' .
+            'Use output methods on ConsoleIo instead.'
+        );
         if (is_array($message)) {
             foreach ($message as $k => $v) {
                 $message[$k] = "<$messageType>" . $v . "</$messageType>";
@@ -849,6 +857,7 @@ public function abort($message, $exitCode = self::CODE_ERROR)
      */
     public function error($title, $message = null, $exitCode = self::CODE_ERROR)
     {
+        deprecationWarning('Shell::error() is deprecated. `Use Shell::abort() instead.');
         $this->_io->err(sprintf('<error>Error:</error> %s', $title));
 
         if (!empty($message)) {

tests/TestCase/Console/ConsoleOptionParserTest.php

@@ -26,6 +26,23 @@
 class ConsoleOptionParserTest extends TestCase
 {
 
+    /**
+     * test setting the console description
+     *
+     * @group deprecated
+     * @return void
+     */
+    public function testDescriptionDeprecated()
+    {
+        $this->deprecated(function() {
+            $parser = new ConsoleOptionParser('test', false);
+            $result = $parser->description('A test');
+
+            $this->assertEquals($parser, $result, 'Setting description is not chainable');
+            $this->assertEquals('A test', $parser->description(), 'getting value is wrong.');
+        });
+    }
+
     /**
      * test setting the console description
      *
@@ -34,13 +51,30 @@ class ConsoleOptionParserTest extends TestCase
     public function testDescription()
     {
         $parser = new ConsoleOptionParser('test', false);
-        $result = $parser->description('A test');
+        $result = $parser->setDescription('A test');
 
         $this->assertEquals($parser, $result, 'Setting description is not chainable');
-        $this->assertEquals('A test', $parser->description(), 'getting value is wrong.');
+        $this->assertEquals('A test', $parser->getDescription(), 'getting value is wrong.');
+
+        $result = $parser->setDescription(['A test', 'something']);
+        $this->assertEquals("A test\nsomething", $parser->getDescription(), 'getting value is wrong.');
+    }
+
+    /**
+     * test setting the console description
+     *
+     * @group deprecated
+     * @return void
+     */
+    public function testEplilogDeprecated()
+    {
+        $this->deprecated(function() {
+            $parser = new ConsoleOptionParser('test', false);
+            $result = $parser->epilog('A test');
 
-        $result = $parser->description(['A test', 'something']);
-        $this->assertEquals("A test\nsomething", $parser->description(), 'getting value is wrong.');
+            $this->assertEquals($parser, $result, 'Setting epilog is not chainable');
+            $this->assertEquals('A test', $parser->epilog(), 'getting value is wrong.');
+        });
     }
 
     /**
@@ -51,13 +85,13 @@ public function testDescription()
     public function testEpilog()
     {
         $parser = new ConsoleOptionParser('test', false);
-        $result = $parser->epilog('A test');
+        $result = $parser->setEpilog('A test');
 
         $this->assertEquals($parser, $result, 'Setting epilog is not chainable');
-        $this->assertEquals('A test', $parser->epilog(), 'getting value is wrong.');
+        $this->assertEquals('A test', $parser->getEpilog(), 'getting value is wrong.');
 
-        $result = $parser->epilog(['A test', 'something']);
-        $this->assertEquals("A test\nsomething", $parser->epilog(), 'getting value is wrong.');
+        $result = $parser->setEpilog(['A test', 'something']);
+        $this->assertEquals("A test\nsomething", $parser->getEpilog(), 'getting value is wrong.');
     }
 
     /**
@@ -789,7 +823,7 @@ public function testHelpSubcommandHelpArray()
     public function testHelpWithRootName()
     {
         $parser = new ConsoleOptionParser('sample', false);
-        $parser->description('A command!')
+        $parser->setDescription('A command!')
             ->setRootName('tool')
             ->addOption('test', ['help' => 'A test option.']);
 
@@ -874,8 +908,8 @@ public function testBuildFromArray()
         ];
         $parser = ConsoleOptionParser::buildFromArray($spec);
 
-        $this->assertEquals($spec['description'], $parser->description());
-        $this->assertEquals($spec['epilog'], $parser->epilog());
+        $this->assertEquals($spec['description'], $parser->getDescription());
+        $this->assertEquals($spec['epilog'], $parser->getEpilog());
 
         $options = $parser->options();
         $this->assertTrue(isset($options['name']));
@@ -897,7 +931,7 @@ public function testCreateFactory()
     {
         $parser = ConsoleOptionParser::create('factory', false);
         $this->assertInstanceOf('Cake\Console\ConsoleOptionParser', $parser);
-        $this->assertEquals('factory', $parser->command());
+        $this->assertEquals('factory', $parser->getCommand());
     }
 
     /**
@@ -908,7 +942,7 @@ public function testCreateFactory()
     public function testCommandInflection()
     {
         $parser = new ConsoleOptionParser('CommandLine');
-        $this->assertEquals('command_line', $parser->command());
+        $this->assertEquals('command_line', $parser->getCommand());
     }
 
     /**

tests/TestCase/Console/ConsoleOutputTest.php

@@ -36,7 +36,7 @@ public function setUp()
         $this->output = $this->getMockBuilder('Cake\Console\ConsoleOutput')
             ->setMethods(['_write'])
             ->getMock();
-        $this->output->outputAs(ConsoleOutput::COLOR);
+        $this->output->setOutputAs(ConsoleOutput::COLOR);
     }
 
     /**
@@ -221,14 +221,32 @@ public function testFormattingMultipleSameTags()
         $this->output->write('<error>Bad</error> <error>Warning</error> Regular', false);
     }
 
+    /**
+     * test deprecated outputAs
+     *
+     * @group deprecated
+     * @return void
+     */
+    public function testOutputAsPlain()
+    {
+        $this->deprecated(function() {
+            $this->output->outputAs(ConsoleOutput::PLAIN);
+            $this->assertSame(ConsoleOutput::PLAIN, $this->output->outputAs());
+            $this->output->expects($this->once())->method('_write')
+                ->with('Bad Regular');
+
+            $this->output->write('<error>Bad</error> Regular', false);
+        });
+    }
+
     /**
      * test raw output not getting tags replaced.
      *
      * @return void
      */
-    public function testOutputAsRaw()
+    public function testSetOutputAsRaw()
     {
-        $this->output->outputAs(ConsoleOutput::RAW);
+        $this->output->setOutputAs(ConsoleOutput::RAW);
         $this->output->expects($this->once())->method('_write')
             ->with('<error>Bad</error> Regular');
 
@@ -240,9 +258,9 @@ public function testOutputAsRaw()
      *
      * @return void
      */
-    public function testOutputAsPlain()
+    public function testSetOutputAsPlain()
     {
-        $this->output->outputAs(ConsoleOutput::PLAIN);
+        $this->output->setOutputAs(ConsoleOutput::PLAIN);
         $this->output->expects($this->once())->method('_write')
             ->with('Bad Regular');
 
@@ -254,7 +272,7 @@ public function testOutputAsPlain()
      *
      * @return void
      */
-    public function testSetOutputAsPlain()
+    public function testSetSetOutputAsPlain()
     {
         $this->output->setOutputAs(ConsoleOutput::PLAIN);
         $this->output->expects($this->once())->method('_write')
@@ -279,9 +297,9 @@ public function testSetOutputWrongType()
      *
      * @return void
      */
-    public function testOutputAsPlainSelectiveTagRemoval()
+    public function testSetOutputAsPlainSelectiveTagRemoval()
     {
-        $this->output->outputAs(ConsoleOutput::PLAIN);
+        $this->output->setOutputAs(ConsoleOutput::PLAIN);
         $this->output->expects($this->once())->method('_write')
             ->with('Bad Regular <b>Left</b> <i>behind</i> <name>');
 

tests/TestCase/Console/HelpFormatterTest.php

@@ -34,7 +34,7 @@ class HelpFormatterTest extends TestCase
     public function testWidthFormatting()
     {
         $parser = new ConsoleOptionParser('test', false);
-        $parser->description('This is fifteen This is fifteen This is fifteen')
+        $parser->setDescription('This is fifteen This is fifteen This is fifteen')
             ->addOption('four', ['help' => 'this is help text this is help text'])
             ->addArgument('four', ['help' => 'this is help text this is help text'])
             ->addSubcommand('four', ['help' => 'this is help text this is help text']);
@@ -115,8 +115,8 @@ public function testHelpWithChoices()
     public function testHelpDescriptionAndEpilog()
     {
         $parser = new ConsoleOptionParser('mycommand', false);
-        $parser->description('Description text')
-            ->epilog('epilog text')
+        $parser->setDescription('Description text')
+            ->setEpilog('epilog text')
             ->addOption('test', ['help' => 'A test option.'])
             ->addArgument('model', ['help' => 'The model to make.', 'required' => true]);
 
@@ -384,8 +384,8 @@ public function testXmlHelpWithChoices()
     public function testXmlHelpDescriptionAndEpilog()
     {
         $parser = new ConsoleOptionParser('mycommand', false);
-        $parser->description('Description text')
-            ->epilog('epilog text')
+        $parser->setDescription('Description text')
+            ->setEpilog('epilog text')
             ->addOption('test', ['help' => 'A test option.'])
             ->addArgument('model', ['help' => 'The model to make.', 'required' => true]);