Replace underscore with modern JS (#772)

* _.extend => Object.assign

* Rewrite plugman/variable-merge w/out underscore

* _.find => Array.prototype.find

* _.clone => Object.assign

* Rewrite plugman/util/dependencies w/out underscore

* Remove last occurence of underscore usage

* Remove dependency on underscore

chore: drop node 6 and 8 support (#799)

chore: bump version to 10.0.0-dev (#798)

Use up-to-date fixtures in tests (#770)

* Add helper to get up-to-date test fixtures

* Use up-to-date project fixture in E2E plugin dependency tests

* Use up-to-date platform fixture in plugman.install tests

* Use up-to-date platform in HooksRunner E2E tests

* Use up-to-date platform fixture in plugman.uninstall tests

* Use up-to-date project fixture in pkgJson plugin E2E tests

* Use up-to-date platform fixture in plugin.spec E2E tests

* Remove unused and outdated Android project fixture

* Remove unused project fixture "base"

* Use cordova-test-platform to test platform installation from local path

* Remove unused cordova-browser fixture

* Use cordova-test-platform to test getInstalledPlatformsWithVersions

* Remove unused cordova-android fixture

* Use testPlatformApp fixture to test cordova/util.getPlatformApiFunction

* Finally remove shelljs (again)

Had to be kept in devDeps before because of some outdated fixtures

* Make platformApi fixture project a minimal stub

HooksRunner code & spec cleanup (#796)

* Cleanup HooksRunner code

* Merge the 'plugin hooks' blocks in HooksRunner.spec

* Reduce amount of fixtures for HooksRunner test

fix: error message during plugin installation w/ missing engine (#791)

chore: add Node.js 12 to CI services (#777)

Quick workaround for e2e failure on AppVeyor CI (#786)

nyc@14 update in devDependencies (#783)

Merge pull request #775 from raphinesse/cpj-cleanup

Cleanup plugman.createPackageJson

Simpler and better cordova/util.getPlatformApiFunction (#767)

* Simpler and better cordova/util.getPlatformApiFunction

* Apply suggestions from code review

Co-Authored-By: raphinesse <raphinesse@gmail.com>

Collapse declaration of globals to a single line

Get rid of hacky default.json file for init-defaults

Remove unused parameter `test` from `readDeps`

The parameter `test` was actually never used in `readDeps`. That means
that both `dependencies` and `devDependencies` will be populated with
_all_ packages installed to node_modules XD

Remove unused key `defaults.bugs` in createpackagejson

Remove futile attempts to read npm config values

Our init-defaults.js apparently is a slightly modified copy of
init-package-json/default-input.js. When npm calls `init-package-json`
they probably pass an object parsed from their configuration file.

We on the other hand always pass `{}`. Which means that any call to
`config.get` will return `undefined`.

This commit replaces all these calls in our code with `undefined` and
does some constant folding to tidy things up.

Make src/plugman/init-defaults.js lintable (#774)

Work around using the reserved word `package` to be able to lint
src/plugman/init-defaults.js

Have plugman.createPackageJson create file in plugin dir, not in cwd (#773)

Right now `plugman createpackagejson <plugin dir>` reads values from `plugin.xml` in `<plugin dir>` but creates the `package.json` in `process.cwd()`. This seems to be undocumented behavior, and is considered a bug.

With this patch, we always create the `package.json` in `<plugin dir>` next to `plugin.xml` instead.

Prevent masking of errors during testing (#771)

Node.js throws an Error if the error event is emitted on a EventEmitter instance that has no handlers attached for it. This is usually the case during testing and often masks the actual error that is causing the issue.

So we attach a listener to just log error events to console. During passing tests runs, no additional output is generated.

Proper async code in src/plugman/createpackagejson.js (#768)

The existing code returned an instantly resolved promise despite the fact that it was doing async operations.

Merge pull request #764 from raphinesse/expressive-pkgJson.spec

Expressive pkg json.spec

Remove unnecessary spy (#763)

The stubbing is not actually needed for the tests. It rather causes problems if the tests are using an up-to-date cordova-android as that currently requires ActionStack.prototype.process to return a Q instance.

Remove unused fixtures

Merge describe blocks in pkgJson.spec

Create projects instead of asserting their properties in pkgJson.spec

Factor out common project test helpers

Remove unused fixtures (#762)

These are only test fixtures, so this is a patch-level change.

Related to #757

Merge pull request #761 from raphinesse/misc-test-improvements

Various test improvements

Minor cleanup of CI configs (#760)

* Cleanup CI configs

* Shallowly clone in AppVeyor CI

Fix some test descriptions in cordova/util.spec

Stop cordova/util.spec from messing with the user's home directory!

Proper temp folder usage in cordova/util.spec

Remove outdated test from cordova/util.spec

isCordova does not actually care about the user's home directory.

Remove rewire/revert anti-pattern

There is no need to clear all the test doubles in afterEach if we
re-set them in beforeEach anyway.

Remove superspawn faking from HooksRunner.spec

This sets executable bit for plugin hooks launched by superspawn,
so that they actually run.

Incremented package version to -dev

Update version and RELEASENOTES for release 9.0.1

Fix faulty Promise handling in plugman.uninstall (#759)

* Fix unhandled rejections with dependent plugin uninstall

This also includes two changes to improve the robustness of plugman_uninstall.spec:
* Proper mock return value for npmUninstall
* Get rid of fs-stubbing

Refactor plugin_fetch.spec (#758)

- Refactored the test to be more readable, robust and self contained
- Removed the reliance on the outdated fixtures
- Moved the test to spec/ since it is actually a unit test

Fix restoring plugins from package.json

Some of the code removed in GH-750 had side effects on the rest of the
plugin restoration procedure. Namely, despite wanting to migrate to
package.json, we actually relied on config.xml for retrieving the plugin
spec and variables. When we stopped syncing changes back to config.xml,
those plugins stopped getting restored.

I've refactored this significantly to reduce complexity and make it
easier to read and understand.

The existing tests did neither describe nor test the new behavior
sufficiently.

Do not wrap engine script path in quotes (#755)

Fixes #754
Fixes apache/cordova#52

GH-10 Incremented package version to -dev

GH-10 Updated version and RELEASENOTES.md for release 9.0.0

Remove saving platforms/plugins to config.xml (#750)

We've had one major version of cordova-lib that attempted to ensure both
config.xml and package.json were kept in sync, and now it's time to push
people more strongly towards using package.json.

We will still attempt to read values from config.xml and update them in
package.json, but we will no longer reflect changes back to config.xml.
Whatever's saved in package.json should always have priority over what
is read from config.xml.

The next phase will be to improve the handling of package.json updates,
and then in the next major to completely remove the code that looks at
config.xml for platforms/plugins.

Pass project config.xml path to platform's prepare (#751)

Currently we're passing in an already-constructed ConfigParser instance
as part of the fake projectInfo object, but the problem is that it's
constructed with the version of cordova-common depended on by
*cordova-lib* as opposed to the version of cordova-common depended on by
the *platform*. By passing the path in, we can construct the
ConfigParser at the Platform API level, using the correct version.

In the meantime, for compatibility reasons, we have to keep passing in a
ConfigParser instance, and the Platform API needs to handle the case
where rootConfigXml is undefined and it needs to fetch the path out of
the existing ConfigParser instance and then overwrite it with a newly
constructed ConfigParser.

We should at some point look into making this "root project info" object
a class in cordova-common so that we can document what it's expected to
contain and rely on that both here and in the platforms.

Merge pull request #749 from erisu/cordova-9-prep

Cordova Lib Release Preparation (Cordova 9)

Remove unused property apiCompatibleSince from platformsConfig.json

Property is unused since 9f1fbf4918cd

Fix plugin dependency tests when using npm >= 5

npm >= 5 automatically prunes any unsaved packages during installation
(see https://github.com/npm/npm/issues/16853). Therefore, installing
platforms and plugins without saving them to package.json is essentially
broken right now.

Bump All Platform Pinning and Support Minor SemVer

Bump dependency jasmine@^3.3.1

Bump dependency globby@^9.1.0

Bump dependency underscore@^1.9.1

Bump dependency semver@^5.6.0

Bump dependency read-chunk@^3.1.0

Bump dependency init-package-json@^1.10.3

Bumped dependency fs-extra@^7.0.1

Added missing dev dependency shelljs to fix failing tests

Remove handling of legacy .fetch.json files (#748)

The files in question have last been used in Cordova 4.3.0 and have been
removed by commit c49eaa86c92b.

Updated Dev Dependencies

- codecov@^3.2.0
- delay@^4.1.0

Updated nyc code coverage dependency

Updated ESlint dependencies with lint corrections

Updated Package Cordova Dependencies

- cordova-common@^3.1.0
- cordova-create@^2.0.0
- cordova-fetch@^2.0.0
- cordova-serve@^3.0.0

Merge pull request #709 from raphinesse/next-gen-requireCordovaModule

- Throw error if non-cordova module is required (Resolves #689)
- Remove outdated compatibility layer from 2015
- Lazily load Context.cordova to avoid cyclical dependency
- Use const for imports

CB-14166: (cli) Fixed issue when install plugins on windows (#622)

The npm package name is already wrapped in quotes to prevent special characters from being execute. However the additional win32 code was further injecting quotes around the version causing npm to return a EINVALIDTAGNAME error. The win32 code is redundant and seems to not be required.

Add or update GitHub pull request and issue template

Add Electron Platform (#744)

* Add Electron Platform
* Updated test spec to include Electron

Updated Platform Config Git URL Paths (#743)

Cleanup indentation spacing in */jasmine.json (#742)

Fix crash in `cordova requirements` due to an unbound function (#741)

Functions in utils.js expect to be called with `this` bound to the
module. So this call crashed with an error
    this.isCordova is not a function

Fixes #740.

remove JIRA link

Fix 2 integration tests failures on macOS

Merge pull request #710 from raphinesse/q-nomore

Drop dependency on Q, use native promises

Remove Q :tada:

Fix some unhandled promises

Add ESLint rule against unhandled promises

Rewrite cordova/requirements to get rid of `Q.allSettled`

Behavior of cordova/requirements should remain unchanged.

Use `superspawn` to get rid of `Q.defer` in plugman/install

Replace `Q.delay` with npm package `delay`

Fix linting error `prefer-promise-reject-errors`

Search and replace usage of Q's static API

Replace leftover usage of Q's instance API

Merge pull request #687 from raphinesse/to-serve-and-protect

Test, fix and cleanup `cordova serve`

Use const for imports

Lazily load Context.cordova

Remove compatibility layer from 2015

Throw error if non-cordova module is required

Resolves #689

Merge pull request #707 from raphinesse/deprecate-requireCordovaModule

Deprecate `requireCordovaModule` for non-Cordova modules

Update JSDoc and clarify inline comments

Emit deprecation warning when requiring non-cordova modules

Fix for module names that start with "cordova-lib"

Add unit tests for `Context#requireCordovaModule`

Some of these are failing at this point

Dereference possible symlinks when copying plugin (#705)

When `npm install`ing a local plugin, npm will create a symlink to the source. When we then attempt to copy that symlink, the result will be invalid on Windows OS. Thus, we need to dereference the link/junction and create a proper copy.

Fixes #700
Fixes #704

Increase timeout for cordova.platform e2e tests (#699)

Increase plugman install test timeout (#698)

Update cordova-windows pattern to ~6.0.0 (#691)

Start 9.0.0-dev (#690)

Fix waiting for launched server

Add failing test for handling failed server launch

Organize imports

Improve tests after improving testability