@raketa-cloud/admin 0.0.88 → 0.0.90

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

@@ -0,0 +1,105 @@
+import Tags from "../../../cloud/Tags";
+import TagsHandler from "../../../handlers/TagsHandler";
+import Factory from "../../support/factory";
+
+describe("Tags Handler", () => {
+  function buildCtx(tag) {
+    return {
+      client: {
+        tags: new Tags(() => ({}), "token"),
+      },
+      params: { id: tag.id },
+      resourceParams: {
+        tag: {
+          name: "Updated Tag",
+          slug: "updated-tag",
+        },
+      },
+      locale: "en",
+    };
+  }
+
+  test("update successful", async () => {
+    const tag = Factory.build("tag", {
+      id: "1",
+      name: "Original Tag",
+      slug: "original-tag",
+    });
+    const ctx = buildCtx(tag);
+
+    const mockedUpdate = jest
+      .spyOn(ctx.client.tags, "update")
+      .mockImplementation(() => ({
+        tag: { ...tag, name: "Updated Tag", slug: "updated-tag" },
+        errors: null,
+      }));
+
+    const result = await TagsHandler.update(ctx);
+
+    expect(mockedUpdate).toHaveBeenCalledWith(
+      "1",
+      {
+        name: "Updated Tag",
+        slug: "updated-tag",
+      },
+      "en"
+    );
+
+    expect(result).toEqual({
+      redirect: "/en/tags",
+    });
+  });
+
+  test("update failed", async () => {
+    const tag = Factory.build("tag", {
+      id: "1",
+      name: "Original Tag",
+      slug: "original-tag",
+    });
+    const ctx = buildCtx(tag);
+
+    const errors = [{ key: "name", message: "already taken" }];
+
+    const mockedUpdate = jest
+      .spyOn(ctx.client.tags, "update")
+      .mockImplementation(() => ({
+        tag: null,
+        errors,
+      }));
+
+    const mockedFormatErrors = jest
+      .spyOn(ctx.client.tags, "formatErrors")
+      .mockImplementation(() => ({
+        name: "already taken",
+      }));
+
+    const result = await TagsHandler.update(ctx);
+
+    expect(mockedUpdate).toHaveBeenCalledWith(
+      "1",
+      {
+        name: "Updated Tag",
+        slug: "updated-tag",
+      },
+      "en"
+    );
+
+    expect(mockedFormatErrors).toHaveBeenCalledWith(errors);
+
+    expect(result).toEqual({
+      assigns: {
+        tag: {
+          id: "1",
+          name: "Updated Tag",
+          slug: "updated-tag",
+        },
+        errors: {
+          tag: {
+            name: "already taken",
+          },
+        },
+        _translations: [],
+      },
+    });
+  });
+});

package/src/tests/handlers/articles/TranslationHandler.test/create.test.js

