From da3e4a7356539f85da4df5d1256b91fc5015a6a4 Mon Sep 17 00:00:00 2001 From: Hans Larsen Date: Mon, 19 Dec 2016 13:26:10 -0800 Subject: [PATCH] chore(deps): remove any mentions of bower. Ref #3199. --- .eslintignore | 1 - .../blueprints/ng2/files/gitignore | 1 - packages/angular-cli/commands/init.ts | 18 ----- packages/angular-cli/commands/new.ts | 1 - .../ember-cli/lib/models/blueprint.js | 78 ------------------- .../lib/models/installation-checker.js | 20 +---- .../ember-cli/lib/models/project.js | 57 -------------- packages/angular-cli/utilities/completion.sh | 4 +- tests/acceptance/destroy.spec.js | 2 +- tests/acceptance/generate-class.spec.js | 2 +- tests/acceptance/generate-component.spec.js | 2 +- tests/acceptance/generate-directive.spec.js | 2 +- tests/acceptance/generate-module.spec.js | 2 +- tests/acceptance/generate-pipe.spec.js | 2 +- tests/acceptance/generate-route.spec.js | 2 +- tests/acceptance/generate-service.spec.js | 2 +- tests/acceptance/github-pages-deploy.spec.js | 2 +- tests/acceptance/init.spec.js | 26 +++---- tests/acceptance/new.spec.ts | 14 ++-- 19 files changed, 32 insertions(+), 206 deletions(-) diff --git a/.eslintignore b/.eslintignore index f07735d14909..974911dfbb03 100644 --- a/.eslintignore +++ b/.eslintignore @@ -1,4 +1,3 @@ -# /node_modules and /bower_components ignored by default dist/ .git/ tmp/ diff --git a/packages/angular-cli/blueprints/ng2/files/gitignore b/packages/angular-cli/blueprints/ng2/files/gitignore index 1eb14e472190..fe6f8a76b049 100755 --- a/packages/angular-cli/blueprints/ng2/files/gitignore +++ b/packages/angular-cli/blueprints/ng2/files/gitignore @@ -6,7 +6,6 @@ # dependencies /node_modules -/bower_components # IDEs and editors /.idea diff --git a/packages/angular-cli/commands/init.ts b/packages/angular-cli/commands/init.ts index cf7bd381b1ad..bd4978f759ee 100644 --- a/packages/angular-cli/commands/init.ts +++ b/packages/angular-cli/commands/init.ts @@ -20,7 +20,6 @@ const InitCommand: any = Command.extend({ { name: 'verbose', type: Boolean, default: false, aliases: ['v'] }, { name: 'link-cli', type: Boolean, default: false, aliases: ['lc'] }, { name: 'skip-npm', type: Boolean, default: false, aliases: ['sn'] }, - { name: 'skip-bower', type: Boolean, default: true, aliases: ['sb'] }, { name: 'name', type: String, default: '', aliases: ['n'] }, { name: 'source-dir', type: String, default: 'src', aliases: ['sd'] }, { name: 'style', type: String, default: 'css' }, @@ -36,7 +35,6 @@ const InitCommand: any = Command.extend({ run: function (commandOptions: any, rawArgs: string[]) { if (commandOptions.dryRun) { commandOptions.skipNpm = true; - commandOptions.skipBower = true; } const installBlueprint = new this.tasks.InstallBlueprint({ @@ -73,15 +71,6 @@ const InitCommand: any = Command.extend({ }); } - let bowerInstall: any; - if (!commandOptions.skipBower) { - bowerInstall = new this.tasks.BowerInstall({ - ui: this.ui, - analytics: this.analytics, - project: this.project - }); - } - const project = this.project; const packageName = commandOptions.name !== '.' && commandOptions.name || project.name(); @@ -138,13 +127,6 @@ const InitCommand: any = Command.extend({ if (commandOptions.linkCli) { return linkCli.run(); } - }) - .then(function () { - if (!commandOptions.skipBower) { - return bowerInstall.run({ - verbose: commandOptions.verbose - }); - } }); } }); diff --git a/packages/angular-cli/commands/new.ts b/packages/angular-cli/commands/new.ts index 4a9dcdf9b9af..c2a9bc70decb 100644 --- a/packages/angular-cli/commands/new.ts +++ b/packages/angular-cli/commands/new.ts @@ -18,7 +18,6 @@ const NewCommand = Command.extend({ { name: 'verbose', type: Boolean, default: false, aliases: ['v'] }, { name: 'link-cli', type: Boolean, default: false, aliases: ['lc'] }, { name: 'skip-npm', type: Boolean, default: false, aliases: ['sn'] }, - { name: 'skip-bower', type: Boolean, default: true, aliases: ['sb'] }, { name: 'skip-git', type: Boolean, default: false, aliases: ['sg'] }, { name: 'directory', type: String, aliases: ['dir'] }, { name: 'source-dir', type: String, default: 'src', aliases: ['sd'] }, diff --git a/packages/angular-cli/ember-cli/lib/models/blueprint.js b/packages/angular-cli/ember-cli/lib/models/blueprint.js index f167fdbdfbcb..04925d9b778f 100644 --- a/packages/angular-cli/ember-cli/lib/models/blueprint.js +++ b/packages/angular-cli/ember-cli/lib/models/blueprint.js @@ -986,84 +986,6 @@ Blueprint.prototype.removePackagesFromProject = function(packages) { }); }; -/** - Used to add a package to the projects `bower.json`. - - Generally, this would be done from the `afterInstall` hook, to - ensure that a package that is required by a given blueprint is - available. - - `localPackageName` and `target` may be thought of as equivalent - to the key-value pairs in the `dependency` or `devDepencency` - objects contained within a bower.json file. - - Examples: - - addBowerPackageToProject('jquery', '~1.11.1'); - addBowerPackageToProject('old_jquery', 'jquery#~1.9.1'); - addBowerPackageToProject('bootstrap-3', 'http://twitter.github.io/bootstrap/assets/bootstrap'); - - @method addBowerPackageToProject - @param {String} localPackageName - @param {String} target - @param {Object} installOptions - @return {Promise} -*/ -Blueprint.prototype.addBowerPackageToProject = function(localPackageName, target, installOptions) { - // var lpn = localPackageName; - // var tar = target; - // if (localPackageName.indexOf('#') >= 0) { - // if (arguments.length === 1) { - // var parts = localPackageName.split('#'); - // lpn = parts[0]; - // tar = parts[1]; - // this.ui.writeDeprecateLine('passing ' + localPackageName + - // ' directly to `addBowerPackageToProject` will soon be unsupported. \n' + - // 'You may want to replace this with ' + - // '`addBowerPackageToProject(\'' + lpn + '\', \'' + tar + '\')`'); - // } else { - // this.ui.writeDeprecateLine('passing ' + localPackageName + - // ' directly to `addBowerPackageToProject` will soon be unsupported'); - // } - // } - // var packageObject = bowEpParser.json2decomposed(lpn, tar); - // return this.addBowerPackagesToProject([packageObject], installOptions); - return Promise.resolve(); -}; - -/** - Used to add an array of packages to the projects `bower.json`. - - Generally, this would be done from the `afterInstall` hook, to - ensure that a package that is required by a given blueprint is - available. - - Expects each array item to be an object with a `name`. Each object - may optionally have a `target` to specify a specific version. - - @method addBowerPackagesToProject - @param {Array} packages - @param {Object} installOptions - @return {Promise} -*/ -Blueprint.prototype.addBowerPackagesToProject = function(packages, installOptions) { - var task = this.taskFor('bower-install'); - var installText = (packages.length > 1) ? 'install bower packages' : 'install bower package'; - var packageNames = []; - var packageNamesAndVersions = packages.map(function (pkg) { - pkg.source = pkg.source || pkg.name; - packageNames.push(pkg.name); - return pkg; - }).map(bowEpParser.compose); - - this._writeStatusToUI(chalk.green, installText, packageNames.join(', ')); - - return task.run({ - verbose: true, - packages: packageNamesAndVersions, - installOptions: installOptions - }); -}; /** Used to retrieve a task with the given name. Passes the new task diff --git a/packages/angular-cli/ember-cli/lib/models/installation-checker.js b/packages/angular-cli/ember-cli/lib/models/installation-checker.js index b9a0ed967a37..2037aca45b72 100644 --- a/packages/angular-cli/ember-cli/lib/models/installation-checker.js +++ b/packages/angular-cli/ember-cli/lib/models/installation-checker.js @@ -13,12 +13,12 @@ function InstallationChecker(options) { } /** -* Check if npm and bower installation directories are present, +* Check if npm directories are present, * and raise an error message with instructions on how to proceed. * * If some of these package managers aren't being used in the project * we just ignore them. Their usage is considered by checking the -* presence of your manifest files: package.json for npm and bower.json for bower. +* presence of your manifest files: package.json for npm. */ InstallationChecker.prototype.checkInstallations = function() { var commands = []; @@ -27,10 +27,6 @@ InstallationChecker.prototype.checkInstallations = function() { debug('npm dependencies not installed'); commands.push('`npm install`'); } - if (this.usingBower() && this.bowerDependenciesNotPresent()) { - debug('bower dependencies not installed'); - commands.push('`bower install`'); - } if (commands.length) { var commandText = commands.join(' and '); throw new SilentError('No dependencies installed. Run ' + commandText + ' to install missing dependencies.'); @@ -50,18 +46,6 @@ function readJSON(path) { } } -InstallationChecker.prototype.hasBowerDeps = function() { - return hasDependencies(readJSON(path.join(this.project.root, 'bower.json'))); -}; - -InstallationChecker.prototype.usingBower = function() { - return existsSync(path.join(this.project.root, 'bower.json')) && this.hasBowerDeps(); -}; - -InstallationChecker.prototype.bowerDependenciesNotPresent = function() { - return !existsSync(this.project.bowerDirectory); -}; - InstallationChecker.prototype.hasNpmDeps = function() { return hasDependencies(readJSON(path.join(this.project.root, 'package.json'))); }; diff --git a/packages/angular-cli/ember-cli/lib/models/project.js b/packages/angular-cli/ember-cli/lib/models/project.js index d0c4fa0232bb..fda4ed524a51 100644 --- a/packages/angular-cli/ember-cli/lib/models/project.js +++ b/packages/angular-cli/ember-cli/lib/models/project.js @@ -41,7 +41,6 @@ function Project(root, pkg, ui, cli) { this.addonPackages = {}; this.addons = []; this.liveReloadFilterPatterns = []; - this.setupBowerDirectory(); this.setupNodeModulesPath(); this.addonDiscovery = new AddonDiscovery(this.ui); this.addonsFactory = new AddonsFactory(this, this); @@ -52,46 +51,6 @@ function Project(root, pkg, ui, cli) { }; } -/** - Set when the `Watcher.detectWatchman` helper method finishes running, - so that other areas of the system can be aware that watchman is being used. - - For example, this information is used in the broccoli build pipeline to know - if we can watch additional directories (like bower_components) "cheaply". - - Contains `enabled` and `version`. - - @private - @property _watchmanInfo - @returns {Object} - @default false -*/ - -/** - Sets the name of the bower directory for this project - - @private - @method setupBowerDirectory - */ -Project.prototype.setupBowerDirectory = function() { - var bowerrcPath = path.join(this.root, '.bowerrc'); - - debug('bowerrc path: %s', bowerrcPath); - - if (existsSync(bowerrcPath)) { - var bowerrcContent = fs.readFileSync(bowerrcPath); - try { - this.bowerDirectory = JSON.parse(bowerrcContent).directory; - } catch (exception) { - debug('failed to parse bowerc: %s', exception); - this.bowerDirectory = null; - } - } - - this.bowerDirectory = this.bowerDirectory || 'bower_components'; - debug('bowerDirectory: %s', this.bowerDirectory); -}; - Project.prototype.hasDependencies = function() { return !!this.nodeModulesPath; }; @@ -306,22 +265,6 @@ Project.prototype.dependencies = function(pkg, excludeDevDeps) { return assign({}, devDependencies, pkg['dependencies']); }; -/** - Returns the bower dependencies for this project. - - @private - @method bowerDependencies - @param {String} bower Path to bower.json - @return {Object} Bower dependencies - */ -Project.prototype.bowerDependencies = function(bower) { - if (!bower) { - var bowerPath = path.join(this.root, 'bower.json'); - bower = (existsSync(bowerPath)) ? require(bowerPath) : {}; - } - return assign({}, bower['devDependencies'], bower['dependencies']); -}; - /** Provides the list of paths to consult for addons that may be provided internally to this project. Used for middleware addons with built-in support. diff --git a/packages/angular-cli/utilities/completion.sh b/packages/angular-cli/utilities/completion.sh index 3b867977d059..eb4b562b8585 100644 --- a/packages/angular-cli/utilities/completion.sh +++ b/packages/angular-cli/utilities/completion.sh @@ -12,8 +12,8 @@ build_opts='--aot --base-href --environment --i18n-file --i18n-format --locale - generate_opts='class component directive enum module pipe route service c cl d e m p r s --help' github_pages_deploy_opts='--base-href --environment --gh-token --gh-username --message --skip-build --target --user-page -bh -e -t' help_opts='--json --verbose -v' -init_opts='--dry-run inline-style inline-template --link-cli --mobile --name --prefix --routing --skip-bower --skip-npm --source-dir --style --verbose -d -is -it -lc -n -p -sb -sd -sn -v' -new_opts='--directory --dry-run inline-style inline-template --link-cli --mobile --prefix --routing --skip-bower --skip-git --skip-npm --source-dir --style --verbose -d -dir -is -it -lc -p -sb -sd -sg -sn -v' +init_opts='--dry-run inline-style inline-template --link-cli --mobile --name --prefix --routing --skip-npm --source-dir --style --verbose -d -is -it -lc -n -p -sb -sd -sn -v' +new_opts='--directory --dry-run inline-style inline-template --link-cli --mobile --prefix --routing --skip-git --skip-npm --source-dir --style --verbose -d -dir -is -it -lc -p -sb -sd -sg -sn -v' serve_opts='--aot --environment --hmr --host --i18n-file --i18n-format --live-reload --live-reload-base-url --live-reload-host --live-reload-live-css --live-reload-port --locale --open --port --proxy-config --sourcemap --ssl --ssl-cert --ssl-key --target --watcher -H -e -lr -lrbu -lrh -lrp -o -p -pc -sm -t -w' set_opts='--global -g' test_opts='--browsers --build --code-coverage --colors --lint --log-level --port --reporters --single-run --sourcemap --watch -cc -l -sm -sr -w' diff --git a/tests/acceptance/destroy.spec.js b/tests/acceptance/destroy.spec.js index fac5327d4533..eabb2ef77a19 100644 --- a/tests/acceptance/destroy.spec.js +++ b/tests/acceptance/destroy.spec.js @@ -11,7 +11,7 @@ describe('Acceptance: ng destroy', function () { return tmp.setup('./tmp').then(function () { process.chdir('./tmp'); }).then(function () { - return ng(['new', 'foo', '--skip-npm', '--skip-bower']); + return ng(['new', 'foo', '--skip-npm']); }); }); diff --git a/tests/acceptance/generate-class.spec.js b/tests/acceptance/generate-class.spec.js index 0c173d90127d..a219cf23981e 100644 --- a/tests/acceptance/generate-class.spec.js +++ b/tests/acceptance/generate-class.spec.js @@ -16,7 +16,7 @@ describe('Acceptance: ng generate class', function () { return tmp.setup('./tmp').then(function () { process.chdir('./tmp'); }).then(function () { - return ng(['new', 'foo', '--skip-npm', '--skip-bower']); + return ng(['new', 'foo', '--skip-npm']); }); }); diff --git a/tests/acceptance/generate-component.spec.js b/tests/acceptance/generate-component.spec.js index 07f04348bc38..40d7d8f2cf26 100644 --- a/tests/acceptance/generate-component.spec.js +++ b/tests/acceptance/generate-component.spec.js @@ -20,7 +20,7 @@ describe('Acceptance: ng generate component', function () { return tmp.setup('./tmp').then(function () { process.chdir('./tmp'); }).then(function () { - return ng(['new', 'foo', '--skip-npm', '--skip-bower']); + return ng(['new', 'foo', '--skip-npm']); }); }); diff --git a/tests/acceptance/generate-directive.spec.js b/tests/acceptance/generate-directive.spec.js index efeca10e03be..4a4ef87570bd 100644 --- a/tests/acceptance/generate-directive.spec.js +++ b/tests/acceptance/generate-directive.spec.js @@ -19,7 +19,7 @@ describe('Acceptance: ng generate directive', function () { return tmp.setup('./tmp').then(function () { process.chdir('./tmp'); }).then(function () { - return ng(['new', 'foo', '--skip-npm', '--skip-bower']); + return ng(['new', 'foo', '--skip-npm']); }); }); diff --git a/tests/acceptance/generate-module.spec.js b/tests/acceptance/generate-module.spec.js index a4e38274da4c..26119548215e 100644 --- a/tests/acceptance/generate-module.spec.js +++ b/tests/acceptance/generate-module.spec.js @@ -15,7 +15,7 @@ describe('Acceptance: ng generate module', function () { return tmp.setup('./tmp').then(function () { process.chdir('./tmp'); }).then(function () { - return ng(['new', 'foo', '--skip-npm', '--skip-bower']); + return ng(['new', 'foo', '--skip-npm']); }); }); diff --git a/tests/acceptance/generate-pipe.spec.js b/tests/acceptance/generate-pipe.spec.js index 767df23995bf..ae8c578afce5 100644 --- a/tests/acceptance/generate-pipe.spec.js +++ b/tests/acceptance/generate-pipe.spec.js @@ -20,7 +20,7 @@ describe('Acceptance: ng generate pipe', function () { return tmp.setup('./tmp').then(function () { process.chdir('./tmp'); }).then(function () { - return ng(['new', 'foo', '--skip-npm', '--skip-bower']); + return ng(['new', 'foo', '--skip-npm']); }); }); diff --git a/tests/acceptance/generate-route.spec.js b/tests/acceptance/generate-route.spec.js index afe616bf5d26..99469635b42e 100644 --- a/tests/acceptance/generate-route.spec.js +++ b/tests/acceptance/generate-route.spec.js @@ -21,7 +21,7 @@ xdescribe('Acceptance: ng generate route', function () { return tmp.setup('./tmp').then(function () { process.chdir('./tmp'); }).then(function () { - return ng(['new', 'foo', '--skip-npm', '--skip-bower']); + return ng(['new', 'foo', '--skip-npm']); }); }); diff --git a/tests/acceptance/generate-service.spec.js b/tests/acceptance/generate-service.spec.js index 7d407c575554..8a011bfc28f9 100644 --- a/tests/acceptance/generate-service.spec.js +++ b/tests/acceptance/generate-service.spec.js @@ -19,7 +19,7 @@ describe('Acceptance: ng generate service', function () { return tmp.setup('./tmp').then(function () { process.chdir('./tmp'); }).then(function () { - return ng(['new', 'foo', '--skip-npm', '--skip-bower']); + return ng(['new', 'foo', '--skip-npm']); }); }); diff --git a/tests/acceptance/github-pages-deploy.spec.js b/tests/acceptance/github-pages-deploy.spec.js index cd44a3cc078a..3f8d0d851de6 100644 --- a/tests/acceptance/github-pages-deploy.spec.js +++ b/tests/acceptance/github-pages-deploy.spec.js @@ -37,7 +37,7 @@ describe('Acceptance: ng github-pages:deploy', function() { this.timeout(10000); return tmp.setup('./tmp') .then(() => process.chdir('./tmp')) - .then(() => ng(['new', project, '--skip-npm', '--skip-bower'])) + .then(() => ng(['new', project, '--skip-npm'])) .then(() => setupDist()) .finally(() => execStub = new ExecStub()); }); diff --git a/tests/acceptance/init.spec.js b/tests/acceptance/init.spec.js index 629767618e44..bb9ddd6265c5 100644 --- a/tests/acceptance/init.spec.js +++ b/tests/acceptance/init.spec.js @@ -103,8 +103,7 @@ describe('Acceptance: ng init', function () { it('ng init', function () { return ng([ 'init', - '--skip-npm', - '--skip-bower' + '--skip-npm' ]).then(confirmBlueprinted); }); @@ -121,7 +120,6 @@ describe('Acceptance: ng init', function () { return ng([ 'init', '--skip-npm', - '--skip-bower', '--name', 'tmp' ]); @@ -133,15 +131,15 @@ describe('Acceptance: ng init', function () { }); it('init an already init\'d folder', function () { - return ng(['init', '--skip-npm', '--skip-bower']) + return ng(['init', '--skip-npm']) .then(function () { - return ng(['init', '--skip-npm', '--skip-bower']); + return ng(['init', '--skip-npm']); }) .then(confirmBlueprinted); }); it('init a single file', function () { - return ng(['init', 'package.json', '--skip-npm', '--skip-bower']) + return ng(['init', 'package.json', '--skip-npm']) .then(function () { return 'package.json'; }) @@ -149,15 +147,15 @@ describe('Acceptance: ng init', function () { }); it('init a single file on already init\'d folder', function () { - return ng(['init', '--skip-npm', '--skip-bower']) + return ng(['init', '--skip-npm']) .then(function () { - return ng(['init', 'package.json', '--skip-npm', '--skip-bower']); + return ng(['init', 'package.json', '--skip-npm']); }) .then(confirmBlueprinted); }); it('init multiple files by glob pattern', function () { - return ng(['init', 'src/**', '--skip-npm', '--skip-bower']) + return ng(['init', 'src/**', '--skip-npm']) .then(function () { return 'src/**'; }) @@ -165,15 +163,15 @@ describe('Acceptance: ng init', function () { }); it('init multiple files by glob pattern on already init\'d folder', function () { - return ng(['init', '--skip-npm', '--skip-bower']) + return ng(['init', '--skip-npm']) .then(function () { - return ng(['init', 'src/**', '--skip-npm', '--skip-bower']); + return ng(['init', 'src/**', '--skip-npm']); }) .then(confirmBlueprinted); }); it('init multiple files by glob patterns', function () { - return ng(['init', 'src/**', 'package.json', '--skip-npm', '--skip-bower']) + return ng(['init', 'src/**', 'package.json', '--skip-npm']) .then(function () { return '{src/**,package.json}'; }) @@ -181,9 +179,9 @@ describe('Acceptance: ng init', function () { }); it('init multiple files by glob patterns on already init\'d folder', function () { - return ng(['init', '--skip-npm', '--skip-bower']) + return ng(['init', '--skip-npm']) .then(function () { - return ng(['init', 'src/**', 'package.json', '--skip-npm', '--skip-bower']); + return ng(['init', 'src/**', 'package.json', '--skip-npm']); }) .then(confirmBlueprinted); }); diff --git a/tests/acceptance/new.spec.ts b/tests/acceptance/new.spec.ts index 9ce962d81d1d..454c3c010484 100644 --- a/tests/acceptance/new.spec.ts +++ b/tests/acceptance/new.spec.ts @@ -83,7 +83,7 @@ describe('Acceptance: ng new', function () { }); it('ng new foo, where foo does not yet exist, works', function () { - return ng(['new', 'foo', '--skip-npm', '--skip-bower']).then(confirmBlueprinted); + return ng(['new', 'foo', '--skip-npm']).then(confirmBlueprinted); }); it('ng new with empty app does throw exception', function () { @@ -95,7 +95,7 @@ describe('Acceptance: ng new', function () { }); it('ng new with app name creates new directory and has a dasherized package name', function () { - return ng(['new', 'FooApp', '--skip-npm', '--skip-bower', '--skip-git']).then(function () { + return ng(['new', 'FooApp', '--skip-npm', '--skip-git']).then(function () { expect(!existsSync('FooApp')); const pkgJson = JSON.parse(fs.readFileSync('package.json', 'utf8')); @@ -104,7 +104,7 @@ describe('Acceptance: ng new', function () { }); it('ng new has a .editorconfig file', function () { - return ng(['new', 'FooApp', '--skip-npm', '--skip-bower', '--skip-git']).then(function () { + return ng(['new', 'FooApp', '--skip-npm', '--skip-git']).then(function () { expect(!existsSync('FooApp')); const editorConfig = fs.readFileSync('.editorconfig', 'utf8'); @@ -113,9 +113,9 @@ describe('Acceptance: ng new', function () { }); it('Cannot run ng new, inside of angular-cli project', function () { - return ng(['new', 'foo', '--skip-npm', '--skip-bower', '--skip-git']) + return ng(['new', 'foo', '--skip-npm', '--skip-git']) .then(function () { - return ng(['new', 'foo', '--skip-npm', '--skip-bower', '--skip-git']).then(() => { + return ng(['new', 'foo', '--skip-npm', '--skip-git']).then(() => { throw new SilentError('Cannot run ng new, inside of ember-cli project should fail.'); }, () => { expect(!existsSync('foo')); @@ -125,7 +125,7 @@ describe('Acceptance: ng new', function () { }); it('ng new without skip-git flag creates .git dir', function () { - return ng(['new', 'foo', '--skip-npm', '--skip-bower']).then(function () { + return ng(['new', 'foo', '--skip-npm']).then(function () { expect(existsSync('.git')); }); }); @@ -140,7 +140,7 @@ describe('Acceptance: ng new', function () { }); it('ng new with --directory uses given directory name and has correct package name', function () { - return ng(['new', 'foo', '--skip-npm', '--skip-bower', '--skip-git', '--directory=bar']) + return ng(['new', 'foo', '--skip-npm', '--skip-git', '--directory=bar']) .then(function () { const cwd = process.cwd(); expect(cwd).to.not.match(/foo/, 'does not use app name for directory name');