@raketa-cloud/admin 0.0.33 → 0.0.36

package/src/tests/handlers/page_revisions/LocksHandler.test/create_form.test.js

@@ -0,0 +1,55 @@
+import Factory from "../../../support/factory";
+import PageRevisions from "../../../../cloud/PageRevisions";
+import LocksHandler from "../../../../handlers/page_revisions/LocksHandler";
+
+function buildCtx(revision) {
+  return {
+    client: {
+      pageRevisions: new PageRevisions(() => ({}), "token"),
+    },
+    params: { parent_id: revision.id },
+    locale: "en",
+  };
+}
+
+describe("New page revision lock", () => {
+  test("create_form for locking page revision", async () => {
+    const revision = Factory.build("page_revision");
+    const ctx = buildCtx(revision);
+
+    const mockFind = jest
+      .spyOn(ctx.client.pageRevisions, "find")
+      .mockImplementation(() => revision);
+
+    const result = await LocksHandler.create_form(ctx);
+
+    expect(mockFind).toHaveBeenCalledWith(revision.id, "en");
+
+    expect(result).toEqual({
+      assigns: {
+        resource: {
+          id: revision.id,
+          originalId: revision.originalId,
+          message: "",
+        },
+        errors: {},
+        _translations: [],
+      },
+    });
+  });
+
+  test("create_form for non existing page revision", async () => {
+    const nonExistentRevision = Factory.build("page_revision");
+    const ctx = buildCtx(nonExistentRevision);
+
+    const mockFind = jest
+      .spyOn(ctx.client.pageRevisions, "find")
+      .mockImplementation(() => null);
+
+    await expect(LocksHandler.create_form(ctx)).rejects.toThrow(
+      `Missing revision #${nonExistentRevision.id}`
+    );
+
+    expect(mockFind).toHaveBeenCalledWith(nonExistentRevision.id, "en");
+  });
+});

package/src/tests/handlers/page_revisions/UnlocksHandler.test/create.test.js

@@ -0,0 +1,80 @@
+import Factory from "../../../support/factory";
+import PageRevisions from "../../../../cloud/PageRevisions";
+import UnlocksHandler from "../../../../handlers/page_revisions/UnlocksHandler";
+
+function buildCtx(revision) {
+  return {
+    client: {
+      pageRevisions: new PageRevisions(() => ({}), "token"),
+    },
+    params: { parent_id: revision.id },
+    locale: "en",
+    resourceConfig: {
+      pageType: "page",
+      pageTypePlural: "pages",
+    },
+  };
+}
+
+describe("Page revision unlock", () => {
+  test("unlock successful", async () => {
+    const revision = Factory.build("page_revision");
+    const ctx = buildCtx(revision);
+
+    const mockUnlock = jest
+      .spyOn(ctx.client.pageRevisions, "unlock")
+      .mockImplementation(() => ({
+        revision: revision,
+        errors: null,
+        pageErrors: null,
+      }));
+
+    const result = await UnlocksHandler.create(ctx);
+
+    expect(mockUnlock).toHaveBeenCalledWith(revision.id, "en");
+
+    expect(result).toEqual({
+      redirect: `/en/pages/${revision.originalId}`,
+    });
+  });
+
+  test("unlock failed with errors", async () => {
+    const revision = Factory.build("page_revision");
+    const ctx = buildCtx(revision);
+    const errors = [
+      { key: "generalError", message: "Failed to unlock revision" },
+    ];
+
+    const mockUnlock = jest
+      .spyOn(ctx.client.pageRevisions, "unlock")
+      .mockImplementation(() => ({ revision: null, errors, pageErrors: null }));
+
+    await expect(UnlocksHandler.create(ctx)).rejects.toThrow(
+      `Failed to unlock revision: ${JSON.stringify(errors)}`
+    );
+
+    expect(mockUnlock).toHaveBeenCalledWith(revision.id, "en");
+  });
+
+  test("unlock failed with page errors", async () => {
+    const revision = Factory.build("page_revision");
+    const ctx = buildCtx(revision);
+    const pageErrors = [
+      { key: "generalError", message: "Failed to start revision" },
+    ];
+
+    const mockUnlock = jest
+      .spyOn(ctx.client.pageRevisions, "unlock")
+      .mockImplementation(() => ({
+        revision: null,
+        errors: null,
+        pageErrors,
+      }));
+
+    await expect(UnlocksHandler.create(ctx)).rejects.toThrow(
+      `Failed to start revision: ${JSON.stringify(pageErrors)}`
+    );
+
+    expect(mockUnlock).toHaveBeenCalledWith(revision.id, "en");
+  });
+});

