is-generator-function - npm Package Compare versions

Comparing version 1.0.2 to 1.0.3

index.js

@@ -8,2 +8,3 @@ 'use strict';
 module.exports = function isGeneratorFunction(fn) {
+	if (typeof fn !== 'function') { return false; }
 	var fnStr = toStr.call(fn);
@@ -10,0 +11,0 @@ return (fnStr === '[object Function]' || fnStr === '[object GeneratorFunction]') && isFnRegex.test(fnToStr.call(fn));

package.json

 {
 	"name": "is-generator-function",
-	"version": "1.0.2",
+	"version": "1.0.3",
 	"description": "Determine if a function is an ES6 generator function or not.",
 	"main": "index.js",
 	"scripts": {
-		"test": "npm run lint && node --harmony test/index.js",
+		"test": "npm run lint && node --es-staging --harmony test/index.js",
 		"coverage": "covert test/index.js",
@@ -34,3 +34,4 @@ "coverage-quiet": "covert test/index.js --quiet",
 		"covert": "1.0.0",
-		"jscs": "~1.10.0"
+		"jscs": "~1.10.0",
+		"semver": "~4.2.0"
 	},
@@ -37,0 +38,0 @@ "testling": {

test/index.js

@@ -6,2 +6,3 @@ 'use strict';
 var generatorFunc = require('make-generator-function');
+var hasToStringTag = typeof Symbol === 'function' && typeof Symbol.toStringTag === 'symbol';
 
@@ -59,2 +60,8 @@ var forEach = function (arr, func) {
 
+test('returns false for non-generator function with faked @@toStringTag', { skip: !hasToStringTag }, function (t) {
+	var fakeGenFunction = { valueOf: function () { return generatorFunc; }, toString: function () { return String(generatorFunc); } };
+	fakeGenFunction[Symbol.toStringTag] = 'GeneratorFunction';
+	t.notOk(isGeneratorFunction(fakeGenFunction), 'fake GeneratorFunction with @@toStringTag "GeneratorFunction" is not a generator function');
+	t.end();
+});
 
@@ -61,0 +68,0 @@ test('returns true for generator functions', function (t) {

No alert changes

Improved metrics

Total package byte prevSize

13549

increased by58.8%

Number of package files

11

increased by22.22%

Lines of code

113

increased by7.62%

Worsened metrics

Dev dependency count

5

increased by25%

No dependency changes