@raketa-cloud/admin 0.0.102 → 0.0.103

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

@@ -0,0 +1,297 @@
+import ArticleRevisions from "../../../../cloud/ArticleRevisions";
+import Articles from "../../../../cloud/Articles";
+import TranslationsHandler from "../../../../handlers/articles/TranslationsHandler";
+import Factory from "../../../support/factory";
+
+function buildCtx(params, article) {
+  return {
+    client: {
+      articles: new Articles(() => ({}), "token"),
+      articleRevisions: new ArticleRevisions(() => ({}), "token"),
+    },
+    params: { parent_id: article.id },
+    resourceParams: {
+      resource: params,
+    },
+    locale: "en",
+    resourceConfig: {
+      articleType: "article",
+      articleTypePlural: "articles",
+    },
+  };
+}
+
+describe("Articles Translation Handler", () => {
+  test("create fails with translation errors", async () => {
+    const article = Factory.build("article", {
+      translations: {},
+      categories: [{ id: "cat1" }],
+    });
+    const params = {
+      languages: ["fr"],
+    };
+
+    const ctx = buildCtx(params, article);
+
+    const mockFind = jest
+      .spyOn(ctx.client.articles, "find")
+      .mockImplementation(() => article);
+
+    const translationErrors = [{ key: "title", message: "cannot be blank" }];
+    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")
+      .mockImplementation(() => ({ title: "cannot be blank" }));
+
+    const result = await TranslationsHandler.create(ctx);
+
+    expect(mockFind).toHaveBeenCalledWith(article.id, "en");
+    expect(mockTranslateBatch).toHaveBeenCalled();
+    expect(mockFormatErrors).toHaveBeenCalledWith(translationErrors);
+
+    expect(result).toEqual({
+      assigns: {
+        resource: {
+          title: article.title,
+          tags: [],
+          visibility: "private",
+          slug: article.slug,
+          settings: article.settings,
+          widgets: article.widgets,
+          categories: ["cat1"],
+          publishAt: article.publishAt,
+          translations: {},
+          languages: ["fr"],
+        },
+        blueprint: article,
+        targetLocale: "fr",
+        errors: { resource: { title: "cannot be blank" } },
+      },
+    });
+  });
+
+  test("create successfully multiple translations with already one is existing", async () => {
+    const article = Factory.build("article", {
+      translations: { de: "de-id" },
+      categories: [{ id: "cat1" }],
+    });
+
+    const params = {
+      languages: ["fr", "es", "de"],
+    };
+
+    const ctx = buildCtx(params, article);
+
+    const mockFind = jest
+      .spyOn(ctx.client.articles, "find")
+      .mockImplementation(() => article);
+
+    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(mockTranslateBatch).toHaveBeenCalledTimes(1);
+    expect(mockTranslateBatch).toHaveBeenCalledWith(
+      article.id,
+      ["fr", "es"],
+      expect.objectContaining({
+        title: article.title,
+        tags: [],
+        visibility: "private",
+        slug: article.slug,
+      })
+    );
+
+    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 if any translation fails", async () => {
+    const article = Factory.build("article", {
+      translations: {},
+      categories: [{ id: "some-category-uno" }],
+    });
+
+    const params = {
+      languages: ["fr", "es"],
+    };
+
+    const ctx = buildCtx(params, article);
+
+    const mockFind = jest
+      .spyOn(ctx.client.articles, "find")
+      .mockImplementation(() => article);
+
+    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")
+      .mockImplementation(() => ({
+        title: "invalid for spanish",
+      }));
+
+    const mockCreateRevision = jest
+      .spyOn(ctx.client.articleRevisions, "create")
+      .mockImplementation(() => ({ errors: null }));
+
+    const result = await TranslationsHandler.create(ctx);
+
+    expect(mockFind).toHaveBeenCalledWith(article.id, "en");
+
+    expect(mockTranslateBatch).toHaveBeenCalledWith(
+      article.id,
+      ["fr", "es"],
+      expect.objectContaining({
+        title: article.title,
+        tags: [],
+        visibility: "private",
+        slug: article.slug,
+      })
+    );
+
+    expect(result).toEqual({
+      assigns: {
+        resource: {
+          title: article.title,
+          tags: [],
+          visibility: "private",
+          slug: article.slug,
+          settings: article.settings,
+          widgets: article.widgets,
+          categories: ["some-category-uno"],
+          publishAt: article.publishAt,
+          translations: {
+            fr: {
+              id: "translated-article-fr",
+              slug: article.slug,
+              visibility: article.visibility,
+            },
+          },
+          languages: ["fr", "es"],
+        },
+        blueprint: article,
+        targetLocale: "es",
+        errors: { resource: { title: "invalid for spanish" } },
+      },
+    });
+  });
+
+  test("create translation fails with revision errors", async () => {
+    const article = Factory.build("article", {
+      translations: {},
+      categories: [{ id: "cat1" }],
+    });
+    const params = {
+      languages: ["fr"],
+    };
+
+    const ctx = buildCtx(params, article);
+
+    const mockFind = jest
+      .spyOn(ctx.client.articles, "find")
+      .mockImplementation(() => article);
+
+    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 mockCreateRevisionBatch = jest
+      .spyOn(ctx.client.articleRevisions, "createBatch")
+      .mockImplementation(() => [{ errors: revisionErrors }]);
+
+    const mockFormatErrors = jest
+      .spyOn(ctx.client.articles, "formatErrors")
+      .mockImplementation(() => ({ base: "something went wrong" }));
+
+    const result = await TranslationsHandler.create(ctx);
+
+    expect(mockCreateRevisionBatch).toHaveBeenCalledWith([
+      { articleId: "translated-article-fr", locale: "fr" },
+    ]);
+    expect(mockFormatErrors).toHaveBeenCalledWith(revisionErrors);
+
+    expect(result).toEqual({
+      assigns: {
+        resource: {
+          title: article.title,
+          tags: [],
+          visibility: "private",
+          slug: article.slug,
+          settings: article.settings,
+          widgets: article.widgets,
+          categories: ["cat1"],
+          publishAt: article.publishAt,
+          translations: {
+            fr: {
+              id: "translated-article-fr",
+              slug: article.slug,
+              visibility: article.visibility,
+            },
+          },
+          languages: ["fr"],
+        },
+        blueprint: article,
+        targetLocale: "fr",
+        errors: { resource: { base: "something went wrong" } },
+      },
+    });
+  });
+});

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

