Target next major version 6.0.0 (#217)

Minor formatting improvements in cordova.js (#218)

ESLint improvements (#204)

* Add proper ESLint configuration

* Move test-build script for easier linting

* Fix ESLint violations instead of ignoring them

* Remove unused function

* Remove node-only code

* Check that we only use ES5 in our browser code

Drop unused test helpers (#214)

* Remove unused module mockxhr

* Remove unnecessary test utils

* Remove unused placeholders

* Inline testmodule fixture

* Move test-platform modules to test/

Collect & report test coverage for our browser code (#209)

Make built JS output a bit prettier (#208)

In #205 I took great care to preserve the _exact_ format of the built
JS code, so the build refactoring could be verified by diffing the
build output. However, some of the existing formatting is sub-optimal
so this commit optimizes it a little.

Most notably, this commit will change file path comments to be relative
paths, not absolute ones. An example for this:

```diff
-// file: /home/raphinesse/code/cordova-android/cordova-js-src/exec.js
+// file: ../cordova-android/cordova-js-src/exec.js
```

Add platforms used during tests as devDependencies (#207)

Completely refactor build system (#205)

The goals of this refactoring are the following:

- Improve code quality of the build system
- Prepare code base for adding code coverage
- Prepare code base for exposing the build system as a package export,
  so platforms can build their `cordova.js` during their build (or
  creation) process. This would make the JS build using coho obsolete.
- Prepare code base to remove dependency on Grunt

The build process now lives under `build-tools` (was `tasks`). It does
not depend on Grunt anymore but is written in plain Node.js.

The original Grunt interface for building (as used by coho) was
preserved and is implemented in `Gruntfile.js` using the new build
system. The `platformName` option and support for getting platform paths
from the `cordova-platforms` key in `package.json` have been removed
from the Grunt interface, but neither of those are used in coho.

The logic that is specific to the test build has been extracted from
the rest of the build. It now lives in `test/build.js` and is run
automatically during `npm test`.

The following dependencies have been added:

- execa
- fs-extra
- globby

I have taken extra care to preserve the exact format of the built file.
This means that the correctness of the refactoring can be verified by
simply diffing the build artifacts in `pkg` created with and without
this change.

Add Node.js 12 to CI Services (#199)

Document argscheck.checkArgs (#198)

Improve README (#201)

- add title
- fix headline levels
- fix typo
- fix indentation

Fix parsing of some parameter formats in argscheck.checkArgs (#200)

Use .gitgnore as .eslintignore (#197)

Allow to define modules whose name appears on `Object.prototype` (#196)

One of the CommonJS tests adopted in #195 exposed a defect in Cordova's module system:

When trying to define a module with a name that also is a property of Object.prototype, an exception would occur.

Integrate unused CommonJS tests where applicable (#195)

There has been an unused copy of [The Official Specs for CommonJS][1]
lying around in our repository for who knows how long. They needed to be
built using a python script and then manually executed in a browser.
This commit integrates all of those tests that also apply to our module
system into our existing test suite for `require` & `define`.

Adds adapted versions of the following CommonJs tests to our
`test.require.js`:

    absolute, transitive, determinism, method, nested, hasOwnProperty

The hasOwnProperty test actually uncovered a bug in our module system
implementation and thus is deactivated right now.

The following CommonJS tests were not applicable to Cordova's module
system since it does not support cyclic dependencies at all:

    cyclic, monkeys

The following CommonJS tests were already covered by tests in
`test.require.js`:

- missing: covered by Test#005
- relative: covered by Test#012
- exactExports: covered by multiple tests

[1]: https://github.com/commonjs/commonjs

Minor cleanup of require/define tests (#194)

* Remove Test#012 which duplicated Test#008

* Fix/Improve some test descriptions

* Reset modules after each require/define test

* Be more specific in Test#008 & Test#011

* Cosmetic changes

Update README (#193)

* Remove outdated section about `contextify` from README

* Update test instructions

* Advocate `npx` instead of global `grunt-cli` installation

Merge pull request #192 from raphinesse/update-dependencies

Update dependencies & fix resulting ESLint violations

Fix new ESLint object-curly-spacing violations

Update dependencies

Drop build/test support for Node.js 6 (#190)

Some npm packages, including some of our build/test dependencies have
begun dropping support for Node.js 6 in their latest versions. To
facilitate dependency updates, we remove support for Node.js 6 here.

This package is usually not built and tested on our user's machines,
but is instead bundled with the platforms by us. Thus, this change
should not be considered a breaking change.

Lint (almost) all JS files (#191)

* Lint (almost) all commited JS files

* Fix all new ESLint violations

Remove obsolete stuff from AppVeyor CI config (#189)

Add Electron Support (#188)

Add or update GitHub pull request and issue template

GH-10 Incremented package version to -dev

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

Merge pull request #176 from raphinesse/karma

Migrate tests to Jasmine 3 driven by Karma

remove JIRA link

Remove test & cover tasks + orphaned dependencies

Run tests in browser using Karma

Fix access to Cordova's define & require in tests

Fix test cleanup to not break other tests

Jasmine 3: Replace usage of Spec#after

Jasmine 3: Replace usage of `waitsFor` and `runs`

Jasmine 3: Use Spy#calls to access call data

Co-authored-by: Raphael von der Grün <raphinesse@gmail.com>

Jasmine 3: Use Spy#and to access SpyStrategy

Jasmine 3: Use `toThrowError` to match error messages

Co-authored-by: Raphael von der Grün <raphinesse@gmail.com>

Merge pull request #175 from raphinesse/remove-unused-stuff

Remove unused stuff

Remove unused task whitespace-check

Remove unused dependency `express`

Remove unused jasmine suite template

Remove unused util for drawing require-graph

Remove unused module `modulereplacer`

Remove unused vendor/jasmine folder

Remove leftover JSHint stuff

Remove support for deprecated platforms

Add test labels from #140 that were missed in #172 (#174)

Remove task that depended on browserify (#173)

add test labels (GH-140) (#172)

Co-authored-by: audreyso <auso@adobe.com>
Co-authored-by: Christopher J. Brody <chris.brody@gmail.com>

Merge pull request #162 from brodybits/cb-remove-browserify

[CB-14156] Remove browserify

Remove browserify instructions from README.md

Remove module to write license header for browserify build

Remove dependency on browserify

Remove browserify sources src/**/*_b.js

Remove browserify tasks tasks/**/*--browserify.js

Remove browserify tasks from Gruntfile & scripts

Mark 5.0.0-dev (#170)

Needed since we did already drop Node.js 4 support and are planning to remove browserify

Merge pull request #168 from erisu/CB-14071

CB-14071: Drop Node 4 Support

CB-14071: Drop Node 4 Support

readme fixes (#166)

* Add quote

* Fix spelling in readme

CB-14152 Increment package version to 4.2.5-dev

CB-14152 Update version, RELEASENOTES.md for 4.2.4

CB-14155 fix btest part 2: export npm script

Followup to btest fix in PR #160

CB-9366 Log error.stack ...

as it helps with diagnosing the source of the error

JavaScript fixed by @brodybits (Christopher J. Brody)

Fixes #118

Co-authored-by: Nikhil Khandelwal <nikhilkh@microsoft.com>
Co-authored-by: Christopher J. Brody <chris.brody@gmail.com>

Fix btest by using express (instead of connect...)

THANKS for GUIDANCE:
https://medium.freecodecamp.org/getting-off-the-ground-with-expressjs-89ada7ef4e59

CB-14145: package.json use jasmine-node@^1.14.6

resolve npm audit issues

CB-14145 drop open & other deps for broken btest

resolves some npm audit issues

CI add Node.js 10

appveyor.yml use npm@5

(QUICK FIX WORKAROUND for Node.js 4.x)

Increment package version to 4.2.4-dev

Update version & RELEASENOTES.md for release 4.2.3

.travis.yml add spacing

remove grunt jshint artifacts not needed

resolves some of the npm audit warnings

Merge pull request #150 from audreyso/CB-13501

CB-13501 : added support for node 8

CB-13501 : added support for node 8

CB-13380 Incremented package version to -dev

CB-13380 Updated version and RELEASENOTES.md for release 4.2.2

updated browserify to 14.4.0

CB-12017 : updated dependencies for browserify

 This closes #141

CB-12762 : point package.json repo items to github mirrors instead of apache repos

 This closes #145

CB-12895 : added eslint to repo

CB-13232: added test for cordova's unique local style require

 This closes #148

Revert "CB-13163: fix using relative paths in calls to require."

This reverts commit 65fd0f61cca4d979e653ab5cf2dbc32b50416ba9.

CB-8990: bump nodejs requirement to 4.0.0+

CB-13163: fix using relative paths in calls to require.

CB-12847: fixed `bugs` entry in package.json.

CB-12847: added `bugs` entry to package.json.

CB-12748 - Update CI to test node 4 and 6

 This closes #142

CB-12358 Incremented package version to -dev

CB-12358 Updated version and RELEASENOTES.md for release 4.2.1

CB-12358: updated deps for release

CB-12039 Incremented package version to -dev

CB-12039 added license to package.json

CB-12039 Updated version and RELEASENOTES.md for release 4.2.0

CB-12017: updated dependencies in package.json

 This closes #139

Update connect dependency to 1.9.2 for node version incompatibility

this closes #136

Fix CB-3785

Enable support of EventListener interface for
Channel.prototype.subscribe / unsubscribe

 This closes #130

Add github pull request template