GH-693 Update version & RELEASENOTES.md for release 8.1.0

CB-14166 distinguish npm.cmd and npm.exe in win32 (#697)

to check if escape is necessary

(workaround solution in 8.1.x only)

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

src/plugman/uninstall.js workaround for Node.js 4

src/hooks/HooksRunner.js workarounds for Node.js 4

EXTRA pkgJson-restore.spec.js workaround for Node.js 4

pkgJson.spec.js workarounds for Node.js 4

pkgJson-restore.spec.js workarounds for Node.js 4

spec/cordova/plugin/remove.spec.js workaround for Node.js 4

spec/helpers.js workaround for deprecated Node.js 4

HooksRunner test fixes for deprecated Node.js 4 (8.1.x)

Downgrade to fs-extra@4 & move to devDependencies

- fs-extra@4 needed to support deprecated Node.js version 4
- fs-extra currently not used outside of spec & integration-tests

Partially Revert "CB-14065: Drop support for Node 4 ..."

This partially reverts commit 61395d2946a743a0a5ad5bf273a57701af15a145
but keeps Node.js 10 in AppVeyor CI and Travis CI

Start 8.1.0-dev (minor release)

fetch.spec: 30% shorter runtime by factoring out project setup (#640)

Fix false alarm for long first line in HooksRunner.spec

Only warn about "too long shebang" if the file really starts with one

Use shebang-command module to extract shebang

Covers a few more cases and reduces maintenance for us.

Fix reading garbage shebang in HooksRunner.spec

Fixes #653

Add test for obscure HooksRunner bug (#653)

Fix sporadic timeout of HooksRunner test 012 (#656)

Test 012 sporadically takes slightly longer than the current default
timeout of 10s, so this increases the timeout for this test to 20s.

Related to #642

Organize imports in HooksRunner.spec (#654)

Minor cleanup of the imports in HooksRunner.spec. No functional changes.

Fix setDefaultTimeout for beforeAll and afterAll (#648)

Previously, the timeout defined by setDefaultTimeout did not apply for
beforeAll and afterAll blocks. This change fixes that.

The new implementation makes use of the fact that Jasmine processes
beforeAll in FIFO order and afterAll in LIFO order.

Avoid requireNoCache in pkgJson.spec (#647)

See #644

Cleanup pkgJson.spec (#635)

This commit does not drop any test cases. However, it fixes various bugs
in the existing expectations and often asserts stronger expectations
than the original tests.

Changes include:

- DRY code
- Improve formatting
- Reduce excessive comments
- Simplify platform list generation
- Wrap all tests in common describe to allow common setup/teardown
- Use a fresh tmpDir for every test to avoid caching issues
- Use fs-extra for setup/teardown
- FIX Wait for promise
- Properly set default timeout for the whole suite
- Don't symlink into fixtures
- Sane config file handling
- Re-enable disabled test 005 that was fixed by #628
- Do not quote props when not needed
- Do not pass array when we only have scalar
- Nicer imports
- Save test platforms/plugins as constants

Update android platform to latest release (#646)

This will remove most noise during E2E tests.

CB-14148 remove nonsense "www" platform from Cordova listing (#620)

and update the tests

Co-authored-by: Christopher J. Brody <chris.brody@gmail.com>
Co-authored-by: Raphael von der Grün <raphinesse@gmail.com>

Avoid requireNoCache in pkgJson-restore.spec (#645)

See #644

Merge pull request #641 from raphinesse/create-not-assert

Create test projects instead of asserting their properties

Delete obsolete fixtures

Create test projects instead of asserting them

Honest specs in pkgJson-restore.spec (#638)

This aligns the code of various specs with their respective
descriptions. Affects tests 001, 002, 003, 004 and 016.

Pin Jasmine to 3.1 to work around Promise issues (#639)

A change in Jasmine 3.2 seems to be responsible for the recent CI failures. Pinning it to 3.1 for now.

See https://github.com/apache/cordova-ios/issues/393,
See https://github.com/jasmine/jasmine/issues/1590
See #636

Make coverage script consider e2e tests too (#637)

Uses nyc for code coverage

Merge pull request #634 from raphinesse/cleanup-spec-restore

Cleanup pkgJson-restore.spec

pkgJson-restore.spec: Better names for various tests

This commit is separated from the previous cleanup to make the diff more
readable.

pkgJson-restore.spec: Merge more describe blocks

This commit is separated from the previous cleanup to make the diff more
readable.

Cleanup pkgJson-restore.spec

This commit does not drop any test cases. However, it fixes various bugs
in the existing expectations and often asserts stronger expectations
than the original tests.

Changes include:

- DRY code
- Improve formatting
- Reduce excessive comments
- Simplify platform list generation
- More meaningful usage of describe blocks
- Wrap all tests in common describe to allow common setup/teardown
- Use fs-extra for setup/teardown

Merge pull request #633 from raphinesse/minor-spec-improvements

Minor spec improvements

DRY platform.check unit tests

Remove obsolete ESLint comments

Randomize test tmpDir to avoid caching issues

Add helper to set default timeout properly

More compact jasmine helper definition

Use more verbose test reporter

This new reporter gives a better idea which tests are currently running,
how long individual tests take and during which tests output occurred.

All of this information is especially helpful when tests are run in CI.

Merge pull request #631 from amaldonado/CB-14243

CB-14243: change dash to underscore for save-exact key reference

CB-14243: change dash to underscore for save-exact key reference

Improve plugman/uninstall.js messages (#630)

* Improve plugman/uninstall.js messages
* Adapt plugman_uninstall.spec.js to new error messages

Fix promise handling when removing multiple plugins (#628)

CB-14033 Support symbolic dir links on Windows (#621)

This uses junctions whenever creating directory symlinks on Windows
since that does not require any special privileges.

Merge pull request #624 from raphinesse/fetch-nomore

CB-13055 Fold all fetch options to `true`

CB-13055 Fold all fetch options to `true`

Mock out various functions that call cordova-fetch in tests

Simplify rewire usage in platform.remove spec

Fix test that was outside of an `it` call

restore.spec: Wait for prepare promise to resolve (#626)

One test in pkgJson-restore.spec did NOT wait for the promise returned
by `prepare` to be resolved before continuing with other tasks.
Especially on Windows this could lead to EBUSY failures during cleanup
and in turn to failures during following tests.

Note: The order of emptyPlatformList() and prepare() does not seem to matter.
Thus I chose the variant that was used throughout all other tests.

Set the fetch option to true in plugman (#617)

Clean up messy HooksRunner tests (#623)

These tests tended to cause spurious fails on CI.
Hopefully, this commit will stabilize them.

Noteworthy changes include:
- Reset test files before each test
- Don't disguise setup/teardown as tests
- Test 020 was dropped since it was actual a weaker version of 019
- Compare objects instead of JSON strings
- Don't use reserved words as variable names
- Reduce test duration by about 50%
- Fix linting errors
- Remove shelljs usage
- DRY and simplify code
- 200 LOC less

CB-14140 Add fs-extra to allow for incremental adoption

Work around AppVeyor/Node10 issue

Let Jasmine handle async test results (#619)

There was a lot of sub-optimal error handling in the tests.

Instead of handling Promise rejection and resolution ourselves, we now
just return them from the test functions and let Jasmine handle them.

Moreover, this also fixes tons of negative tests that looked something like this:

    f().then(function () {
        expect('foo').toBe('bar');
    }).catch(function (err) {
        expect('' + err).toMatch('some message');
    });

Problems being:
- `expect('foo').toBe('bar')` is less obvious than `fail(...)`
- catches its own Error from the `.then` only to fail because `err` is missing
- Does not check that err is an Error and that the message is in `.message`

So these were normalized to look like this:

    f().then(function () {
        fail('Expected promise to be rejected');
    }).catch(function (err) {
        expect(err).toEqual(jasmine.any(Error));
        expect(err.message).toMatch('some message');
    });

All in all, this commit has the following benefits:
- Removes 700 LOC w/out removing any testing
- Improves output when async tests fail
- Should make async tests less error prone
- Reduces Q-API usage to prepare for removal of Q (CB-14159)

to pass the appveyor tests, adjusting timeouts, cleaning event listeners and update cordova-browser to 5.0.3

improve tests using ios@4.5.4, adjusting timeout params and improve plugin remove sequentially

CB-13532: Find plugins in devDependencies

Merge pull request #612 from raphinesse/drop-node4

CB-14065: Drop support for Node 4 and add Node 10 to CI

CB-14065: Drop support for Node 4 and add Node 10 to CI

Also removed before_install from Travis configuration since Node 6
already comes with npm 3 there

Merge pull request #611 from dpogue/dependency-cleanup

Dependency cleanup

Fix failing test from inconsistent error format

Fix ElementTree error about unclosed XML tag

Update (and remove) several dependencies

Added browser default engine

This should be replaced at some point with a more dynamic method of looking up platform engines.

CB-13674 Incremented package version to -dev

CB-13674 Updated version and RELEASENOTES.md for release 8.0.0

CB-13057: updated based on feedback

 This closes #586

CB-13057 : removed platformsJson from integration tests

CB-13057 : removed save function and updated unit-tests after these changes

CB-13056: fixed tests

CB-13056 : updated browser test files

 This closes #588

CB-13056 : added error message back in

CB-13056 : removed PlatformApiPoly and tests

CB-13056: rebased from master and updated unit tests

CB-13056 : removed parsers and platformApi polyfill for webos, blackberry10, and ubuntu

CB-13674: updated pinned platforms

CB-13055: fixed failing tests

Merge pull request #10 from stevengill/master

Fixed failing tests

 This closes #608

CB-13674: updated cordova dependencies

CB-13055: fixed failing tests

CB-13055 : updated after PR feedback

CB-13055 : added info to plugin util file and rm registry and updated tests

CB-13055 : updated integration tests, removed lazy_load.js, removed gitclone and nofetch, and npm dependency

CB-13055 : updated after PR feedback

 This closes #608

CB-13055 : added info to plugin util file and rm registry and updated tests

CB-13055 : updated integration tests, removed lazy_load.js, removed gitclone and nofetch, and npm dependency

fixed failing test

Merge pull request #607 from audreyso/CB-13532

CB-13532 : updated to include a check for pkgJson devDependencies

Merge pull request #603 from audreyso/CB-12361-check.js

CB-12361 : added unit tests for check.js

Merge pull request #606 from audreyso/CB-12810

CB-12810 : keep existing formatting in package.json and fix eslint errors

CB-12810 : updated indenting for pkgJson and fixed eslint errors

:CB-13532 : updated to include a check for pkgJson devDependencies

Merge pull request #604 from audreyso/CB-13501

CB-13501 : added support for node 8 and cleaned up eslint

CB-13501 : added support for node 8 and cleaned up eslint