not-node 5.1.4 → 5.1.7

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "not-node",
-  "version": "5.1.4",
+  "version": "5.1.7",
   "description": "node complimentary part for client side notFramework.",
   "main": "index.js",
   "scripts": {
@@ -40,13 +40,13 @@
     "cors": "^2.8.5",
     "deep-diff": "*",
     "deepmerge": "^4.2.2",
-    "ejs": "^3.1.6",
+    "ejs": "^3.1.8",
     "escape-string-regexp": "*",
-    "express": "^4.17.3",
-    "express-fileupload": "^1.3.1",
-    "express-session": "^1.17.2",
+    "express": "^4.18.1",
+    "express-fileupload": "^1.4.0",
+    "express-session": "^1.17.3",
     "fs-extra": "*",
-    "helmet": "^5.0.2",
+    "helmet": "^5.1.0",
     "lower-case": "*",
     "method-override": "^3.0.0",
     "mock-require": "^3.0.3",
@@ -61,9 +61,9 @@
     "not-log": "^0.0.20",
     "not-monitor": "^0.0.13",
     "not-path": "*",
-    "not-validation": "^0.0.7",
-    "rate-limiter-flexible": "^2.3.6",
-    "redis": "^4.0.6",
+    "not-validation": "^0.0.8",
+    "rate-limiter-flexible": "^2.3.7",
+    "redis": "^4.1.1",
     "rfdc": "^1.3.0",
     "rmdir": "^1.2.0",
     "serve-static": "*",
@@ -75,18 +75,18 @@
     "babel-eslint": "^10.1.0",
     "chai": "*",
     "chai-as-promised": "*",
-    "eslint": "^8.12.0",
+    "eslint": "^8.18.0",
     "eslint-plugin-node": "^11.1.0",
     "eslint-plugin-sonarjs": "^0.13.0",
     "ink-docstrap": "^1.3.2",
-    "ioredis": "^5.0.3",
+    "ioredis": "^5.1.0",
     "jsdoc": "^3.6.10",
     "mocha": "*",
     "mocha-suppress-logs": "^0.3.1",
-    "mongodb-memory-server": "^8.4.2",
+    "mongodb-memory-server": "^8.7.2",
     "npm-run-all": "^4.1.5",
     "nyc": "^15.1.0",
-    "retire": "^3.0.6"
+    "retire": "^3.0.7"
   },
   "homepage": "https://github.com/interrupter/not-node#readme",
   "nyc": {

package/src/auth/exceptions.js

@@ -0,0 +1,27 @@
+const { notError } = require("not-error");
+
+class RolesExceptionRoleSetIsNotValid extends notError {
+    constructor(name) {
+        super("not-node:auth_roles_role_set_is_not_valid", { name });
+    }
+}
+module.exports.RolesExceptionRoleSetIsNotValid =
+    RolesExceptionRoleSetIsNotValid;
+
+class RolesExceptionNoRolesSupremacyOrder extends notError {
+    constructor() {
+        super("not-node:auth_roles_no_roles_supremacy_order");
+    }
+}
+
+module.exports.RolesExceptionNoRolesSupremacyOrder =
+    RolesExceptionNoRolesSupremacyOrder;
+
+class RolesExceptionSupremacyOrderElementIsNotAString extends notError {
+    constructor() {
+        super("not-node:auth_roles_supremacy_order_element_is_not_a_string");
+    }
+}
+
+module.exports.RolesExceptionSupremacyOrderElementIsNotAString =
+    RolesExceptionSupremacyOrderElementIsNotAString;

package/src/auth/roles.js

@@ -1,4 +1,9 @@
 const ABSTRACT = require("./abstract");
+const {
+    RolesExceptionRoleSetIsNotValid,
+    RolesExceptionNoRolesSupremacyOrder,
+    RolesExceptionSupremacyOrderElementIsNotAString,
+} = require("./exceptions.js");
 
 function compareRolesArrayAgainstArray(userRoles, actionRoles, strict) {
     if (strict) {
@@ -48,7 +53,7 @@ function compareRoles(userRoles, actionRoles, strict = true) {
 
 function sanitizeAndValidateRoleSet(roleSet, name) {
     if (!Array.isArray(roleSet) && !ABSTRACT.isObjectString(roleSet)) {
-        throw new Error(`${name} role set is not valid`);
+        throw new RolesExceptionRoleSetIsNotValid(name);
     } else {
         if (!Array.isArray(roleSet)) {
             roleSet = [roleSet];
@@ -70,14 +75,14 @@ function checkSupremacy(base, against, roles) {
     against = sanitizeAndValidateRoleSet(against, "Against");
 
     if (!Array.isArray(roles)) {
-        throw new Error("No roles supremacy order!");
+        throw new RolesExceptionNoRolesSupremacyOrder();
     }
 
     let baseIndex = -1;
     let againstIndex = -1;
     roles.forEach((role, index) => {
         if (!ABSTRACT.isObjectString(role)) {
-            throw new Error("Supremacy order element is not a string");
+            throw new RolesExceptionSupremacyOrderElementIsNotAString();
         }
         if (baseIndex === -1 && base.indexOf(role) > -1) {
             baseIndex = index;

package/src/generic/logic.js

@@ -125,7 +125,7 @@ module.exports = ({
             });
         }
 
-        async _updateOne({
+        static async _updateOne({
             targetId,
             data,
             activeUser,
@@ -146,7 +146,7 @@ module.exports = ({
             if (shouldOwn) {
                 query[ownerFieldName] = activeUser._id;
             }
-            const result = await getModel().update(query, data).exec();
+            const result = await getModel().update(query, data);
             LogAction(
                 {
                     action,
@@ -222,7 +222,7 @@ module.exports = ({
          * @param {boolean}  prepared.shouldOwn     if user should be owner of target
          * @returns {Promise<Object>}               requested document
          **/
-        async _getOne({
+        static async _getOne({
             targetId,
             action,
             ip,
@@ -312,7 +312,7 @@ module.exports = ({
          * @param {boolean}  prepared.shouldOwn     if user should be owner of target
          * @returns {Promise<Object>}               requested document
          **/
-        async _getOneRaw({
+        static async _getOneRaw({
             targetId,
             activeUser,
             ip,

package/src/model/exceptions.js

@@ -0,0 +1,40 @@
+const { notError } = require("not-error");
+
+class VersioningExceptionSameOldData extends notError {
+    constructor() {
+        super("not-node:versioning_error_same_old_data");
+    }
+}
+module.exports.VersioningExceptionSameOldData = VersioningExceptionSameOldData;
+
+class VersioningExceptionNoPpreviousVersions extends notError {
+    constructor() {
+        super("not-node:versioning_error_no_previous_versions");
+    }
+}
+module.exports.VersioningExceptionNoPpreviousVersions =
+    VersioningExceptionNoPpreviousVersions;
+
+class IncrementExceptionIDGeneratorRebaseFailed extends notError {
+    constructor() {
+        super("not-node:increment_id_generator_rebase_failed");
+    }
+}
+module.exports.IncrementExceptionIDGeneratorRebaseFailed =
+    IncrementExceptionIDGeneratorRebaseFailed;
+
+class IncrementExceptionIDGenerationFailed extends notError {
+    constructor() {
+        super("not-node:increment_id_generation_failed");
+    }
+}
+module.exports.IncrementExceptionIDGenerationFailed =
+    IncrementExceptionIDGenerationFailed;
+
+class IncrementExceptionFieldsNotExistInDataObject extends notError {
+    constructor(miss) {
+        super("not-node:increment_fields_not_exist_in_data_object", { miss });
+    }
+}
+module.exports.IncrementExceptionFieldsNotExistInDataObject =
+    IncrementExceptionFieldsNotExistInDataObject;

package/src/model/increment.js

@@ -1,6 +1,11 @@
 /** @module Model/Increment */
 
 const { updateResponseSuccess } = require("./utils.js");
+const {
+    IncrementExceptionIDGeneratorRebaseFailed,
+    IncrementExceptionIDGenerationFailed,
+    IncrementExceptionFieldsNotExistInDataObject,
+} = require("./exceptions.js");
 
 const thisSchema = {
     id: {
@@ -84,9 +89,7 @@ function newGetNext() {
             if (miss.length === 0) {
                 id = formId(modelName, filterFields, data);
             } else {
-                throw new Error(
-                    "Fields not exist in data object: " + miss.join(", ")
-                );
+                throw new IncrementExceptionFieldsNotExistInDataObject(miss);
             }
         }
         let which = {
@@ -106,7 +109,7 @@ function newGetNext() {
             const doc = await thisModel.findOne({ id });
             return doc.seq;
         } else {
-            throw new Error("ID generation failed");
+            throw new IncrementExceptionIDGenerationFailed();
         }
     };
 }
@@ -136,7 +139,7 @@ function newRebase() {
         if (updateResponseSuccess(res)) {
             return ID;
         } else {
-            throw new Error("ID generator rebase failed");
+            throw new IncrementExceptionIDGeneratorRebaseFailed();
         }
     };
 }

package/src/model/versioning.js

@@ -11,8 +11,10 @@ const TECH_FIELDS = [
     "__closed",
 ];
 
-const ERR_NO_PREVIOUS_VERSIONS = "No previous version!";
-const ERR_SAME_OLD_DATA = "Same old data";
+const {
+    VersioningExceptioNoPpreviousVersions,
+    VersioningExceptionSameOldData,
+} = require("./exceptions.js");
 
 class ModelVersioning {
     static versionsHistoryExists(data) {
@@ -72,7 +74,7 @@ class ModelVersioning {
                 previous.toObject()
             );
         } else {
-            throw new Error(ERR_NO_PREVIOUS_VERSIONS);
+            throw new VersioningExceptioNoPpreviousVersions();
         }
     }
 
@@ -103,7 +105,7 @@ class ModelVersioning {
                 versionDoc
             );
         }
-        throw new Error(ERR_SAME_OLD_DATA);
+        throw new VersioningExceptionSameOldData();
     }
 
     /**
@@ -162,7 +164,4 @@ class ModelVersioning {
     }
 }
 
-ModelVersioning.ERR_NO_PREVIOUS_VERSIONS = ERR_NO_PREVIOUS_VERSIONS;
-ModelVersioning.ERR_SAME_OLD_DATA = ERR_SAME_OLD_DATA;
-
 module.exports = ModelVersioning;