diff --git a/src/helpers/helpers.core.js b/src/helpers/helpers.core.js index a2a67245de2..bc715a74df1 100644 --- a/src/helpers/helpers.core.js +++ b/src/helpers/helpers.core.js @@ -39,7 +39,6 @@ var helpers = { */ isArray: function(value) { var type = Object.prototype.toString.call(value); - // Typed array if (type.substr(0, 7) === '[object' && type.substr(-6) === 'Array]') { return true; } diff --git a/test/.eslintrc.yml b/test/.eslintrc.yml index 9d98c452878..124624f681d 100644 --- a/test/.eslintrc.yml +++ b/test/.eslintrc.yml @@ -1,4 +1,8 @@ +parserOptions: + ecmaVersion: 5 # don't rely on default, since its changed by env: es6 + env: + es6: true # also changes default ecmaVersion to 6 jasmine: true globals: diff --git a/test/specs/helpers.core.tests.js b/test/specs/helpers.core.tests.js index e3993de635a..cdda01b9674 100644 --- a/test/specs/helpers.core.tests.js +++ b/test/specs/helpers.core.tests.js @@ -21,6 +21,15 @@ describe('Chart.helpers.core', function() { expect(helpers.isArray([42])).toBeTruthy(); expect(helpers.isArray(new Array())).toBeTruthy(); expect(helpers.isArray(Array.prototype)).toBeTruthy(); + expect(helpers.isArray(new Int8Array(2))).toBeTruthy(); + expect(helpers.isArray(new Uint8Array())).toBeTruthy(); + expect(helpers.isArray(new Uint8ClampedArray([128, 244]))).toBeTruthy(); + expect(helpers.isArray(new Int16Array())).toBeTruthy(); + expect(helpers.isArray(new Uint16Array())).toBeTruthy(); + expect(helpers.isArray(new Int32Array())).toBeTruthy(); + expect(helpers.isArray(new Uint32Array())).toBeTruthy(); + expect(helpers.isArray(new Float32Array([1.2]))).toBeTruthy(); + expect(helpers.isArray(new Float64Array([]))).toBeTruthy(); }); it('should return false if value is not an array', function() { expect(helpers.isArray()).toBeFalsy();