bug(@angular/cli): Show module file update during dry run (#4836)

Fixes #4801

Author: Brocco

packages/@angular/cli/blueprints/component/index.ts

@@ -203,17 +203,19 @@ export default Blueprint.extend({
   },
 
   afterInstall: function (options: any) {
-    if (options.dryRun) {
-      return;
-    }
-
     const returns: Array<any> = [];
     const className = stringUtils.classify(`${options.entity.name}Component`);
     const fileName = stringUtils.dasherize(`${options.entity.name}.component`);
     const componentDir = path.relative(path.dirname(this.pathToModule), this.generatePath);
     const importPath = componentDir ? `./${componentDir}/${fileName}` : `./${fileName}`;
 
     if (!options.skipImport) {
+      if (options.dryRun) {
+        this._writeStatusToUI(chalk.yellow,
+          'update',
+          path.relative(this.project.root, this.pathToModule));
+        return;
+      }
       const preChange = fs.readFileSync(this.pathToModule, 'utf8');
 
       returns.push(

packages/@angular/cli/blueprints/directive/index.ts

@@ -132,10 +132,6 @@ export default Blueprint.extend({
   },
 
   afterInstall: function(options: any) {
-    if (options.dryRun) {
-      return;
-    }
-
     const returns: Array<any> = [];
     const className = stringUtils.classify(`${options.entity.name}Directive`);
     const fileName = stringUtils.dasherize(`${options.entity.name}.directive`);
@@ -145,6 +141,12 @@ export default Blueprint.extend({
     const importPath = relativeDir ? `./${relativeDir}/${fileName}` : `./${fileName}`;
 
     if (!options.skipImport) {
+      if (options.dryRun) {
+        this._writeStatusToUI(chalk.yellow,
+          'update',
+          path.relative(this.project.root, this.pathToModule));
+        return;
+      }
       returns.push(
         astUtils.addDeclarationToModule(this.pathToModule, className, importPath)
           .then((change: any) => change.apply(NodeHost))
@@ -155,9 +157,9 @@ export default Blueprint.extend({
             }
             return result;
           }));
-      this._writeStatusToUI(chalk.yellow,
-                            'update',
-                            path.relative(this.project.root, this.pathToModule));
+        this._writeStatusToUI(chalk.yellow,
+          'update',
+          path.relative(this.project.root, this.pathToModule));
     }
 
     return Promise.all(returns);

packages/@angular/cli/blueprints/pipe/index.ts

@@ -112,10 +112,6 @@ export default Blueprint.extend({
   },
 
   afterInstall: function(options: any) {
-    if (options.dryRun) {
-      return;
-    }
-
     const returns: Array<any> = [];
     const className = stringUtils.classify(`${options.entity.name}Pipe`);
     const fileName = stringUtils.dasherize(`${options.entity.name}.pipe`);
@@ -125,6 +121,12 @@ export default Blueprint.extend({
     const importPath = relativeDir ? `./${relativeDir}/${fileName}` : `./${fileName}`;
 
     if (!options.skipImport) {
+      if (options.dryRun) {
+        this._writeStatusToUI(chalk.yellow,
+          'update',
+          path.relative(this.project.root, this.pathToModule));
+        return;
+      }
       returns.push(
         astUtils.addDeclarationToModule(this.pathToModule, className, importPath)
           .then((change: any) => change.apply(NodeHost))
@@ -135,9 +137,9 @@ export default Blueprint.extend({
             }
             return result;
           }));
-      this._writeStatusToUI(chalk.yellow,
-                            'update',
-                            path.relative(this.project.root, this.pathToModule));
+        this._writeStatusToUI(chalk.yellow,
+          'update',
+          path.relative(this.project.root, this.pathToModule));
     }
 
     return Promise.all(returns);