@@ -0,0 +1,71 @@
+import Articles from "../../../../cloud/Articles";
+import TranslationsHandler from "../../../../handlers/articles/TranslationsHandler";
+import Factory from "../../../support/factory";
+
+function buildCtx(article) {
+  return {
+    client: {
+      articles: new Articles(() => ({}), "token"),
+    },
+    params: { parent_id: article.id },
+    query: { target_locale: "" },
+    locale: "en",
+    resourceConfig: {
+      articleType: "article",
+      articleTypePlural: "articles",
+    },
+  };
+}
+
+describe("Article translation form", () => {
+  test("create_form when a article exists with target locale", async () => {
+    const article = Factory.build("article");
+    const ctx = buildCtx(article);
+    ctx.query.target_locale = "fr";
+
+    const mockFindArticle = jest
+      .spyOn(ctx.client.articles, "find")
+      .mockImplementation(() => article);
+
+    const result = await TranslationsHandler.create_form(ctx);
+
+    expect(mockFindArticle).toHaveBeenCalledWith(article.id, "en");
+
+    const expectedResource = structuredClone(article);
+
+    expect(result).toEqual({
+      assigns: {
+        resource: expectedResource,
+        blueprint: article,
+        targetLocale: "fr",
+        errors: {},
+        _translations: [],
+      },
+    });
+  });
+
+  test("create_form when a article exists without target locale", async () => {
+    const article = Factory.build("article");
+    const ctx = buildCtx(article);
+
+    const mockFindArticle = jest
+      .spyOn(ctx.client.articles, "find")
+      .mockImplementation(() => article);
+
+    const result = await TranslationsHandler.create_form(ctx);
+
+    expect(mockFindArticle).toHaveBeenCalledWith(article.id, "en");
+
+    const expectedResource = structuredClone(article);
+
+    expect(result).toEqual({
+      assigns: {
+        resource: expectedResource,
+        blueprint: article,
+        targetLocale: "",
+        errors: {},
+        _translations: [],
+      },
+    });
+  });
+});

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

