@admc.com/eslintplugin-sn-test 3.8.0 → 3.9.0

package/bad1Scripts/all/sp_widget.client_script/namedFn.js

@@ -1,4 +1,4 @@
-function dummy($scope) {
+function _unused($scope) {
     "use strict";
     const fName = "spclient/" + $scope.widget.id + ":" + $scope.$id;
 	console.warn("UPDATE02");

package/bad1Scripts/all/sp_widget.link/backticks.js

@@ -0,0 +1,5 @@
+(scope) => {
+    "use strict";
+    const wid = scope.widget.id;
+	console.warn(`${wid} UPDATE02`);
+}

package/bad1Scripts/all/sys_ui_context_menu/backticks.js

@@ -0,0 +1,2 @@
+// eslint-disable-next-line max-len
+console.info(`The built-in variables are g_list ${g_list}, g_fieldName (${g_fieldName}), g_sysId ${g_sysId}`);

package/{bad1Scripts → bad2Scripts}/all/sp_widget.client_script/fnAssignment.js

@@ -1,5 +1,8 @@
-dummy = function($scope) {
+_unused = function($scope) {
+    /* I had to put this script under bad2Scripts instead of bad1Scripts to accommodate an
+     * ESLint bug where varsIgnorePatterns are sometimes not honored, including for "_unused". */
     "use strict";
+    /* global _unused:writable */
     const fName = "spclient/" + $scope.widget.id + ":" + $scope.$id;
 	console.warn("UPDATE02");
     console.warn("$scope", $scope);

package/bad2Scripts/all/sp_widget.client_script/ootbBoilerplate.js

@@ -0,0 +1,8 @@
+api.controller=function() {
+  // Have to satisfy "use strict" or would have to complicate eslintplugin-sn-test design to
+  // allow arbitrarily-specified number of failures, or at least add support for 3 failures.
+  // Doing this instead of eslint-disable notation since that notation would conflict with wrapping:
+  "use strict";
+  /* widget controller */
+  var c = this;
+};

package/goodScripts/all/sp_widget.link/cleanedBoilerplate.js

@@ -0,0 +1,4 @@
+function link(scope, element, attrs, controller) {  // eslint-disable-line no-unused-vars
+    "use strict";
+  
+}

package/goodScripts/all/sp_widget.link/good.js

@@ -0,0 +1,9 @@
+(scope) => {
+    "use strict";
+    const fName = "splink/" + scope.widget.id + ":" + scope.$id;
+	console.warn("UPDATE02");
+    console.warn("scope", scope);
+	console.info("Executing " + fName + " with this", this);
+	this.thisVar = "I am here";  // To facilitate checking for this exact instance
+	scope.pneu = "new string";
+}

package/goodScripts/all/sys_ui_context_menu/good.js

@@ -0,0 +1,2 @@
+console.info("The built-in variables are g_list " + g_list + ", g_fieldName ("
+  + g_fieldName + "), g_sysId " + g_sysId);

package/goodScripts/scoped-es12/sp_widget.script/arrow.js

@@ -0,0 +1,7 @@
+(() => {
+    "use strict";
+    if (input.datum) {
+        data.datum = input.datum;
+        gs.info("Set datum for client" + input.datum);
+    }
+})();

package/package.json

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

package/sneslintrc.json

@@ -29,7 +29,7 @@
   "overrides": [
     {  // ES6 overrides
       "files": [ "**/@(noniso|iso)/*.js",
-                 "**/@(sys_ui_script|sys_script_validator|sp_widget.client_script|sys_ux_data_broker_scriptlet)/all/*.js"],  // eslint-disable-line max-len
+                 "**/@(sys_ui_script|sys_script_validator|sp_widget.client_script|sys_ux_data_broker_scriptlet|sp_widget.link|sys_ui_context_menu)/all/*.js"],  // eslint-disable-line max-len
       "rules": {
         "strict": ["error", "function"],
         "prefer-arrow-callback": "error",
@@ -57,7 +57,7 @@
         // Reverting eslint-plugin-sn exceptions back, because we overrode them generally above
         // when raising warnings to errors.
         "files":
-          ["**/@(sys_ui_script|sp_widget.script|sp_widget.client_script|sp_widget.link)/*/*.js"],
+          ["**/@(sys_ui_script|sp_widget.script|sp_widget.client_script|sp_widget.link|sp_widget.link|sys_ui_context_menu)/*/*.js"],
         "rules": {
             "prefer-template": "off",
             "no-template-curly-in-string": "off"