@raketa-cloud/admin 0.0.93 → 0.0.94

package/jest.config.js

@@ -11,8 +11,8 @@ const config = {
   ],
   coverageThreshold: {
     global: {
-      branches: 71,
-      functions: 74,
+      branches: 72,
+      functions: 75,
       lines: 78,
       statements: 78,
     },

package/package.json

@@ -1,10 +1,10 @@
 {
   "name": "@raketa-cloud/admin",
-  "version": "0.0.93",
+  "version": "0.0.94",
   "main": "src/index.js",
   "scripts": {
     "dev": "npx tailwindcss -i ./src/styles/application.css -o ../../apps/admin/static/application.css --watch",
-    "test": "jest --coverage --coverageReporters=text --coverageReporters=\"text-summary\""
+    "test": "jest --coverage --coverageReporters=html --coverageReporters=\"text-summary\""
   },
   "devDependencies": {
     "@babel/core": "7.24.7",

package/src/handlers/RedirectsHandler.js

@@ -102,7 +102,7 @@ const destroy = async (ctx) => {
   const { errors } = await ctx.client.redirects.delete(id);
 
   if (errors) {
-    throw `Failed to delete redirect #${id}. WTF?`;
+    throw `Failed to delete redirect #${id}.`;
   }
 
   return {

package/src/handlers/TagsHandler.js

@@ -92,7 +92,7 @@ const destroy = async (ctx) => {
   const { errors } = await ctx.client.tags.delete(id, ctx.locale);
 
   if (errors) {
-    throw `Failed to delete tag #${id}. WTF?`;
+    throw `Failed to delete tag #${id}.`;
   }
 
   return {

package/src/handlers/categories/index.js

@@ -106,7 +106,7 @@ const destroy = async (ctx) => {
 
   return {
     headers: {
-      "HX-redirect": `/${ctx.locale}/${ctx.resourceConfig.categoryTypePlural}`,
+      "HX-Redirect": `/${ctx.locale}/${ctx.resourceConfig.categoryTypePlural}`,
     },
     json: {},
   };

package/src/helpers/HTMLBuilder.js

@@ -31,7 +31,7 @@ export default class HTMLBuilder {
     }, "");
 
     if (children || nonVoid)
-      return `<${tag} ${htmlAttributes}>${children}</${tag}>`;
+      return `<${tag} ${htmlAttributes}>${children ? children : ""}</${tag}>`;
 
     return `<${tag} ${htmlAttributes} />`;
   }

package/src/setupAuthentication.js

@@ -5,13 +5,16 @@ import currentPath from "./currentPath.cjs";
 import AuthHelper from "./helpers/AuthHelper";
 // TODO: Maybe change the encryption mechanism?
 import simpleCipher from "./simpleCipher";
+import getConfig from "../../../apps/admin/src/getConfig";
 
 // Generate a random signing secret on every server restart
-const SIGNING_SECRET = randomBytes(64).toString("hex");
+// const SIGNING_SECRET = randomBytes(64).toString("hex");
 const DELIMITER = ",";
 
 function calculateHash(text) {
-  return createHmac("sha256", SIGNING_SECRET).update(text).digest("hex");
+  return createHmac("sha256", getConfig().secrets.signing_secret)
+    .update(text)
+    .digest("hex");
 }
 
 function obfuscateToken(token) {

package/src/simpleCipher.js

@@ -1,14 +1,8 @@
+import getConfig from "../../../apps/admin/src/getConfig";
+
 const ALPHABET =
   "ABCDEFGHIJKLMNOPQRSTUVWXYZabcdefghijklmnopqrstuvwxyz0123456789,.";
 
-function generateKey() {
-  const shuffled = ALPHABET.split("")
-    .sort(() => Math.random() - 0.5)
-    .join("");
-
-  return shuffled;
-}
-
 function generateEncryptionLookup(key) {
   const lookup = {};
 
@@ -29,22 +23,38 @@ function generateDecryptionLookup(key) {
   return lookup;
 }
 
+// function generateKey() {
+//   const shuffled = ALPHABET.split("")
+//     .sort(() => Math.random() - 0.5)
+//     .join("");
+
+//   return shuffled;
+// }
+
 // Scramble lookup maps on every server restart
-const CIPHER_KEY = generateKey();
-const ENCRYPT_LOOKUP = generateEncryptionLookup(CIPHER_KEY);
-const DECRYPT_LOOKUP = generateDecryptionLookup(CIPHER_KEY);
+// const CIPHER_KEY = generateKey();
+// const ENCRYPT_LOOKUP = generateEncryptionLookup(CIPHER_KEY);
+// const DECRYPT_LOOKUP = generateDecryptionLookup(CIPHER_KEY);
 
 function encrypt(plaintext) {
+  const ecryptLookup = generateEncryptionLookup(
+    getConfig().secrets.crypto_secret
+  );
+
   return plaintext
     .split("")
-    .map((char) => ENCRYPT_LOOKUP[char] || char)
+    .map((char) => ecryptLookup[char] || char)
     .join("");
 }
 
 function decrypt(ciphertext) {
+  const decryptLookup = generateDecryptionLookup(
+    getConfig().secrets.crypto_secret
+  );
+
   return ciphertext
     .split("")
-    .map((char) => DECRYPT_LOOKUP[char] || char)
+    .map((char) => decryptLookup[char] || char)
     .join("");
 }
 

package/src/tests/Logger.test.js

@@ -0,0 +1,33 @@
+import Logger from "../Logger";
+
+describe("Logger", () => {
+  describe("error", () => {
+    test("Logs an error to the console", () => {
+      const logSpy = jest.spyOn(global.console, "error");
+
+      Logger.error({ foo: "BAR" });
+
+      expect(logSpy).toHaveBeenCalledWith("[ERROR]", "{ foo: 'BAR' }");
+    });
+  });
+
+  describe("info", () => {
+    test("Logs info to the console", () => {
+      const logSpy = jest.spyOn(global.console, "info");
+
+      Logger.info({ foo: "BAR" });
+
+      expect(logSpy).toHaveBeenCalledWith("[INFO]", "{ foo: 'BAR' }");
+    });
+  });
+
+  describe("debug", () => {
+    test("Logs debug information to the console", () => {
+      const logSpy = jest.spyOn(global.console, "debug");
+
+      Logger.debug({ foo: "BAR" });
+
+      expect(logSpy).toHaveBeenCalledWith("[DEBUG]", "{ foo: 'BAR' }");
+    });
+  });
+});

package/src/tests/handlers/RedirectsHandler.test/destroy.test.js

@@ -45,7 +45,7 @@ describe("Redirects", () => {
       }));
 
     await expect(() => RedirectsHandler.destroy(ctx)).rejects.toMatch(
-      "Failed to delete redirect #1. WTF?"
+      "Failed to delete redirect #1."
     );
 
     expect(mockDelete).toHaveBeenCalledWith(1);

package/src/tests/handlers/TagsHandler.test/destroy.test.js

@@ -43,7 +43,7 @@ describe("Tags Handler", () => {
       }));
 
     expect(() => TagsHandler.destroy(ctx)).rejects.toMatch(
-      "Failed to delete tag #1. WTF?"
+      "Failed to delete tag #1."
     );
 
     expect(mockedDelete).toHaveBeenCalledWith("1", "en");

package/src/tests/handlers/categories/destroy.test.js

@@ -1,34 +1,38 @@
-import RedirectsHandler from "../../../handlers/RedirectsHandler";
-import Redirects from "../../../cloud/Redirects";
+import CategoriesHandler from "../../../handlers/categories";
+import Categories from "../../../cloud/Categories";
 
 import Factory from "../../support/factory";
 
 function buildCtx() {
   return {
     client: {
-      redirects: new Redirects(() => ({}), "token"),
+      categories: new Categories(() => ({}), "token"),
     },
     params: { id: 1 },
     locale: "en",
+    resourceConfig: {
+      categoryType: "cat",
+      categoryTypePlural: "cats",
+    },
   };
 }
 
-describe("Redirects", () => {
+describe("Categories", () => {
   test("destroy successful", async () => {
     const redirect = Factory.build("redirect");
     const ctx = buildCtx();
 
     const mockDelete = jest
-      .spyOn(ctx.client.redirects, "delete")
+      .spyOn(ctx.client.categories, "delete")
       .mockImplementation(() => ({ redirect: redirect, errors: null }));
 
-    const result = await RedirectsHandler.destroy(ctx);
+    const result = await CategoriesHandler.destroy(ctx);
 
-    expect(mockDelete).toHaveBeenCalledWith(1);
+    expect(mockDelete).toHaveBeenCalledWith(1, "en");
 
     expect(result).toEqual({
       headers: {
-        "HX-Redirect": "/en/redirects",
+        "HX-Redirect": "/en/cats",
       },
       json: {},
     });
@@ -38,16 +42,16 @@ describe("Redirects", () => {
     const ctx = buildCtx();
 
     const mockDelete = jest
-      .spyOn(ctx.client.redirects, "delete")
+      .spyOn(ctx.client.categories, "delete")
       .mockImplementation(() => ({
         redirect: null,
         errors: [{ key: "generalError", message: "Not found" }],
       }));
 
-    await expect(() => RedirectsHandler.destroy(ctx)).rejects.toMatch(
-      "Failed to delete redirect #1. WTF?"
+    await expect(() => CategoriesHandler.destroy(ctx)).rejects.toEqual(
+      "Failed to delete category #1."
     );
 
-    expect(mockDelete).toHaveBeenCalledWith(1);
+    expect(mockDelete).toHaveBeenCalledWith(1, "en");
   });
 });

package/src/tests/helpers/HTMLBuilder.test.js

@@ -0,0 +1,44 @@
+import HTMLBuilder from "../../helpers/HTMLBuilder";
+
+describe("HTMLBuilder", () => {
+  function normalizeHTML(html) {
+    return html.replace(/>\s+</g, "><").replace(/\s+/g, " ").trim();
+  }
+
+  describe("tag", () => {
+    test("renders a self closing tag", () => {
+      const builder = new HTMLBuilder();
+
+      const result = builder.tag("img", null, { src: "/some-image.jpg" });
+
+      expect(normalizeHTML(result)).toEqual(
+        normalizeHTML('<img src="/some-image.jpg" />')
+      );
+    });
+
+    test("renders a label tag", () => {
+      const builder = new HTMLBuilder();
+
+      const result = builder.tag("label", "Heelo", { for: "some-input-id" });
+
+      expect(normalizeHTML(result)).toEqual(
+        normalizeHTML('<label for="some-input-id">Heelo</label>')
+      );
+    });
+
+    test("renders a textarea tag", () => {
+      const builder = new HTMLBuilder();
+
+      const result = builder.tag(
+        "textarea",
+        undefined,
+        { id: "some-input-id", value: "Text?" },
+        { nonVoid: true }
+      );
+
+      expect(normalizeHTML(result)).toEqual(
+        normalizeHTML('<textarea id="some-input-id" value="Text?"></textarea>')
+      );
+    });
+  });
+});