@@ -0,0 +1,91 @@
+import ArticleRevisions from "../../../../cloud/ArticleRevisions";
+import Articles from "../../../../cloud/Articles";
+import TranslationsHandler from "../../../../handlers/articles/TranslationsHandler";
+import Factory from "../../../support/factory";
+
+function buildCtx() {
+  const originalArticle = Factory.build("article", {
+    id: "original-article-id",
+    title: "Test Article Title",
+  });
+
+  return {
+    client: {
+      articles: new Articles(() => ({}), "token"),
+      articleRevisions: new ArticleRevisions(() => ({}), "token"),
+    },
+    params: { parent_id: originalArticle.id },
+    query: { translations: "fr:fr-article-id;es:es-article-id" },
+    locale: "en",
+    resourceConfig: {
+      articleType: "article",
+      articleTypePlural: "articles",
+    },
+    originalArticle,
+  };
+}
+
+describe("Translations Handler", () => {
+  test("index", async () => {
+    const frArticle = Factory.build("article", { id: "fr-article-id" });
+    const esArticle = Factory.build("article", { id: "es-article-id" });
+    const esRevision = Factory.build("article_revision", {
+      id: "es-revision-id",
+    });
+    const frRevision = Factory.build("article_revision", {
+      id: "fr-revision-id",
+    });
+    const ctx = buildCtx();
+
+    const mockedFind = jest
+      .spyOn(ctx.client.articles, "find")
+      .mockImplementation((id, locale) => {
+        if (locale === "fr") return frArticle;
+        if (locale === "es") return esArticle;
+        if (id === "original-article-id") return ctx.originalArticle;
+      });
+
+    const mockedRevisions = jest
+      .spyOn(ctx.client.articleRevisions, "all")
+      .mockImplementation((id, locale) => ({
+        list: locale === "fr" ? [frRevision] : [esRevision],
+      }));
+
+    const mockedList = jest
+      .spyOn(ctx.client.articles, "list")
+      .mockImplementation(() => ({}));
+
+    const result = await TranslationsHandler.index(ctx);
+
+    expect(mockedFind).toHaveBeenCalledWith("fr-article-id", "fr");
+    expect(mockedFind).toHaveBeenCalledWith("es-article-id", "es");
+    expect(mockedRevisions).toHaveBeenCalledWith("fr-article-id", "fr");
+    expect(mockedRevisions).toHaveBeenCalledWith("es-article-id", "es");
+    expect(mockedList).toHaveBeenCalledWith(
+      { translations: "fr:fr-article-id;es:es-article-id" },
+      "en",
+      "article"
+    );
+
+    expect(result).toEqual({
+      assigns: {
+        items: {},
+        visibilityOptions: ctx.client.articles.visibilityOptions(),
+        translations: [
+          {
+            ...frArticle,
+            translationLocale: "fr",
+            revisionId: "fr-revision-id",
+          },
+          {
+            ...esArticle,
+            translationLocale: "es",
+            revisionId: "es-revision-id",
+          },
+        ],
+        parent_id: "original-article-id",
+        articleTitle: "Test Article Title",
+      },
+    });
+  });
+});

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

