eslint-plugin-webgl-logic - npm Package Compare versions

Comparing version 1.4.0 to 1.5.0

flavio.json

 {
   "name": "eslint-plugin-webgl-logic",
-  "version": "1.4.0",
+  "version": "1.5.0",
   "tag": {
-    "commit": "6009f12c165b16b4512f6c24ea5dbc842ae9cb80",
+    "commit": "5668ed1b2a1100efe557045614535379c4d76cd4",
     "target": {
@@ -7,0 +7,0 @@ "branch": "master"

index.js

@@ -6,5 +6,7 @@ 'use strict';
 var noPromiseAll = require( './rules/noPromiseAll.js' );
+var noPromiseConstructor = require( './rules/noPromiseConstructor.js' );
 var noPromiseRace = require( './rules/noPromiseRace.js' );
 var noSetTimeout = require( './rules/noSetTimeout.js' );
 var maxLinesPerFunction = require( './rules/maxLinesPerFunction.js' );
+var enforceGameImports = require( './rules/enforceGameImports.js' );
 
@@ -16,5 +18,7 @@ module.exports = {
 		"noPromiseAll": noPromiseAll,
+		"noPromiseConstructor": noPromiseConstructor,
 		"noPromiseRace": noPromiseRace,
 		"noSetTimeout": noSetTimeout,
-		"maxLinesPerFunction": maxLinesPerFunction
+		"maxLinesPerFunction": maxLinesPerFunction,
+		"enforceGameImports": enforceGameImports
 	},
@@ -25,7 +29,9 @@ rulesConfig: {
 		"noPromiseAll": 2,
+		"noPromiseConstructor": 2,
 		"noPromiseRace": 2,
 		"noSetTimeout": 2,
-		"maxLinesPerFunction": 1
+		"maxLinesPerFunction": 1,
+		"enforceGameImports": 2
 	}
 };
 

package.json

 {
   "name": "eslint-plugin-webgl-logic",
-  "version": "1.4.0",
+  "version": "1.5.0",
   "description": "Custom eslint rules for the webgl logic module",
   "main": "index.js",
   "scripts": {
-    "test": "node test/test.js && node test/maxLinesPerFunction.js"
+    "test": "node test/test.js && node test/maxLinesPerFunction.js && node test/enforceGameImports.js"
   },
@@ -9,0 +9,0 @@ "repository": {

README.md

@@ -0,0 +0,0 @@ # eslint-plugin-webgl-logic

rules/noPromiseAll.js

@@ -0,0 +0,0 @@ 'use strict';

rules/noPromiseRace.js

@@ -0,0 +0,0 @@ 'use strict';

test/test.js

@@ -7,2 +7,3 @@ "use strict";
 var noPromiseRace = require( "../rules/noPromiseRace.js" );
+var noPromiseConstructor = require( "../rules/noPromiseConstructor.js" );
 var noSetTimeout = require( "../rules/noSetTimeout.js" );
@@ -121,2 +122,32 @@ var RuleTester = require( "eslint" ).RuleTester;
 
+ruleTester.run( "noPromiseConstructor", noPromiseConstructor, {
+	valid: [
+		"function prom() { return Promise.resolve(); }",
+		"function prom() { return Promise.reject(); }",
+		"Promise.someOtherProperty = {};"
+	],
+
+	invalid: [
+		{
+			code: "var val = new Promise( function( res, rej ) { return 0; } );",
+			errors: [{ message: "Promise constructor is forbidden" }]
+		},
+		{
+			code: "var val = new Promise( ( res, rej ) => { return 0; } );",
+			parser: 'babel-eslint',
+			errors: [{ message: "Promise constructor is forbidden" }]
+		},
+		{
+			code: "var val = new global.Promise( ( res, rej ) => { return 0; } );",
+			parser: 'babel-eslint',
+			errors: [{ message: "Promise constructor is forbidden" }]
+		},
+		{
+			code: "var val = new window.Promise( ( res, rej ) => { return 0; } );",
+			parser: 'babel-eslint',
+			errors: [{ message: "Promise constructor is forbidden" }]
+		}
+	]
+} );
+
 ruleTester.run( "noPromiseRace", noPromiseRace, {
@@ -123,0 +154,0 @@ valid: [

New alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Fixed alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Improved metrics

Total package byte prevSize

87557

increased by25.91%

Number of package files

17

increased by21.43%

Lines of code

2234

increased by23.7%

No dependency changes