@admc.com/eslintplugin-sn-test 1.0.2 → 1.0.3

package/badScripts/global/sys_script/initToUndefined.js

@@ -1,2 +1,4 @@
+(function() {
 var unusedVar = undefined;
 if (!unusedVar) gs.log("notset", "2nd");
+})(previous, current);

package/badScripts/global/sys_script/noIifeParams.js

@@ -0,0 +1,3 @@
+(function() {
+    console.log("This will fail", "blaine");
+}();  // Invocation params here is what is constrained

package/badScripts/global/sys_script/noiife.js

@@ -0,0 +1 @@
+console.log("here without IIFE", "blaine");

package/badScripts/global/sys_script/unusedVar.js

@@ -1 +1,3 @@
+(function() {
 var unusedVar = 3;
+})(previous, current);

package/badScripts/global/sys_script/wrongIifeParams.js

@@ -0,0 +1,3 @@
+(function() {
+    console.log("This will fail", "blaine");
+}(alpha, beta);  // Invocation params here is what is constrained

package/badScripts/global/sysauto_script/badIife.js

@@ -0,0 +1,2 @@
+function fn() { gs.log("I do nothing", "src"); }
+fn();

package/badScripts/global/sysauto_script/grNoValidate.js

@@ -0,0 +1,4 @@
+const gr = new GlideRecord("sys_script_fix");
+gr.setLimit(1);
+gr.next();
+gs.log("Got sys_script_fix " + gr.sys_id, "src");

package/badScripts/global/sysauto_script/traditionalFor.js

@@ -0,0 +1,3 @@
+const ar = ["alpha", "beta", "gamma"];
+var i = 0;
+for (i = 0; i < ar.length; i++) gs.log("#" + (i+1) + " = " + ar[i], "src");

package/badScripts/scoped/sys_script/let.js

@@ -1,2 +1,4 @@
+(function() {
 let x = 3;
 gs.log(3, 'src');
+})(previous, current);

package/goodScripts/ecc_agent_script/good.js

@@ -1,4 +1,4 @@
-var TstCls = Class.create();
+const TstCls = Class.create();
 TstCls.prototype.initialize = function(o) {
     this.o = o;
     this.p = String(Packages.java.lang.System.getProperty(o));

package/goodScripts/global/sys_script/good.js

@@ -1,3 +1,4 @@
+(function() {
 // Test availability of global vars from the different component lists:
 gs.log("a message", "a source");  // coreServerObjects.txt
 gs.log(SNC.CMDBUtil.getTables0("sys_db_object"), "src");  // directly in exports.js
@@ -9,3 +10,4 @@ gs.log(varsAreAllowed, "2nd");
 const con = "a val";
 con += "addition";  // Can't test for const changes
 gs.log(con, "2nd");
+})(previous, current);

package/goodScripts/global/sys_script_include/createOtherClass.js

@@ -1,7 +1,7 @@
 // This passes test because we assign to the new class.
 // This is how SI classes are generally instantiated on SN, because this
 // is how the auto-generated code templates do it.
-var OtherAjaxProcessor = Class.create();
+const OtherAjaxProcessor = Class.create();
  
 OtherAjaxProcessor.prototype = Object.extendsObject(AbstractAjaxProcessor, {
     type: "OtherAjaxProcessor",

package/goodScripts/scoped/sys_script/good.js

@@ -1,3 +1,4 @@
+(function() {
 // Test availability of scoped vars from the different component lists:
 gs.info(new sn_ws_err.NotAcceptableError("fake err"));  // coreServerObjects.txt
 gs.info(Object.keys(new sn_codesearch.CodeSearch()).length);  // SIScopes
@@ -8,3 +9,4 @@ gs.info(varsAreAllowed);
 const con = "a val";
 con += "addition";  // Can't test for const changes
 gs.info(con);
+})(previous, current);

package/package.json

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

package/sneslintrc.json

@@ -21,7 +21,8 @@
     "no-template-curly-in-string": "error",
     "eqeqeq": "error",
     "semi": "error",
-    "no-extra-parens": "error"
+    "no-extra-parens": "error",
+    "@admc.com/sn/prefer-array-iterator": "error"
   },
   "overrides": [
     {