@@ -0,0 +1,160 @@
+import URLsHandler from "../../../../handlers/articles/URLsHandler";
+import Articles from "../../../../cloud/Articles";
+import Factory from "../../../support/factory";
+import { remoteConfig } from "../../../support/handlersHelper";
+
+function buildCtx(article, updateSettings) {
+  return {
+    client: {
+      articles: new Articles(() => ({}), "token"),
+    },
+    params: { parent_id: article.id },
+    resourceParams: {
+      resource: updateSettings,
+    },
+    locale: "en",
+    remoteConfig: remoteConfig(),
+    resourceConfig: {
+      articleType: "article",
+      articleTypePlural: "articles",
+    },
+  };
+}
+
+describe("Article change url", () => {
+  test("create successful", async () => {
+    const article = Factory.build("article");
+    const updateSettings = {
+      slug: "new-slug",
+      redirect: "1",
+    };
+    const ctx = buildCtx(article, updateSettings);
+
+    const mockFind = jest
+      .spyOn(ctx.client.articles, "find")
+      .mockImplementation(() => article);
+
+    const mockChange = jest
+      .spyOn(ctx.client.articles, "updateSlug")
+      .mockImplementation(() => ({ article, errors: null }));
+
+    const result = await URLsHandler.create(ctx);
+
+    expect(mockFind).toHaveBeenCalledWith(article.id, "en");
+    expect(mockChange).toHaveBeenCalledWith(
+      article.id,
+      updateSettings.slug,
+      "en",
+      {
+        url: `/en/blog/${article.slug}`,
+        destination: `/en/blog/${updateSettings.slug}`,
+        httpResponseCode: 301,
+        note: "Created with changed URL for a resource of type article",
+      }
+    );
+
+    expect(result).toEqual({
+      redirect: `/en/articles/${article.id}`,
+    });
+  });
+
+  test("create failed due to slug error", async () => {
+    const article = Factory.build("article");
+    const updateSettings = {
+      slug: "new-slug",
+    };
+    const ctx = buildCtx(article, updateSettings);
+
+    const mockFind = jest
+      .spyOn(ctx.client.articles, "find")
+      .mockImplementation(() => article);
+
+    const mockChange = jest
+      .spyOn(ctx.client.articles, "updateSlug")
+      .mockImplementation(() => ({
+        article: null,
+        errors: [{ codes: [], key: "slug", message: "has already been taken" }],
+      }));
+
+    const result = await URLsHandler.create(ctx);
+
+    expect(mockFind).toHaveBeenCalledWith(article.id, "en");
+    expect(mockChange).toHaveBeenCalledWith(
+      article.id,
+      updateSettings.slug,
+      "en",
+      null
+    );
+
+    expect(result).toEqual({
+      assigns: {
+        resource: { ...updateSettings, id: article.id },
+        errors: { resource: { slug: "has already been taken" } },
+        _translations: [],
+      },
+    });
+  });
+
+  test("create failed due to redirect create", async () => {
+    const article = Factory.build("article");
+    const updateSettings = {
+      slug: "new-slug",
+      redirect: "1",
+    };
+    const ctx = buildCtx(article, updateSettings);
+
+    const mockFind = jest
+      .spyOn(ctx.client.articles, "find")
+      .mockImplementation(() => article);
+
+    const mockChange = jest
+      .spyOn(ctx.client.articles, "updateSlug")
+      .mockImplementation(() => ({
+        article: null,
+        errors: [
+          {
+            codes: [],
+            key: "generalError",
+            message: "url: has already been taken",
+          },
+        ],
+      }));
+
+    const result = await URLsHandler.create(ctx);
+
+    expect(mockFind).toHaveBeenCalledWith(article.id, "en");
+    expect(mockChange).toHaveBeenCalledWith(
+      article.id,
+      updateSettings.slug,
+      "en",
+      {
+        url: `/en/blog/${article.slug}`,
+        destination: `/en/blog/${updateSettings.slug}`,
+        httpResponseCode: 301,
+        note: "Created with changed URL for a resource of type article",
+      }
+    );
+
+    expect(result).toEqual({
+      assigns: {
+        resource: { ...updateSettings, id: article.id },
+        errors: { resource: { redirect: "url: has already been taken" } },
+        _translations: [],
+      },
+    });
+  });
+
+  test("create for non existing article", async () => {
+    const ctx = buildCtx({ id: "42" }, {});
+
+    const mockFind = jest
+      .spyOn(ctx.client.articles, "find")
+      .mockImplementation(() => null);
+
+    expect(async () => {
+      await URLsHandler.create(ctx);
+    }).rejects.toThrow("Cannot find article with id 42");
+
+    expect(mockFind).toHaveBeenCalledWith("42", "en");
+  });
+});

package/src/tests/handlers/articles/URLsHandler.test/create_form.test.js

@@ -0,0 +1,55 @@
+import Articles from "../../../../cloud/Articles";
+import URLsHandler from "../../../../handlers/articles/URLsHandler";
+import Factory from "../../../support/factory";
+
+function buildCtx(article) {
+  return {
+    client: {
+      articles: new Articles(() => ({}), "token"),
+    },
+    params: { parent_id: article.id },
+    locale: "en",
+    resourceConfig: {
+      articleType: "article",
+      articleTypePlural: "articles",
+    },
+  };
+}
+
+describe("Article change url", () => {
+  test("create_form for existing article", async () => {
+    const article = Factory.build("article");
+    const ctx = buildCtx(article);
+
+    const mockFind = jest
+      .spyOn(ctx.client.articles, "find")
+      .mockImplementation(() => article);
+
+    const result = await URLsHandler.create_form(ctx);
+
+    expect(mockFind).toHaveBeenCalledWith(article.id, "en");
+
+    expect(result).toEqual({
+      assigns: {
+        article,
+        resource: { slug: article.slug, id: article.id, redirect: null },
+        errors: {},
+        _translations: [],
+      },
+    });
+  });
+
+  test("create_form for non existing article", async () => {
+    const ctx = buildCtx({ id: "42" });
+
+    const mockFind = jest
+      .spyOn(ctx.client.articles, "find")
+      .mockImplementation(() => null);
+
+    expect(async () => {
+      await URLsHandler.create_form(ctx);
+    }).rejects.toThrow("Cannot find article with id 42");
+
+    expect(mockFind).toHaveBeenCalledWith("42", "en");
+  });
+});