feat(serve): Persist serve options in angular-cli.json

Author: delasteve

packages/angular-cli/commands/serve.ts

@@ -1,12 +1,15 @@
 import { BuildOptions } from '../models/webpack-config';
 import { BaseBuildCommandOptions } from './build';
-
+import { CliConfig } from '../models/config';
 const PortFinder = require('portfinder');
 const Command = require('../ember-cli/lib/models/command');
+const config = CliConfig.fromProject() || CliConfig.fromGlobal();
 
 PortFinder.basePort = 49152;
 
-const defaultPort = process.env.PORT || 4200;
+const defaultPort = process.env.PORT || config.get('defaults.serve.port');
+const defaultHost = config.get('defaults.serve.host');
+
 
 export interface ServeTaskOptions extends BuildOptions {
   port?: number;
@@ -34,9 +37,9 @@ const ServeCommand = Command.extend({
     {
       name: 'host',
       type: String,
-      default: 'localhost',
+      default: defaultHost,
       aliases: ['H'],
-      description: 'Listens only on localhost by default'
+      description: `Listens only on ${defaultHost} by default`
     },
     { name: 'proxy-config',         type: 'Path',                          aliases: ['pc'] },
     { name: 'live-reload',          type: Boolean, default: true,          aliases: ['lr'] },

packages/angular-cli/lib/config/schema.json

@@ -247,6 +247,23 @@
               "default": true
             }
           }
+        },
+        "serve": {
+          "type": "object",
+          "properties": {
+            "port": {
+              "type": "number",
+              "default": 4200
+            },
+            "liveReloadPort": {
+              "type": "number",
+              "default": 49152
+            },
+            "host": {
+              "type": "string",
+              "default": "localhost"
+            }
+          }
         }
       },
       "additionalProperties": false

tests/e2e/tests/misc/default-port.ts

@@ -0,0 +1,20 @@
+import { request } from '../../utils/http';
+import { killAllProcesses } from '../../utils/process';
+import { ngServe } from '../../utils/project';
+import { updateJsonFile } from '../../utils/project';
+
+export default function() {
+  return Promise.resolve()
+    .then(() => updateJsonFile('angular-cli.json', configJson => {
+      const app = configJson.defaults;
+      app.serve = { port: 4201 };
+    }))
+    .then(() => ngServe())
+    .then(() => request('http://localhost:4201/'))
+    .then(body => {
+      if (!body.match(/<app-root>Loading...<\/app-root>/)) {
+        throw new Error('Response does not match expected value.');
+      }
+    })
+    .then(() => killAllProcesses(), (err) => { killAllProcesses(); throw err; });
+}