@admc.com/eslintplugin-sn-test 3.1.0 → 3.1.1

package/{badScripts → bad1Scripts}/scoped-es12/sys_script/changeConst.js

@@ -1,6 +1,6 @@
+(function() {
 "use strict";
 
-(function() {
 const dbl = v => v+v;
 const x = 3;
 x++;

package/bad1Scripts/scoped-es12/sys_script/noStrict.js

@@ -0,0 +1,6 @@
+(function() {
+const dbl = v => v+v;
+let x = 3;
+x++;
+gs.info(`double x is '${dbl(x)}'`);
+})(previous, current);

package/bad1Scripts/scoped-es12/sys_script_include/noStrict.js

@@ -0,0 +1,4 @@
+const dbl = v => v+v;
+let x = 3;
+x++;
+gs.info(`double x is '${dbl(x)}'`);

package/bad2Scripts/scoped-es12/sys_script/globalStrict.js

@@ -0,0 +1,8 @@
+"use strict";
+
+(function() {
+const dbl = v => v+v;
+let x = 3;
+x++;
+gs.info(`double x is '${dbl(x)}'`);
+})(previous, current);

package/bad2Scripts/scoped-es12/sys_script_include/fnStrict.js

@@ -0,0 +1,5 @@
+function doubler(x) {
+    "use strict";
+    return x + x;
+}
+gs.info(doubler(3));

package/goodScripts/scoped-es12/sys_script/good.js

@@ -1,6 +1,5 @@
-"use strict";
-
 (function() {
+"use strict";
 const dbl = v => v+v;
 let x = 3;
 x++;

package/goodScripts/scoped-es12/sys_script_include/good.js

@@ -0,0 +1,6 @@
+"use strict";
+
+const dbl = v => v+v;
+let x = 3;
+x++;
+gs.info(`double x is '${dbl(x)}'`);

package/masterTest.js

@@ -5,18 +5,21 @@ if (!__filename.startsWith(process.cwd() + "/"))  // eslint-disable-line prefer-
 const relTestPath = __filename.substring(process.cwd().length + 1);
 const path = require("path");
 const fs = require("fs");
-const ex =
-  /^test[/](good|bad|reallybad)[/](?:([\w-]+)[/])?([\w.-]+[/][\w-]+)-test[.]js$/.exec(relTestPath);
+const ex =  // eslint-disable-next-line max-len
+  /^test[/](good|bad1|bad2|badSys)[/](?:([\w-]+)[/])?([\w.-]+[/][\w-]+)-test[.]js$/.exec(relTestPath);
 if (!ex) throw new Error(`Bad test file path: ${relTestPath}`);
 let expectExitVal;
 switch (ex[1]) {
     case "good":
         expectExitVal = 0;
         break;
-    case "bad":
+    case "bad1":
         expectExitVal = 1;
         break;
-    case "reallybad":
+    case "bad2":
+        expectExitVal = 2;
+        break;
+    case "badSys":
         expectExitVal = 253;
         break;
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@admc.com/eslintplugin-sn-test",
-  "version": "3.1.0",
+  "version": "3.1.1",
   "description": "External tests for ESLint plugin @admc.com/eslint-plugin-sn",
   "scripts": {
     "test": "mocha --recursive",

package/populateTest.bash

@@ -50,15 +50,16 @@ Abort() {
 TMPFILE="$TMPDIR/${PROGNAME#.*}.$$"
 trap 'rm -f "$TMPFILE"' EXIT
 [ -d goodScripts ] || Abort "Directory 'goodScripts' is missing"
-[ -d badScripts ] || Abort "Directory 'badScripts' is missing"
-[ -d reallybadScripts ] || Abort "Directory 'badScripts' is missing"
-find goodScripts badScripts reallybadScripts -name '*.js' > "$TMPFILE" ||
+[ -d bad1Scripts ] || Abort "Directory 'bad1Scripts' is missing"
+[ -d bad2Scripts ] || Abort "Directory 'bad2Scripts' is missing"
+[ -d badSysScripts ] || Abort "Directory 'badScripts' is missing"
+find goodScripts bad1Scripts bad2Scripts badSysScripts -name '*.js' > "$TMPFILE" ||
 Abort 'find command failed'
-[ -n "$REPLACE_MODE" ] && rm -rf test/good test/bad test/reallybad
+[ -n "$REPLACE_MODE" ] && rm -rf test/good test/bad1 test/bad2 test/badSys
 while read SCRIPT_PATH; do
     GB=
     case "$SCRIPT_PATH" in
-        badScripts/*|goodScripts/*|reallybadScripts/*) ;;
+        bad1Scripts/*|bad2Scripts/*|goodScripts/*|badSysScripts/*) ;;
         *) Abort "Unexpected SCRIPT_PATH prefix: $SCRIPT_PATH";;
     esac
     GB="${SCRIPT_PATH%%Scripts*}"

package/sneslintrc.json

@@ -29,11 +29,11 @@
     {
       "files": ["**/sys_ui_script/*/*.js"],
       "rules": { "@admc.com/sn/no-uiscript-curlyref": "error" }
-    },{ // ES6 overrides
+    }, { // ES6 overrides
       "files": [ "**/@(noniso|iso)/*.js",
                  "**/@(sys_ui_script|sys_script_validator|sp_widget.client_script)/all/*.js"],
       "rules": {
-        "strict": ["error", "safe"],
+        "strict": ["error", "function"],
         "prefer-arrow-callback": "error",
         "object-shorthand": "error",
         "no-useless-rename": "error",
@@ -41,6 +41,22 @@
         "prefer-template": "error",
         "@admc.com/sn/no-client-gr": "error"
       }
+    }, {  // ES 12 overrides
+      "files": [ "**/scoped-es12/*.js" ],
+      "rules": {
+        "strict": ["error", "global"],
+        "prefer-arrow-callback": "error",
+        "object-shorthand": "error",
+        "no-useless-rename": "error",
+        "no-var": "error",
+        "prefer-template": "error"
+      }
+    }, {  // Overridding for ES12 IIFE scriptlets}
+      "files": [
+        "**/@(sys_script|sys_ws_operation|sys_web_service|sys_processor|sys_script_email|sys_transform_map|sys_transform_script|sp_widget.script|sys_ui_page.processing_script)/scoped-es12/*.js",  // eslint-disable-line max-len
+        "**/sys_ui_action/@(iso|noniso)_scoped-es12/*.js"
+      ],
+      "rules": { "strict": ["error", "function"] }
     }, { // Custom table and scope
       "files": [ "**/custom_table/custom_alt/*.js" ],
       "env": {"@admc.com/sn/sn_server_global": true },