A huge improvement in package build-system

* several files renames (to make the sources more understandable)
* add `builtin` package for babel (see babel/babel/issues/3083 )
* switch testing to the DIST version of package (not sources)
* probably fix the problems in browser testing (`instance of` now works), see #49

.babelrc

@@ -1,3 +1,8 @@
 {
-	presets: ["es2015"]
+	"presets": ["es2015"],
+	"plugins": [
+        ["babel-plugin-transform-builtin-extend", {
+            globals: ["Error"]
+        }]
+    ]
 }

examples/.eslintrc.js

@@ -5,6 +5,7 @@ exports = module.exports = {
 	"globals": {
 		"expect": true,
 		"Route4Me": true,
+		"packageRoot": true,
 	},
 
 	"rules": {

examples/init-examples-suite.js

@@ -1,3 +1,4 @@
 "use strict"
 
-global.Route4Me = require("..")
+// eslint-disable-next-line import/no-dynamic-require
+global.Route4Me = require(`${packageRoot}`)

package.json

@@ -64,6 +64,7 @@
     "ajv": "^4.11.2",
     "babel-core": "^6.23.1",
     "babel-loader": "^6.3.2",
+    "babel-plugin-transform-builtin-extend": "^1.1.2",
     "babel-preset-es2015": "^6.22.0",
     "bluebird": "^3.4.7",
     "chai": "^3.5.0",

test/.eslintrc.js

@@ -3,7 +3,8 @@ exports = module.exports = {
 		"mocha": true,
 	},
 	"globals": {
-		"expect": true
+		"expect": true,
+		"packageRoot": true,
 	},
 
 	"rules": {

test/bootstrap.js

@@ -1,10 +1,12 @@
 "use strict"
 
+const path = require("path")
 const chai = require("chai")
 const chai_jsonschema = require("chai-json-schema")
 
 chai.use(chai_jsonschema)
 // Validate multiple errors for one assert
 chai.tv4.multiple = true
 
+global.packageRoot = path.resolve(path.join(__dirname, "..", "dist"))
 global.expect = chai.expect

test/package.spec.js

@@ -3,8 +3,6 @@
 const pkg = require("../")
 const helper  = require("./helper")
 
-const pkgRoute4Me = require("../src/route4me")
-
 const testApiKey = "11111111111111111111111111111111"
 
 describe(helper.toSuiteName(__filename), () => {
@@ -36,7 +34,7 @@ describe(helper.toSuiteName(__filename), () => {
 			const route4me = new Route4Me(testApiKey)
 
 			expect(route4me).to.be.an("object")
-			expect(route4me).to.be.an.instanceof(pkgRoute4Me)
+			expect(route4me).to.be.an.instanceof(pkg.Route4Me)
 
 			// expect(route4me.url_base).to.equal("http://route4me.com")
 		})
@@ -45,7 +43,7 @@ describe(helper.toSuiteName(__filename), () => {
 			const route4me = pkg(testApiKey)
 
 			expect(route4me).to.be.an("object")
-			expect(route4me).to.be.an.instanceof(pkgRoute4Me)
+			expect(route4me).to.be.an.instanceof(pkg.Route4Me)
 
 			// expect(route4me.url_base).to.equal("http://route4me.com")
 		})

test/resources/activity-feed.spec.js

@@ -5,7 +5,8 @@ const saMock  = require("superagent-mocker")(request)
 
 const helper  = require("./../helper")
 
-const Route4Me = require("../../src/route4me")
+// eslint-disable-next-line import/no-dynamic-require
+const Route4Me = require(`${packageRoot}/route4me`)
 
 const testApiKey = "11111111111111111111111111111111"
 

test/resources/address-book.spec.js

@@ -6,7 +6,8 @@ const saMock  = require("superagent-mocker")(request)
 
 const helper  = require("./../helper")
 
-const Route4Me = require("../../src/route4me")
+// eslint-disable-next-line import/no-dynamic-require
+const Route4Me = require(`${packageRoot}/route4me`)
 
 const testApiKey = "11111111111111111111111111111111"
 

test/resources/addresses.spec.js

@@ -5,7 +5,8 @@ const saMock  = require("superagent-mocker")(request)
 
 const helper  = require("./../helper")
 
-const Route4Me = require("../../src/route4me")
+// eslint-disable-next-line import/no-dynamic-require
+const Route4Me = require(`${packageRoot}/route4me`)
 
 const testApiKey = "11111111111111111111111111111111"
 

test/resources/avoidance-zones.spec.js

@@ -5,7 +5,8 @@ const saMock  = require("superagent-mocker")(request)
 
 const helper  = require("./../helper")
 
-const Route4Me = require("../../src/route4me")
+// eslint-disable-next-line import/no-dynamic-require
+const Route4Me = require(`${packageRoot}/route4me`)
 
 const testApiKey = "11111111111111111111111111111111"
 

test/resources/geocoding.spec.js

@@ -5,7 +5,8 @@ const saMock  = require("superagent-mocker")(request)
 
 const helper  = require("../helper")
 
-const Route4Me = require("../../src/route4me")
+// eslint-disable-next-line import/no-dynamic-require
+const Route4Me = require(`${packageRoot}/route4me`)
 
 const testApiKey = "11111111111111111111111111111111"
 

test/resources/members.spec.js

@@ -5,7 +5,8 @@ const saMock  = require("superagent-mocker")(request)
 
 const helper  = require("./../helper")
 
-const Route4Me = require("../../src/route4me")
+// eslint-disable-next-line import/no-dynamic-require
+const Route4Me = require(`${packageRoot}/route4me`)
 
 const testApiKey = "11111111111111111111111111111111"
 

test/resources/notes.spec.js

@@ -5,7 +5,8 @@ const saMock  = require("superagent-mocker")(request)
 
 const helper  = require("./../helper")
 
-const Route4Me = require("../../src/route4me")
+// eslint-disable-next-line import/no-dynamic-require
+const Route4Me = require(`${packageRoot}/route4me`)
 
 const testApiKey = "11111111111111111111111111111111"
 

test/resources/optimizations.spec.js

@@ -5,7 +5,8 @@ const saMock  = require("superagent-mocker")(request)
 
 const helper  = require("./../helper")
 
-const Route4Me = require("../../src/route4me")
+// eslint-disable-next-line import/no-dynamic-require
+const Route4Me = require(`${packageRoot}/route4me`)
 
 const testApiKey = "11111111111111111111111111111111"
 

test/resources/orders.spec.js

@@ -5,7 +5,8 @@ const saMock  = require("superagent-mocker")(request)
 
 const helper  = require("./../helper")
 
-const Route4Me = require("../../src/route4me")
+// eslint-disable-next-line import/no-dynamic-require
+const Route4Me = require(`${packageRoot}/route4me`)
 
 const testApiKey = "11111111111111111111111111111111"
 

test/resources/routes.spec.js

@@ -5,7 +5,8 @@ const saMock  = require("superagent-mocker")(request)
 
 const helper  = require("./../helper")
 
-const Route4Me = require("../../src/route4me")
+// eslint-disable-next-line import/no-dynamic-require
+const Route4Me = require(`${packageRoot}/route4me`)
 
 const testApiKey = "11111111111111111111111111111111"
 

test/resources/territories.spec.js

@@ -5,16 +5,15 @@ const saMock  = require("superagent-mocker")(request)
 
 const helper  = require("./../helper")
 
-const Route4Me = require("../../src/route4me")
-const errors   = require("../../src/errors")
+// eslint-disable-next-line import/no-dynamic-require
+const route4me = require(`${packageRoot}`)
 
 const testApiKey = "11111111111111111111111111111111"
 
-
 describe(helper.toSuiteName(__filename), () => {
 	describe("SDK methods", () => {
-		const route4me = new Route4Me(testApiKey)
-		const resource = route4me.Territories
+		const route4meClient = new route4me.Route4Me(testApiKey)
+		const resource = route4meClient.Territories
 		let req
 
 		beforeEach(() => {
@@ -155,10 +154,11 @@ describe(helper.toSuiteName(__filename), () => {
 							{ "territory_id": "AAAAAAAAAAAAAAA9D5CA2EA375E08B97" },
 							null
 						)
-
-						expect(err).is.instanceof(errors.Route4MeValidationError)
+console.log(route4me.Route4MeValidationError)
+console.log(err)
 						expect(err).has.property("message")
 							.that.match(/valid/i)
+						expect(err).is.instanceof(route4me.Route4MeValidationError)
 						done()
 					})
 				})

test/resources/tracking.spec.js

@@ -5,7 +5,8 @@ const saMock  = require("superagent-mocker")(request)
 
 const helper  = require("./../helper")
 
-const Route4Me = require("../../src/route4me")
+// eslint-disable-next-line import/no-dynamic-require
+const Route4Me = require(`${packageRoot}/route4me`)
 
 const testApiKey = "11111111111111111111111111111111"
 

test/resources/vehicles.spec.js

@@ -5,7 +5,8 @@ const saMock  = require("superagent-mocker")(request)
 
 const helper  = require("../helper")
 
-const Route4Me = require("../../src/route4me")
+// eslint-disable-next-line import/no-dynamic-require
+const Route4Me = require(`${packageRoot}/route4me`)
 
 const testApiKey = "11111111111111111111111111111111"