Address PR comments

Author: persimmons

lib/services.js

@@ -50,7 +50,6 @@ module.exports = function generateServices(app, options) {
       apiUrl: arguments[2],
     };
   }
-  var oldOptions = options;
 
   options = extend({
     ngModuleName: 'lbServices',

lib/services.template.ejs

@@ -389,6 +389,7 @@ if (typeof module !== 'undefined' && typeof exports !== 'undefined' &&
         },
       };
     }])
+
   /**
    * @ngdoc object
    * @name <%-: moduleName %>.LoopBackResourceProvider

test.e2e/given.js

@@ -58,7 +58,7 @@ define(['angular', 'angularMocks', 'angularResource'], function(angular) {
    * ```
    */
   given.servicesForLoopBackApp = function(options, cb) {
-    if (! options.name) {
+    if (!options.name) {
       options.name = generateUniqueServiceName(options.name);
     }
 

test.e2e/test-server.js

@@ -151,15 +151,16 @@ masterApp.listen(port, function() {
 
 function generateService(generator, lbApp, apiUrl, opts) {
   var servicesScript;
-  if (typeof opts.includeSchema !== 'undefined' ||
-      typeof opts.includeCommonModules !== 'undefined' ||
-      typeof opts.namespaceModels !== 'undefined' ||
-      typeof opts.namespaceDelimiter !== 'undefined') {
+  if (opts.includeSchema !== undefined ||
+      opts.includeCommonModules !== undefined ||
+      opts.namespaceModels !== undefined ||
+      opts.namespaceDelimiter !== undefined) {
     // the new options-based API
 
     // build options object for new options-based API
     var generatorOptions = opts;
     generatorOptions.ngModuleName = opts.name;
+    generatorOptions.apiUrl = apiUrl;
 
     servicesScript = generator.services(lbApp, generatorOptions);
   } else {