@moreapp/common-nodejs 0.7.6 → 0.7.10

package/dist/utils.js

@@ -34,7 +34,7 @@ function currentTraceId() {
 exports.currentTraceId = currentTraceId;
 const emailAddressMaxLength = 254; // https://stackoverflow.com/a/574698
 const emailAddressLocalPattern = /^.+$/;
-const emailAddressDomainPattern = /^(?:[a-z0-9](?:[a-z0-9-]{0,61}[a-z0-9])?\.)+[a-z0-9][a-z0-9-]{0,61}[a-z0-9]$/;
+const emailAddressDomainPattern = /^(?:[A-z0-9À-ž](?:[A-z0-9À-ž-]{0,61}[A-z0-9À-ž])?\.)+[A-z0-9À-ž][A-z0-9À-ž-]{0,61}[A-z0-9]$/;
 // There are many regex patterns to validate email addresses, but they all have flaws and cannot 100% correctly verify
 // every possible email address. For us, it's more important to avoid the majority of typo's instead of catching every
 // invalid email address. We let our email provider handle and false positives (either by their validation or the fact

package/dist/utils.test.js

@@ -43,18 +43,19 @@ describe("isValidEmail", () => {
         test("Subdomain", () => expect((0, utils_1.isValidEmail)("a@sub.domain.dev")).toBeTruthy());
         test("Subdomain with a dash", () => expect((0, utils_1.isValidEmail)("a@sub.dom-ain.dev")).toBeTruthy());
         test("Long TLD", () => expect((0, utils_1.isValidEmail)("a@b.business")).toBeTruthy());
-        test("Diacritics", () => expect((0, utils_1.isValidEmail)("ẞçäöü@moreapp.dev")).toBeTruthy());
+        test("Diacritics", () => expect((0, utils_1.isValidEmail)("ẞçäöü@möręæppß.dev")).toBeTruthy());
+        test("Casing", () => expect((0, utils_1.isValidEmail)("TØmÂs@mŒREÄPp.cOm")).toBeTruthy());
     });
     describe("Invalid email addresses", () => {
         test("Missing '@' character and domain", () => expect((0, utils_1.isValidEmail)("john")).toBeFalsy());
-        test("// Multiple '@' characters", () => expect((0, utils_1.isValidEmail)("a@b@moreapp.dev")).toBeFalsy());
-        test("// Empty", () => expect((0, utils_1.isValidEmail)("")).toBeFalsy());
-        test("// Whitespace before/after", () => expect((0, utils_1.isValidEmail)(" a@moreapp.dev ")).toBeFalsy());
-        test("// Newlines, tabs, ..., before/after", () => expect((0, utils_1.isValidEmail)("\n \ta@moreapp.dev\n ")).toBeFalsy());
-        test("// Missing '@' character", () => expect((0, utils_1.isValidEmail)("ab.com")).toBeFalsy());
-        test("// Double dot in domain", () => expect((0, utils_1.isValidEmail)("a@moreapp..com")).toBeFalsy());
-        test("// Too short TLD", () => expect((0, utils_1.isValidEmail)("a@b.c")).toBeFalsy());
-        test("// Missing local part", () => expect((0, utils_1.isValidEmail)("@moreapp.dev")).toBeFalsy());
-        test("// Too long", () => expect((0, utils_1.isValidEmail)("abcdefghijklmnopqrstuvwxyz-abcdefghijklmnopqrstuvwxyz-abcdefghijklmnopqrstuvwxyz-abcdefghijklmnopqrstuvwxyz-abcdefghijklmnopqrstuvwxyz-abcdefghijklmnopqrstuvwxyz-abcdefghijklmnopqrstuvwxyz-abcdefghijklmnopqrstuvwxyz-abcdefghijklmnopqrstuvwxyz-@moreapp.dev")).toBeFalsy());
+        test("Multiple '@' characters", () => expect((0, utils_1.isValidEmail)("a@b@moreapp.dev")).toBeFalsy());
+        test("Empty", () => expect((0, utils_1.isValidEmail)("")).toBeFalsy());
+        test("Whitespace before/after", () => expect((0, utils_1.isValidEmail)(" a@moreapp.dev ")).toBeFalsy());
+        test("Newlines, tabs, ..., before/after", () => expect((0, utils_1.isValidEmail)("\n \ta@moreapp.dev\n ")).toBeFalsy());
+        test("Missing '@' character", () => expect((0, utils_1.isValidEmail)("ab.com")).toBeFalsy());
+        test("Double dot in domain", () => expect((0, utils_1.isValidEmail)("a@moreapp..com")).toBeFalsy());
+        test("Too short TLD", () => expect((0, utils_1.isValidEmail)("a@b.c")).toBeFalsy());
+        test("Missing local part", () => expect((0, utils_1.isValidEmail)("@moreapp.dev")).toBeFalsy());
+        test("Too long", () => expect((0, utils_1.isValidEmail)("abcdefghijklmnopqrstuvwxyz-abcdefghijklmnopqrstuvwxyz-abcdefghijklmnopqrstuvwxyz-abcdefghijklmnopqrstuvwxyz-abcdefghijklmnopqrstuvwxyz-abcdefghijklmnopqrstuvwxyz-abcdefghijklmnopqrstuvwxyz-abcdefghijklmnopqrstuvwxyz-abcdefghijklmnopqrstuvwxyz-@moreapp.dev")).toBeFalsy());
     });
 });

package/package.json

@@ -1,12 +1,13 @@
 {
   "name": "@moreapp/common-nodejs",
-  "version": "0.7.6",
+  "version": "0.7.10",
   "license": "UNLICENSED",
   "main": "dist/index.js",
   "types": "dist/index.d.ts",
   "scripts": {
     "build": "tsc",
-    "test": "jest",
+    "test": "jest --detectOpenHandles",
+    "test:workflow": "yarn test --coverage",
     "prettier": "prettier './**/*.{ts,json,yaml,md,}'",
     "format:check": "yarn prettier --check",
     "format:fix": "yarn prettier --write",
@@ -49,7 +50,6 @@
     "husky": "8.0.3",
     "jest": "29.6.4",
     "jest-mock-extended": "3.0.5",
-    "jest-sonar-reporter": "2.0.0",
     "lint-staged": "13.3.0",
     "nock": "13.3.8",
     "prettier": "2.8.8",