package/src/views/article_revisions/codes/edit_form.ejs

@@ -15,12 +15,12 @@
           value: JSON.stringify(resource.settings?.codes?.scripts || undefined)
         }) %>
 
-        <%- f.field('resource.settings.codes.structured_data', {
+        <%- f.field('resource.settings.codes.structured_json', {
           label: 'Structured data (JSON)',
           as: 'textarea',
-          hint: 'Click here for a structured data verification tool',
+          hint: '<a target="_blank" href="https://search.google.com/test/rich-results">Click here for a structured data verification tool</a>',
           "data-validations-json": true,
-          value: JSON.stringify(resource.settings?.codes?.structured_data || undefined)
+          value: JSON.stringify(resource.settings?.codes?.structured_json || undefined)
         }) %>
 
         <div class="form-actions">

package/src/views/article_revisions/tags/edit_form.ejs

@@ -9,7 +9,7 @@
     <div>
       <% form_for({ resource }, errors, (f) => { %>
       <form action="<%= `/${_context.locale}/${_context.resourceConfig.articleType}_revisions/${resource.id}/tags` %>" method="POST" class="form">
-        <%- f.field('resource.tags', { as: 'checkbox_group', required: true, options: tags.map(tag => [tag.id, tag.name]) }) %>
+        <%- f.field('resource.tags', { as: 'checkbox_group', options: tags.map(tag => [tag.id, tag.name]) }) %>
 
         <div class="form-actions">
           <div class="btn-toolbar">

package/src/views/articles/_form.ejs

@@ -13,7 +13,7 @@
 
   <raketa-image-picker name="resource[settings][meta][image]" value="<%= JSON.stringify(resource.settings?.meta?.image || {}) %>" label="Meta image"></raketa-image-picker>
 
-  <%- f.field('resource.tags', { as: 'checkbox_group', required: true, options: tags.map(tag => [tag.id, tag.name]) }) %>
+  <%- f.field('resource.tags', { as: 'checkbox_group', options: tags.map(tag => [tag.id, tag.name]) }) %>
 
   <div class="form-actions">
     <div class="btn-toolbar">

package/src/views/articles/create_form.ejs

@@ -1,6 +1,6 @@
 <div class="page-wrapper">
   <div class="page-title">
-    <h1 class="heading-1">Create article</h1>
+    <h1 class="heading-1">Create <%= Inflector.downcase(_context.resourceConfig.articleType) %></h1>
   </div>
 
   <div class="grid gap-4 md:grid-cols-3">

package/src/views/articles/deletes/create_form.ejs

@@ -1,6 +1,6 @@
 <div class="page-wrapper">
   <div class="page-title">
-    <h1 class="heading-1">Delete an <%= _context.resourceConfig.articleType %></h1>
+    <h1 class="heading-1">Delete <%= Inflector.downcase(_context.resourceConfig.articleType) %></h1>
   </div>
 
   <div class="grid gap-4 md:grid-cols-3">

package/src/views/articles/translations/_form.ejs

@@ -18,7 +18,7 @@
   <h6>Original tags:</h6>
   <div><%= blueprint.tags.map((tag) => tag.name).join(", ")%></div>
 
-  <%- f.field('resource.tags', { as: 'checkbox_group', required: true, options: tags.map(tag => [tag.id, tag.name])}) %>
+  <%- f.field('resource.tags', { as: 'checkbox_group', options: tags.map(tag => [tag.id, tag.name])}) %>
 
   <div class="form-actions">
     <div class="btn-toolbar">

package/src/views/articles/translations/create_form.ejs

@@ -1,6 +1,6 @@
 <div class="page-wrapper">
   <div class="page-title">
-    <h1 class="heading-1">Create <%= _context.resourceConfig.articleType %> translation</h1>
+    <h1 class="heading-1">Create <%= Inflector.downcase(_context.resourceConfig.articleType) %> translation</h1>
   </div>
 
   <div class="grid gap-4 md:grid-cols-3">

package/src/views/event_revisions/codes/edit_form.ejs

@@ -15,12 +15,12 @@
           value: JSON.stringify(resource.settings?.codes?.scripts || undefined)
         }) %>
 
-        <%- f.field('resource.settings.codes.structured_data', {
+        <%- f.field('resource.settings.codes.structured_json', {
           label: 'Structured data (JSON)',
           as: 'textarea',
-          hint: 'Click here for a structured data verification tool',
+          hint: '<a target="_blank" href="https://search.google.com/test/rich-results">Click here for a structured data verification tool</a>',
           "data-validations-json": true,
-          value: JSON.stringify(resource.settings?.codes?.structured_data || undefined)
+          value: JSON.stringify(resource.settings?.codes?.structured_json || undefined)
         }) %>
 
         <div class="form-actions">

package/src/views/event_revisions/tags/edit_form.ejs

@@ -9,7 +9,7 @@
     <div>
       <% form_for({ resource }, errors, (f) => { %>
       <form action="<%= `/${_context.locale}/${_context.resourceConfig.eventType}_revisions/${resource.id}/tags` %>" method="POST" class="form">
-        <%- f.field('resource.tags', { as: 'checkbox_group', required: true, options: tags.map(tag => [tag.id, tag.name]) }) %>
+        <%- f.field('resource.tags', { as: 'checkbox_group', options: tags.map(tag => [tag.id, tag.name]) }) %>
 
         <div class="form-actions">
           <div class="btn-toolbar">

package/src/views/events/_form.ejs

@@ -15,7 +15,7 @@
 
   <raketa-image-picker name="resource[settings][meta][image]" value="<%= JSON.stringify(resource.settings?.meta?.image || {}) %>" label="Meta image"></raketa-image-picker>
 
-  <%- f.field('resource.tags', { as: 'checkbox_group', required: true, options: tags.map(tag => [tag.id, tag.name]) }) %>
+  <%- f.field('resource.tags', { as: 'checkbox_group', options: tags.map(tag => [tag.id, tag.name]) }) %>
 
   <div class="form-actions">
     <div class="btn-toolbar">

package/src/views/events/create_form.ejs

@@ -1,6 +1,6 @@
 <div class="page-wrapper">
   <div class="page-title">
-    <h1 class="heading-1">Create event</h1>
+    <h1 class="heading-1">Create <%= Inflector.downcase(_context.resourceConfig.eventType) %></h1>
   </div>
 
   <div class="grid gap-4 md:grid-cols-3">

package/src/views/events/deletes/create_form.ejs

@@ -1,6 +1,6 @@
 <div class="page-wrapper">
   <div class="page-title">
-    <h1 class="heading-1">Delete an <%= _context.resourceConfig.eventType %></h1>
+    <h1 class="heading-1">Delete <%= Inflector.downcase(_context.resourceConfig.eventType) %></h1>
   </div>
 
   <div class="grid gap-4 md:grid-cols-3">

package/src/views/events/translations/_form.ejs

@@ -20,7 +20,7 @@
   <h6>Original tags:</h6>
   <div><%= blueprint.tags.map((tag) => tag.name).join(", ")%></div>
 
-  <%- f.field('resource.tags', { as: 'checkbox_group', required: true, options: tags.map(tag => [tag.id, tag.name])}) %>
+  <%- f.field('resource.tags', { as: 'checkbox_group', options: tags.map(tag => [tag.id, tag.name])}) %>
 
   <div class="form-actions">
     <div class="btn-toolbar">

package/src/views/events/translations/create_form.ejs

@@ -1,6 +1,6 @@
 <div class="page-wrapper">
   <div class="page-title">
-    <h1 class="heading-1">Create <%= _context.resourceConfig.eventType %> translation</h1>
+    <h1 class="heading-1">Create <%= Inflector.downcase(_context.resourceConfig.eventType) %> translation</h1>
   </div>
 
   <div class="grid gap-4 md:grid-cols-3">

package/src/views/page_revisions/codes/edit_form.ejs

@@ -15,12 +15,12 @@
           value: JSON.stringify(resource.settings?.codes?.scripts || undefined)
         }) %>
 
-        <%- f.field('resource.settings.codes.structured_data', {
+        <%- f.field('resource.settings.codes.structured_json', {
           label: 'Structured data (JSON)',
           as: 'textarea',
-          hint: 'Click here for a structured data verification tool',
+          hint: '<a target="_blank" href="https://search.google.com/test/rich-results">Click here for a structured data verification tool</a>',
           "data-validations-json": true,
-          value: JSON.stringify(resource.settings?.codes?.structured_data || undefined)
+          value: JSON.stringify(resource.settings?.codes?.structured_json || undefined)
         }) %>
 
         <div class="form-actions">

package/src/views/pages/_form.ejs

@@ -12,7 +12,7 @@
 
   <raketa-image-picker name="resource[settings][meta][image]" value="<%= JSON.stringify(resource.settings?.meta?.image || {}) %>" label="Meta image"></raketa-image-picker>
 
-  <%- f.field('resource.tags', { as: 'checkbox_group', required: true, options: tags.map(tag => [tag.id, tag.name]) }) %>
+  <%- f.field('resource.tags', { as: 'checkbox_group', options: tags.map(tag => [tag.id, tag.name]) }) %>
 
   <div class="form-actions">
     <div class="btn-toolbar">

package/src/views/pages/create_form.ejs

@@ -1,6 +1,6 @@
 <div class="page-wrapper">
   <div class="page-title">
-    <h1 class="heading-1">Create page</h1>
+    <h1 class="heading-1">Create <%= Inflector.downcase(_context.resourceConfig.pageType) %></h1>
   </div>
 
   <div class="grid gap-4 md:grid-cols-3">

package/src/views/pages/deletes/create_form.ejs

@@ -1,6 +1,6 @@
 <div class="page-wrapper">
   <div class="page-title">
-    <h1 class="heading-1">Delete a <%= _context.resourceConfig.pageType %></h1>
+    <h1 class="heading-1">Delete <%= Inflector.downcase(_context.resourceConfig.pageType) %></h1>
   </div>
 
   <div class="grid gap-4 md:grid-cols-3">

package/src/views/pages/translations/_form.ejs

@@ -17,7 +17,7 @@
   <h6>Original tags:</h6>
   <div><%= blueprint.tags.map((tag) => tag.name).join(", ")%></div>
 
-  <%- f.field('resource.tags', { as: 'checkbox_group', required: true, options: tags.map(tag => [tag.id, tag.name]) }) %>
+  <%- f.field('resource.tags', { as: 'checkbox_group', options: tags.map(tag => [tag.id, tag.name]) }) %>
 
   <div class="form-actions">
     <div class="btn-toolbar">

package/src/views/pages/translations/create_form.ejs

@@ -1,6 +1,6 @@
 <div class="page-wrapper">
   <div class="page-title">
-    <h1 class="heading-1">Create <%= _context.resourceConfig.pageType %> translation</h1>
+    <h1 class="heading-1">Create <%= Inflector.downcase(_context.resourceConfig.pageType) %> translation</h1>
   </div>
 
   <div class="grid gap-4 md:grid-cols-3">