implement code & test for 'remove fields' from cli arguments

Signed-off-by: Joshua Paling <joshua.paling@gmail.com>

Author: joshuapaling

Console/Command/MigrationShell.php

@@ -434,6 +434,7 @@ protected function _generateFromCliArgs(&$migration, &$migrationName, &$comparis
 		}
 
 		$cli = $this->_parseCommandLine($migrationName);
+		// debug($cli); die;
 
 		$action = $cli['action'];
 		$table = $cli['table'];
@@ -445,6 +446,12 @@ protected function _generateFromCliArgs(&$migration, &$migrationName, &$comparis
 			$migration['down']['drop_table'] = array($table);
 		} elseif ($action == 'drop_table') {
 			$migration['up']['drop_table'] = array($table);
+		} elseif ($action == 'drop_field') {
+			$fieldsToDrop = array();
+			foreach ($fields as $name => $value) {
+				$fieldsToDrop[] = $name;
+			}
+			$migration['up']['drop_field'][$table] = $fieldsToDrop;
 		}
 
 /*		if (in_array($action, array('create_table', 'alter_table', 'create_field'))) {

Test/Case/Console/Command/MigrationShellTest.php

@@ -813,6 +813,31 @@ public function testGenerateFromCliParamsDropTable() {
 		$this->assertEquals($expected, $result);
 	}
 
+/**
+ * testGenerateFromCliParamsRemoveFields method
+ * test the case of using a command such as:
+ * app/Console/cake Migrations.migration generate remove_name_and_desc_from_products name description
+ *
+ * @return void
+ */
+	public function testGenerateFromCliParamsRemoveFields() {
+		$this->Shell->expects($this->at(0))->method('in')->will($this->returnValue('n'));
+		$this->assertEmpty(glob(TMP . 'tests' . DS . '*remove_name_and_desc_from_products.php'));
+
+		$this->Shell->args = array('remove_name_and_desc_from_products', 'name', 'description');
+		$this->Shell->params['force'] = true;
+		$this->Shell->generate();
+		$files = glob(TMP . 'tests' . DS . '*remove_name_and_desc_from_products.php');
+		$this->assertNotEmpty($files);
+		$result = $this->_getMigrationVariable(current($files));
+		foreach ($files as $f) {
+			unlink($f);
+		}
+
+		$expected = file_get_contents(CakePlugin::path('Migrations') . '/Test/Fixture/test_migration_remove_fields_from_cli.txt');
+		$this->assertEquals($expected, $result);
+	}
+
 /**
  * TestGenerateDump method
  *

Test/Fixture/test_migration_remove_fields_from_cli.txt

@@ -0,0 +1,9 @@
+	public $migration = array(
+		'up' => array(
+			'drop_field' => array(
+				'products' => array('name', 'description',),
+			),
+		),
+		'down' => array(
+		),
+	);