@@ -38,12 +38,14 @@ describe("Articles Translation Handler", () => {
       .mockImplementation(() => article);
 
     const translationErrors = [{ key: "title", message: "cannot be blank" }];
-    const mockTranslate = jest
-      .spyOn(ctx.client.articles, "translate")
-      .mockImplementation(() => ({
-        article: null,
-        errors: translationErrors,
-      }));
+    const mockTranslateBatch = jest
+      .spyOn(ctx.client.articles, "translateBatch")
+      .mockImplementation((parentId, locales) =>
+        locales.map((locale) => ({
+          article: null,
+          errors: translationErrors,
+        }))
+      );
 
     const mockFormatErrors = jest
       .spyOn(ctx.client.articles, "formatErrors")
@@ -52,7 +54,7 @@ describe("Articles Translation Handler", () => {
     const result = await TranslationsHandler.create(ctx);
 
     expect(mockFind).toHaveBeenCalledWith(article.id, "en");
-    expect(mockTranslate).toHaveBeenCalled();
+    expect(mockTranslateBatch).toHaveBeenCalled();
     expect(mockFormatErrors).toHaveBeenCalledWith(translationErrors);
 
     expect(result).toEqual({
@@ -92,35 +94,32 @@ describe("Articles Translation Handler", () => {
       .spyOn(ctx.client.articles, "find")
       .mockImplementation(() => article);
 
-    const mockTranslate = jest
-      .spyOn(ctx.client.articles, "translate")
-      .mockImplementation((parentId, locale) => ({
-        article: { id: `translated-article-${locale}` },
-        errors: null,
-      }));
-
-    const mockCreateRevision = jest
-      .spyOn(ctx.client.articleRevisions, "create")
-      .mockImplementation(() => ({ errors: null }));
+    const mockTranslateBatch = jest
+      .spyOn(ctx.client.articles, "translateBatch")
+      .mockImplementation((parentId, locales) =>
+        locales.map((locale) => ({
+          article: { id: `translated-article-${locale}` },
+          errors: null,
+        }))
+      );
+
+    const mockCreateRevisionBatch = jest
+      .spyOn(ctx.client.articleRevisions, "createBatch")
+      .mockImplementation((revisionInputs) =>
+        revisionInputs.map(({ locale }) => ({
+          revision: { id: `revision-${locale}` },
+          errors: null,
+        }))
+      );
 
     const result = await TranslationsHandler.create(ctx);
 
     expect(mockFind).toHaveBeenCalledWith(article.id, "en");
 
-    expect(mockTranslate).toHaveBeenCalledTimes(2);
-    expect(mockTranslate).toHaveBeenCalledWith(
-      article.id,
-      "fr",
-      expect.objectContaining({
-        title: article.title,
-        tags: [],
-        visibility: "private",
-        slug: article.slug,
-      })
-    );
-    expect(mockTranslate).toHaveBeenCalledWith(
+    expect(mockTranslateBatch).toHaveBeenCalledTimes(1);
+    expect(mockTranslateBatch).toHaveBeenCalledWith(
       article.id,
-      "es",
+      ["fr", "es"],
       expect.objectContaining({
         title: article.title,
         tags: [],
@@ -129,22 +128,18 @@ describe("Articles Translation Handler", () => {
       })
     );
 
-    expect(mockCreateRevision).toHaveBeenCalledTimes(2);
-    expect(mockCreateRevision).toHaveBeenCalledWith(
-      "translated-article-fr",
-      "fr"
-    );
-    expect(mockCreateRevision).toHaveBeenCalledWith(
-      "translated-article-es",
-      "es"
-    );
+    expect(mockCreateRevisionBatch).toHaveBeenCalledTimes(1);
+    expect(mockCreateRevisionBatch).toHaveBeenCalledWith([
+      { articleId: "translated-article-fr", locale: "fr" },
+      { articleId: "translated-article-es", locale: "es" },
+    ]);
 
     expect(result).toEqual({
       redirect: `/en/articles/${article.id}/translations?translations=fr:translated-article-fr;es:translated-article-es`,
     });
   });
 
-  test("create translations fails if any translation fails", async () => {
+  test("create translations if any translation fails", async () => {
     const article = Factory.build("article", {
       translations: {},
       categories: [{ id: "some-category-uno" }],
@@ -160,17 +155,22 @@ describe("Articles Translation Handler", () => {
       .spyOn(ctx.client.articles, "find")
       .mockImplementation(() => article);
 
-    const translationResponses = {
-      fr: { article: { id: "translated-article-fr" }, errors: null },
-      es: {
-        article: null,
-        errors: [{ key: "title", message: "invalid for spanish" }],
-      },
-    };
-
-    const mockTranslate = jest
-      .spyOn(ctx.client.articles, "translate")
-      .mockImplementation((parentId, locale) => translationResponses[locale]);
+    const mockTranslateBatch = jest
+      .spyOn(ctx.client.articles, "translateBatch")
+      .mockImplementation((parentId, locales) => [
+        {
+          article: {
+            id: "translated-article-fr",
+            slug: "article-revision",
+            visibility: "public",
+          },
+          errors: null,
+        },
+        {
+          article: null,
+          errors: [{ key: "title", message: "invalid for spanish" }],
+        },
+      ]);
 
     const mockFormatErrors = jest
       .spyOn(ctx.client.articles, "formatErrors")
@@ -186,20 +186,9 @@ describe("Articles Translation Handler", () => {
 
     expect(mockFind).toHaveBeenCalledWith(article.id, "en");
 
-    expect(mockTranslate).toHaveBeenCalledWith(
+    expect(mockTranslateBatch).toHaveBeenCalledWith(
       article.id,
-      "fr",
-      expect.objectContaining({
-        title: article.title,
-        tags: [],
-        visibility: "private",
-        slug: article.slug,
-      })
-    );
-
-    expect(mockTranslate).toHaveBeenCalledWith(
-      article.id,
-      "es",
+      ["fr", "es"],
       expect.objectContaining({
         title: article.title,
         tags: [],
@@ -219,7 +208,13 @@ describe("Articles Translation Handler", () => {
           widgets: article.widgets,
           categories: ["some-category-uno"],
           publishAt: article.publishAt,
-          translations: {},
+          translations: {
+            fr: {
+              id: "translated-article-fr",
+              slug: article.slug,
+              visibility: article.visibility,
+            },
+          },
           languages: ["fr", "es"],
         },
         blueprint: article,
@@ -244,17 +239,23 @@ describe("Articles Translation Handler", () => {
       .spyOn(ctx.client.articles, "find")
       .mockImplementation(() => article);
 
-    const mockTranslate = jest
-      .spyOn(ctx.client.articles, "translate")
-      .mockImplementation(() => ({
-        article: { id: "translated-article-id" },
-        errors: null,
-      }));
+    const mockTranslateBatch = jest
+      .spyOn(ctx.client.articles, "translateBatch")
+      .mockImplementation(() => [
+        {
+          article: {
+            id: "translated-article-fr",
+            slug: "article-revision",
+            visibility: "public",
+          },
+          errors: null,
+        },
+      ]);
 
     const revisionErrors = [{ key: "base", message: "something went wrong" }];
-    const mockCreateRevision = jest
-      .spyOn(ctx.client.articleRevisions, "create")
-      .mockImplementation(() => ({ errors: revisionErrors }));
+    const mockCreateRevisionBatch = jest
+      .spyOn(ctx.client.articleRevisions, "createBatch")
+      .mockImplementation(() => [{ errors: revisionErrors }]);
 
     const mockFormatErrors = jest
       .spyOn(ctx.client.articles, "formatErrors")
@@ -262,10 +263,9 @@ describe("Articles Translation Handler", () => {
 
     const result = await TranslationsHandler.create(ctx);
 
-    expect(mockCreateRevision).toHaveBeenCalledWith(
-      "translated-article-id",
-      "fr"
-    );
+    expect(mockCreateRevisionBatch).toHaveBeenCalledWith([
+      { articleId: "translated-article-fr", locale: "fr" },
+    ]);
     expect(mockFormatErrors).toHaveBeenCalledWith(revisionErrors);
 
     expect(result).toEqual({
@@ -279,7 +279,13 @@ describe("Articles Translation Handler", () => {
           widgets: article.widgets,
           categories: ["cat1"],
           publishAt: article.publishAt,
-          translations: {},
+          translations: {
+            fr: {
+              id: "translated-article-fr",
+              slug: article.slug,
+              visibility: article.visibility,
+            },
+          },
           languages: ["fr"],
         },
         blueprint: article,

package/src/tests/handlers/events/TranslationHandler.test/create.test.js

@@ -38,12 +38,14 @@ describe("Events Translation Handler", () => {
       .mockImplementation(() => event);
 
     const translationErrors = [{ key: "title", message: "cannot be blank" }];
-    const mockTranslate = jest
-      .spyOn(ctx.client.events, "translate")
-      .mockImplementation(() => ({
-        event: null,
-        errors: translationErrors,
-      }));
+    const mockTranslateBatch = jest
+      .spyOn(ctx.client.events, "translateBatch")
+      .mockImplementation((parentId, locales) =>
+        locales.map((locale) => ({
+          event: null,
+          errors: translationErrors,
+        }))
+      );
 
     const mockFormatErrors = jest
       .spyOn(ctx.client.events, "formatErrors")
@@ -52,7 +54,7 @@ describe("Events Translation Handler", () => {
     const result = await TranslationsHandler.create(ctx);
 
     expect(mockFind).toHaveBeenCalledWith(event.id, "en");
-    expect(mockTranslate).toHaveBeenCalled();
+    expect(mockTranslateBatch).toHaveBeenCalled();
     expect(mockFormatErrors).toHaveBeenCalledWith(translationErrors);
 
     expect(result).toEqual({
@@ -94,35 +96,32 @@ describe("Events Translation Handler", () => {
       .spyOn(ctx.client.events, "find")
       .mockImplementation(() => event);
 
-    const mockTranslate = jest
-      .spyOn(ctx.client.events, "translate")
-      .mockImplementation((parentId, locale) => ({
-        event: { id: `translated-event-${locale}` },
-        errors: null,
-      }));
-
-    const mockCreateRevision = jest
-      .spyOn(ctx.client.eventRevisions, "create")
-      .mockImplementation(() => ({ errors: null }));
+    const mockTranslateBatch = jest
+      .spyOn(ctx.client.events, "translateBatch")
+      .mockImplementation((parentId, locales) =>
+        locales.map((locale) => ({
+          event: { id: `translated-event-${locale}` },
+          errors: null,
+        }))
+      );
+
+    const mockCreateRevisionBatch = jest
+      .spyOn(ctx.client.eventRevisions, "createBatch")
+      .mockImplementation((revisionInputs) =>
+        revisionInputs.map(({ locale }) => ({
+          revision: { id: `revision-${locale}` },
+          errors: null,
+        }))
+      );
 
     const result = await TranslationsHandler.create(ctx);
 
     expect(mockFind).toHaveBeenCalledWith(event.id, "en");
 
-    expect(mockTranslate).toHaveBeenCalledTimes(2);
-    expect(mockTranslate).toHaveBeenCalledWith(
-      event.id,
-      "fr",
-      expect.objectContaining({
-        title: event.title,
-        tags: [],
-        visibility: "private",
-        slug: event.slug,
-      })
-    );
-    expect(mockTranslate).toHaveBeenCalledWith(
+    expect(mockTranslateBatch).toHaveBeenCalledTimes(1);
+    expect(mockTranslateBatch).toHaveBeenCalledWith(
       event.id,
-      "es",
+      ["fr", "es"],
       expect.objectContaining({
         title: event.title,
         tags: [],
@@ -131,15 +130,11 @@ describe("Events Translation Handler", () => {
       })
     );
 
-    expect(mockCreateRevision).toHaveBeenCalledTimes(2);
-    expect(mockCreateRevision).toHaveBeenCalledWith(
-      "translated-event-fr",
-      "fr"
-    );
-    expect(mockCreateRevision).toHaveBeenCalledWith(
-      "translated-event-es",
-      "es"
-    );
+    expect(mockCreateRevisionBatch).toHaveBeenCalledTimes(1);
+    expect(mockCreateRevisionBatch).toHaveBeenCalledWith([
+      { eventId: "translated-event-fr", locale: "fr" },
+      { eventId: "translated-event-es", locale: "es" },
+    ]);
 
     expect(result).toEqual({
       redirect: `/en/events/${event.id}/translations?translations=fr:translated-event-fr;es:translated-event-es`,
@@ -162,17 +157,22 @@ describe("Events Translation Handler", () => {
       .spyOn(ctx.client.events, "find")
       .mockImplementation(() => event);
 
-    const translationResponses = {
-      fr: { event: { id: "translated-event-fr" }, errors: null },
-      es: {
-        event: null,
-        errors: [{ key: "title", message: "invalid for spanish" }],
-      },
-    };
-
-    const mockTranslate = jest
-      .spyOn(ctx.client.events, "translate")
-      .mockImplementation((parentId, locale) => translationResponses[locale]);
+    const mockTranslateBatch = jest
+      .spyOn(ctx.client.events, "translateBatch")
+      .mockImplementation((parentId, locales) => [
+        {
+          event: {
+            id: "translated-event-fr",
+            slug: "event-revision",
+            visibility: "public",
+          },
+          errors: null,
+        },
+        {
+          event: null,
+          errors: [{ key: "title", message: "invalid for spanish" }],
+        },
+      ]);
 
     const mockFormatErrors = jest
       .spyOn(ctx.client.events, "formatErrors")
@@ -188,20 +188,9 @@ describe("Events Translation Handler", () => {
 
     expect(mockFind).toHaveBeenCalledWith(event.id, "en");
 
-    expect(mockTranslate).toHaveBeenCalledWith(
+    expect(mockTranslateBatch).toHaveBeenCalledWith(
       event.id,
-      "fr",
-      expect.objectContaining({
-        title: event.title,
-        tags: [],
-        visibility: "private",
-        slug: event.slug,
-      })
-    );
-
-    expect(mockTranslate).toHaveBeenCalledWith(
-      event.id,
-      "es",
+      ["fr", "es"],
       expect.objectContaining({
         title: event.title,
         tags: [],
@@ -223,7 +212,13 @@ describe("Events Translation Handler", () => {
           publishAt: event.publishAt,
           startsAt: event.startsAt,
           endsAt: event.endsAt,
-          translations: {},
+          translations: {
+            fr: {
+              id: "translated-event-fr",
+              slug: event.slug,
+              visibility: event.visibility,
+            },
+          },
           languages: ["fr", "es"],
         },
         blueprint: event,
@@ -248,17 +243,23 @@ describe("Events Translation Handler", () => {
       .spyOn(ctx.client.events, "find")
       .mockImplementation(() => event);
 
-    const mockTranslate = jest
-      .spyOn(ctx.client.events, "translate")
-      .mockImplementation(() => ({
-        event: { id: "translated-event-id" },
-        errors: null,
-      }));
+    const mockTranslateBatch = jest
+      .spyOn(ctx.client.events, "translateBatch")
+      .mockImplementation(() => [
+        {
+          event: {
+            id: "translated-event-fr",
+            slug: "event-revision",
+            visibility: "public",
+          },
+          errors: null,
+        },
+      ]);
 
     const revisionErrors = [{ key: "base", message: "something went wrong" }];
-    const mockCreateRevision = jest
-      .spyOn(ctx.client.eventRevisions, "create")
-      .mockImplementation(() => ({ errors: revisionErrors }));
+    const mockCreateRevisionBatch = jest
+      .spyOn(ctx.client.eventRevisions, "createBatch")
+      .mockImplementation(() => [{ errors: revisionErrors }]);
 
     const mockFormatErrors = jest
       .spyOn(ctx.client.events, "formatErrors")
@@ -266,10 +267,9 @@ describe("Events Translation Handler", () => {
 
     const result = await TranslationsHandler.create(ctx);
 
-    expect(mockCreateRevision).toHaveBeenCalledWith(
-      "translated-event-id",
-      "fr"
-    );
+    expect(mockCreateRevisionBatch).toHaveBeenCalledWith([
+      { eventId: "translated-event-fr", locale: "fr" },
+    ]);
     expect(mockFormatErrors).toHaveBeenCalledWith(revisionErrors);
 
     expect(result).toEqual({
@@ -285,7 +285,13 @@ describe("Events Translation Handler", () => {
           publishAt: event.publishAt,
           startsAt: event.startsAt,
           endsAt: event.endsAt,
-          translations: {},
+          translations: {
+            fr: {
+              id: "translated-event-fr",
+              slug: event.slug,
+              visibility: event.visibility,
+            },
+          },
           languages: ["fr"],
         },
         blueprint: event,