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

Author: delasteve

packages/angular-cli/commands/serve.ts

@@ -1,9 +1,14 @@
 const PortFinder = require('portfinder');
 const Command = require('../ember-cli/lib/models/command');
+import { CliConfig } from '../models/config';
 
-PortFinder.basePort = 49152;
+const config = CliConfig.fromProject() || CliConfig.fromGlobal();
 
-const defaultPort = process.env.PORT || 4200;
+const defaultLiveReloadPort = config.get('defaults.serve.liveReloadPort');
+const defaultPort = process.env.PORT || config.get('defaults.serve.port');
+const defaultHost = config.get('defaults.serve.host');
+
+PortFinder.basePort = defaultLiveReloadPort;
 
 export interface ServeTaskOptions {
   port?: number;
@@ -43,9 +48,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: 'watcher',              type: String,  default: 'events',      aliases: ['w'] },
@@ -66,7 +71,7 @@ const ServeCommand = Command.extend({
       name: 'live-reload-port',
       type: Number,
       aliases: ['lrp'],
-      description: '(Defaults to port number within [49152...65535])'
+      description: `Finds the first open port number within [${defaultLiveReloadPort}...65535]`
     },
     {
       name: 'live-reload-live-css',

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

@@ -243,6 +243,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; });
+}